mirror of
https://git.launchpad.net/~ubuntu-release/britney/+git/britney2-ubuntu
synced 2025-06-15 11:41:29 +00:00
Accept all known issues (by file)
Signed-off-by: Niels Thykier <niels@thykier.net>
This commit is contained in:
parent
bb699f7416
commit
dc820756cf
@ -563,4 +563,3 @@ class TestInstTester(unittest.TestCase):
|
|||||||
|
|
||||||
if __name__ == '__main__':
|
if __name__ == '__main__':
|
||||||
unittest.main()
|
unittest.main()
|
||||||
|
|
||||||
|
@ -3,13 +3,60 @@ import unittest
|
|||||||
import pycodestyle
|
import pycodestyle
|
||||||
|
|
||||||
|
|
||||||
|
EXCEPTIONS_BY_FILE = {
|
||||||
|
'britney.py': 36,
|
||||||
|
'britney2/__init__.py': 2,
|
||||||
|
'britney2/excuse.py': 5,
|
||||||
|
'britney2/hints.py': 8,
|
||||||
|
'britney2/installability/tester.py': 4,
|
||||||
|
'britney2/policies/__init__.py': 2,
|
||||||
|
'britney2/policies/policy.py': 27,
|
||||||
|
'britney2/policies/autopkgtest.py': 9,
|
||||||
|
'tests/mock_swift.py': 2,
|
||||||
|
'tests/__init__.py': 31,
|
||||||
|
'tests/test_autopkgtest.py': 338,
|
||||||
|
'tests/test_policy.py': 31,
|
||||||
|
'tests/test_yaml.py': 1,
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
def _on_error(e):
|
||||||
|
raise e
|
||||||
|
|
||||||
|
|
||||||
|
def all_python_files(project_dir):
|
||||||
|
|
||||||
|
for basedir, subdirs, files in os.walk(project_dir, onerror=_on_error):
|
||||||
|
if basedir == project_dir:
|
||||||
|
if '.git' in subdirs:
|
||||||
|
subdirs.remove('.git')
|
||||||
|
if 'doc' in subdirs:
|
||||||
|
subdirs.remove('doc')
|
||||||
|
|
||||||
|
subdirs.sort()
|
||||||
|
files.sort()
|
||||||
|
|
||||||
|
for file in files:
|
||||||
|
if file.endswith('.py'):
|
||||||
|
path = os.path.join(basedir, file)
|
||||||
|
name = path[len(project_dir)+1:]
|
||||||
|
yield path, name
|
||||||
|
|
||||||
|
|
||||||
class TestCodeFormat(unittest.TestCase):
|
class TestCodeFormat(unittest.TestCase):
|
||||||
|
|
||||||
def test_conformance(self):
|
def test_conformance(self):
|
||||||
"""Test that we conform to PEP-8."""
|
"""Test that we conform to PEP-8."""
|
||||||
project_dir = os.path.dirname(os.path.dirname(__file__))
|
project_dir = os.path.dirname(os.path.dirname(__file__))
|
||||||
codestyle_cfg = os.path.join(project_dir, 'setup.cfg')
|
codestyle_cfg = os.path.join(project_dir, 'setup.cfg')
|
||||||
style = pycodestyle.StyleGuide(config_file=codestyle_cfg)
|
for python_file, name in all_python_files(project_dir):
|
||||||
result = style.check_files('.')
|
options = {
|
||||||
self.assertEqual(result.total_errors, 0,
|
'config_file': codestyle_cfg,
|
||||||
"Found code style errors (and warnings).")
|
}
|
||||||
|
style = pycodestyle.StyleGuide(**options)
|
||||||
|
result = style.input_file(python_file)
|
||||||
|
limit = EXCEPTIONS_BY_FILE.get(name, 0)
|
||||||
|
# The number are the "unfixed" errors at time of introduction.
|
||||||
|
# As we fix them, this number should be reduced.
|
||||||
|
self.assertEqual(result, limit,
|
||||||
|
"Found code style errors (and warnings) in %s (%s)." % (name, python_file))
|
||||||
|
Loading…
x
Reference in New Issue
Block a user