Commit ac238dd23ff3c57d52405c63b8319985b0c7e9b8

Authored by Samir Aguiar
1 parent 58095dba

cosmetic changes

Fix some PEP8-related linter complaints.
tests/json/test_output.py
... ... @@ -26,7 +26,8 @@ class TestValidJson(unittest.TestCase):
26 26 of runs that didn't succeed.
27 27 """
28 28  
29   - def iter_test_files(self):
  29 + @staticmethod
  30 + def iter_test_files():
30 31 """ Iterate over all test files in DATA_BASE_DIR """
31 32 for dirpath, _, filenames in os.walk(DATA_BASE_DIR):
32 33 for filename in filenames:
... ... @@ -34,7 +35,6 @@ class TestValidJson(unittest.TestCase):
34 35  
35 36 def run_and_parse(self, program, args, print_output=False, check_return_code=True):
36 37 """ run single program with single file and parse output """
37   - return_code = None
38 38 with OutputCapture() as capturer: # capture stdout
39 39 try:
40 40 return_code = program(args)
... ...
tests/rtfobj/test_is_rtf.py
... ... @@ -60,8 +60,8 @@ class TestIsRtf(unittest.TestCase):
60 60 .format(full_path, expect))
61 61 with open(full_path, 'rb') as handle:
62 62 self.assertEqual(is_rtf(handle), expect,
63   - 'is_rtf(open({0})) did not return {1}'
64   - .format(full_path, expect))
  63 + 'is_rtf(open({0})) did not return {1}'
  64 + .format(full_path, expect))
65 65  
66 66  
67 67 # just in case somebody calls this file as a script
... ...