mirror of
https://github.com/xemu-project/xemu.git
synced 2024-11-28 14:00:44 +00:00
iotests: Fix unspecified-encoding pylint warnings
As of recently, pylint complains when `open()` calls are missing an `encoding=` specified. Everything we have should be UTF-8 (and in fact, everything should be UTF-8, period (exceptions apply)), so use that. Signed-off-by: Hanna Reitz <hreitz@redhat.com> Message-Id: <20210824153540.177128-2-hreitz@redhat.com> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com> Reviewed-by: John Snow <jsnow@redhat.com>
This commit is contained in:
parent
9dbf6455f4
commit
81dcb9ca1f
@ -46,7 +46,7 @@ def is_python_file(filename):
|
||||
if filename.endswith('.py'):
|
||||
return True
|
||||
|
||||
with open(filename) as f:
|
||||
with open(filename, encoding='utf-8') as f:
|
||||
try:
|
||||
first_line = f.readline()
|
||||
return re.match('^#!.*python', first_line) is not None
|
||||
|
@ -610,7 +610,7 @@ class VM(qtest.QEMUQtestMachine):
|
||||
return
|
||||
valgrind_filename = f"{test_dir}/{self._popen.pid}.valgrind"
|
||||
if self.exitcode() == 99:
|
||||
with open(valgrind_filename) as f:
|
||||
with open(valgrind_filename, encoding='utf-8') as f:
|
||||
print(f.read())
|
||||
else:
|
||||
os.remove(valgrind_filename)
|
||||
@ -1121,7 +1121,8 @@ def notrun(reason):
|
||||
# Each test in qemu-iotests has a number ("seq")
|
||||
seq = os.path.basename(sys.argv[0])
|
||||
|
||||
with open('%s/%s.notrun' % (output_dir, seq), 'w') as outfile:
|
||||
with open('%s/%s.notrun' % (output_dir, seq), 'w', encoding='utf-8') \
|
||||
as outfile:
|
||||
outfile.write(reason + '\n')
|
||||
logger.warning("%s not run: %s", seq, reason)
|
||||
sys.exit(0)
|
||||
@ -1135,7 +1136,8 @@ def case_notrun(reason):
|
||||
# Each test in qemu-iotests has a number ("seq")
|
||||
seq = os.path.basename(sys.argv[0])
|
||||
|
||||
with open('%s/%s.casenotrun' % (output_dir, seq), 'a') as outfile:
|
||||
with open('%s/%s.casenotrun' % (output_dir, seq), 'a', encoding='utf-8') \
|
||||
as outfile:
|
||||
outfile.write(' [case not run] ' + reason + '\n')
|
||||
|
||||
def _verify_image_format(supported_fmts: Sequence[str] = (),
|
||||
|
Loading…
Reference in New Issue
Block a user