4 # Copyright (C) 2020 Red Hat, Inc.
6 # This program is free software; you can redistribute it and/or modify
7 # it under the terms of the GNU General Public License as published by
8 # the Free Software Foundation; either version 2 of the License, or
9 # (at your option) any later version.
11 # This program is distributed in the hope that it will be useful,
12 # but WITHOUT ANY WARRANTY; without even the implied warranty of
13 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
14 # GNU General Public License for more details.
16 # You should have received a copy of the GNU General Public License
17 # along with this program. If not, see <http://www.gnu.org/licenses/>.
28 # TODO: Empty this list!
30 '030', '040', '041', '044', '045', '055', '056', '057', '065', '093',
31 '096', '118', '124', '132', '136', '139', '147', '148', '149',
32 '151', '152', '155', '163', '165', '194', '196', '202',
33 '203', '205', '206', '207', '208', '210', '211', '212', '213', '216',
34 '218', '219', '224', '228', '234', '235', '236', '237', '238',
35 '240', '242', '245', '246', '248', '255', '256', '257', '258', '260',
36 '262', '264', '266', '274', '277', '280', '281', '295', '296', '298',
37 '299', '302', '303', '304', '307',
38 'nbd-fault-injector.py', 'qcow2.py', 'qcow2_format.py', 'qed.py'
42 def is_python_file(filename):
43 if not os.path.isfile(filename):
46 if filename.endswith('.py'):
49 with open(filename, encoding='utf-8') as f:
51 first_line = f.readline()
52 return re.match('^#!.*python', first_line) is not None
53 except UnicodeDecodeError: # Ignore binary files
58 named_tests = [f'tests/{entry}' for entry in os.listdir('tests')]
59 check_tests = set(os.listdir('.') + named_tests) - set(SKIP_FILES)
60 files = [filename for filename in check_tests if is_python_file(filename)]
62 iotests.logger.debug('Files to be checked:')
63 iotests.logger.debug(', '.join(sorted(files)))
65 print('=== pylint ===')
68 # Todo notes are fine, but fixme's or xxx's should probably just be
69 # fixed (in tests, at least)
70 env = os.environ.copy()
71 subprocess.run(('pylint-3', '--score=n', '--notes=FIXME,XXX', *files),
77 # We have to call mypy separately for each file. Otherwise, it
78 # will interpret all given files as belonging together (i.e., they
79 # may not both define the same classes, etc.; most notably, they
80 # must not both define the __main__ module).
81 env['MYPYPATH'] = env['PYTHONPATH']
82 for filename in files:
83 p = subprocess.run(('mypy',
84 '--warn-unused-configs',
85 '--disallow-subclassing-any',
86 '--disallow-any-generics',
87 '--disallow-incomplete-defs',
88 '--disallow-untyped-decorators',
89 '--no-implicit-optional',
90 '--warn-redundant-casts',
91 '--warn-unused-ignores',
92 '--no-implicit-reexport',
93 '--namespace-packages',
97 stdout=subprocess.PIPE,
98 stderr=subprocess.STDOUT,
99 universal_newlines=True)
101 if p.returncode != 0:
105 for linter in ('pylint-3', 'mypy'):
106 if shutil.which(linter) is None:
107 iotests.notrun(f'{linter} not found')
109 iotests.script_main(run_linters)