[CLN] more lint fixes

This commit is contained in:
Benedek Racz 2020-02-19 10:20:56 +01:00
parent 4e49107b3c
commit f6ff2fcf4d
3 changed files with 9 additions and 9 deletions

14
tox.ini
View File

@ -1,5 +1,5 @@
# tox.ini
# Run tests of wexpect in multiple configuration.
# Run tests of wexpect in multiple configuration.
[tox]
# The following configuration will run automatically.
@ -12,17 +12,17 @@ description = Unit tests
setenv =
spawn_pipe: WEXPECT_SPAWN_CLASS=SpawnPipe
legacy_wexpect: WEXPECT_SPAWN_CLASS=legacy_wexpect
commands =
# install the dependencies:
pip install .[test]
# Run the test itself
coverage run -m unittest
# Dump coverage summary to console
coverage report --omit tests/*
coverage report --omit=tests/*,site-packages/*
# Convert coverage report to standard xml formula the filename includes the tox environment name
# https://tox.readthedocs.io/en/latest/config.html#environment-variable-substitutions
coverage xml --omit tests/* -o {env:TOX_ENV_NAME}_coverage.xml
coverage xml --omit=tests/*,site-packages -o {env:TOX_ENV_NAME}_coverage.xml

View File

@ -646,7 +646,7 @@ class SpawnBase:
compiled_pattern_list.append(EOF)
elif p is TIMEOUT:
compiled_pattern_list.append(TIMEOUT)
elif type(p) is type(re.compile('')):
elif isinstance(p, type(re.compile(''))):
compiled_pattern_list.append(p)
else:
logger.warning(

View File

@ -145,7 +145,7 @@ def join_args(args):
spaces or any of the characters ^!$%&()[]{}=;'+,`~"""
commandline = []
for arg in args:
if re.search('[\^!$%&()[\]{}=;\'+,`~\s]', arg):
if re.search('[\\^!$%&()[\\]{}=;\'+,`~\\s]', arg):
arg = '"%s"' % arg
commandline.append(arg)
return ' '.join(commandline)