Commit 73c30669 by Ben Patterson Committed by GitHub

Merge pull request #14056 from edx/benp/fix-flaky-paver-test-pep8

Fix flaky paver test pep8
parents 1956e2cf 0faa4459
...@@ -15,7 +15,7 @@ class TestGetReportFiles(unittest.TestCase): ...@@ -15,7 +15,7 @@ class TestGetReportFiles(unittest.TestCase):
def test_get_pylint_reports(self, my_mock): def test_get_pylint_reports(self, my_mock):
my_mock.return_value = iter([ my_mock.return_value = iter([
('/foo', ('',), ('pylint.report',)), ('/foo', (None,), ('pylint.report',)),
('/bar', ('/baz',), ('pylint.report',)) ('/bar', ('/baz',), ('pylint.report',))
]) ])
reports = pavelib.quality.get_violations_reports("pylint") reports = pavelib.quality.get_violations_reports("pylint")
...@@ -24,7 +24,7 @@ class TestGetReportFiles(unittest.TestCase): ...@@ -24,7 +24,7 @@ class TestGetReportFiles(unittest.TestCase):
@patch('os.walk') @patch('os.walk')
def test_get_pep8_reports(self, my_mock): def test_get_pep8_reports(self, my_mock):
my_mock.return_value = iter([ my_mock.return_value = iter([
('/foo', ('',), ('pep8.report',)), ('/foo', (None,), ('pep8.report',)),
('/bar', ('/baz',), ('pep8.report',)) ('/bar', ('/baz',), ('pep8.report',))
]) ])
reports = pavelib.quality.get_violations_reports("pep8") reports = pavelib.quality.get_violations_reports("pep8")
...@@ -34,7 +34,7 @@ class TestGetReportFiles(unittest.TestCase): ...@@ -34,7 +34,7 @@ class TestGetReportFiles(unittest.TestCase):
def test_get_pep8_reports_noisy(self, my_mock): def test_get_pep8_reports_noisy(self, my_mock):
""" Several conditions: different report types, different files, multiple files """ """ Several conditions: different report types, different files, multiple files """
my_mock.return_value = iter([ my_mock.return_value = iter([
('/foo', ('',), ('pep8.report',)), ('/foo', (None,), ('pep8.report',)),
('/fooz', ('/ball',), ('pylint.report',)), ('/fooz', ('/ball',), ('pylint.report',)),
('/fooz', ('/ball',), ('non.report',)), ('/fooz', ('/ball',), ('non.report',)),
('/fooz', ('/ball',), ('lms.xml',)), ('/fooz', ('/ball',), ('lms.xml',)),
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment