Commit eee8b05c by Jusufadis Bakamovic Committed by Roman Lebedev

[tools] Run autopep8 and apply fixes found. (#739)

parent eafa34a5
#!/usr/bin/env python #!/usr/bin/env python
import unittest
""" """
compare.py - versatile benchmark output compare tool compare.py - versatile benchmark output compare tool
""" """
...@@ -244,9 +245,6 @@ def main(): ...@@ -244,9 +245,6 @@ def main():
print(ln) print(ln)
import unittest
class TestParser(unittest.TestCase): class TestParser(unittest.TestCase):
def setUp(self): def setUp(self):
self.parser = create_parser() self.parser = create_parser()
...@@ -402,7 +400,7 @@ class TestParser(unittest.TestCase): ...@@ -402,7 +400,7 @@ class TestParser(unittest.TestCase):
if __name__ == '__main__': if __name__ == '__main__':
#unittest.main() # unittest.main()
main() main()
# vim: tabstop=4 expandtab shiftwidth=4 softtabstop=4 # vim: tabstop=4 expandtab shiftwidth=4 softtabstop=4
......
import unittest
"""report.py - Utilities for reporting statistics about benchmark results """report.py - Utilities for reporting statistics about benchmark results
""" """
import os import os
...@@ -270,9 +271,6 @@ def generate_difference_report( ...@@ -270,9 +271,6 @@ def generate_difference_report(
# Unit tests # Unit tests
import unittest
class TestGetUniqueBenchmarkNames(unittest.TestCase): class TestGetUniqueBenchmarkNames(unittest.TestCase):
def load_results(self): def load_results(self):
import json import json
...@@ -300,6 +298,7 @@ class TestGetUniqueBenchmarkNames(unittest.TestCase): ...@@ -300,6 +298,7 @@ class TestGetUniqueBenchmarkNames(unittest.TestCase):
for i in range(0, len(output_lines)): for i in range(0, len(output_lines)):
self.assertEqual(expect_lines[i], output_lines[i]) self.assertEqual(expect_lines[i], output_lines[i])
class TestReportDifference(unittest.TestCase): class TestReportDifference(unittest.TestCase):
def load_results(self): def load_results(self):
import json import json
......
...@@ -12,6 +12,8 @@ IT_JSON = 1 ...@@ -12,6 +12,8 @@ IT_JSON = 1
IT_Executable = 2 IT_Executable = 2
_num_magic_bytes = 2 if sys.platform.startswith('win') else 4 _num_magic_bytes = 2 if sys.platform.startswith('win') else 4
def is_executable_file(filename): def is_executable_file(filename):
""" """
Return 'True' if 'filename' names a valid file which is likely Return 'True' if 'filename' names a valid file which is likely
...@@ -46,7 +48,7 @@ def is_json_file(filename): ...@@ -46,7 +48,7 @@ def is_json_file(filename):
with open(filename, 'r') as f: with open(filename, 'r') as f:
json.load(f) json.load(f)
return True return True
except: except BaseException:
pass pass
return False return False
...@@ -84,6 +86,7 @@ def check_input_file(filename): ...@@ -84,6 +86,7 @@ def check_input_file(filename):
sys.exit(1) sys.exit(1)
return ftype return ftype
def find_benchmark_flag(prefix, benchmark_flags): def find_benchmark_flag(prefix, benchmark_flags):
""" """
Search the specified list of flags for a flag matching `<prefix><arg>` and Search the specified list of flags for a flag matching `<prefix><arg>` and
...@@ -97,6 +100,7 @@ def find_benchmark_flag(prefix, benchmark_flags): ...@@ -97,6 +100,7 @@ def find_benchmark_flag(prefix, benchmark_flags):
result = f[len(prefix):] result = f[len(prefix):]
return result return result
def remove_benchmark_flags(prefix, benchmark_flags): def remove_benchmark_flags(prefix, benchmark_flags):
""" """
Return a new list containing the specified benchmark_flags except those Return a new list containing the specified benchmark_flags except those
...@@ -105,6 +109,7 @@ def remove_benchmark_flags(prefix, benchmark_flags): ...@@ -105,6 +109,7 @@ def remove_benchmark_flags(prefix, benchmark_flags):
assert prefix.startswith('--') and prefix.endswith('=') assert prefix.startswith('--') and prefix.endswith('=')
return [f for f in benchmark_flags if not f.startswith(prefix)] return [f for f in benchmark_flags if not f.startswith(prefix)]
def load_benchmark_results(fname): def load_benchmark_results(fname):
""" """
Read benchmark output from a file and return the JSON object. Read benchmark output from a file and return the JSON object.
......
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