diff options
author | Luis Lozano <llozano@chromium.org> | 2015-12-15 13:49:30 -0800 |
---|---|---|
committer | Luis Lozano <llozano@chromium.org> | 2015-12-16 17:36:06 +0000 |
commit | f2a3ef46f75d2196a93d3ed27f4d1fcf22b54fbe (patch) | |
tree | 185d243c7eed7c7a0db6f0e640746cadc1479ea9 /crosperf/crosperf_unittest.py | |
parent | 2a66f70fef907c1cb15229cb58e5129cb620ac98 (diff) | |
download | toolchain-utils-f2a3ef46f75d2196a93d3ed27f4d1fcf22b54fbe.tar.gz |
Run pyformat on all the toolchain-utils files.
This gets rid of a lot of lint issues.
Ran by doing this:
for f in *.py; do echo -n "$f " ; if [ -x $f ]; then pyformat -i
--remove_trailing_comma --yapf --force_quote_type=double $f ; else
pyformat -i --remove_shebang --remove_trailing_comma --yapf
--force_quote_type=double $f ; fi ; done
BUG=chromium:567921
TEST=Ran simple crosperf run.
Change-Id: I59778835fdaa5f706d2e1765924389f9e97433d1
Reviewed-on: https://chrome-internal-review.googlesource.com/242031
Reviewed-by: Luis Lozano <llozano@chromium.org>
Commit-Queue: Luis Lozano <llozano@chromium.org>
Tested-by: Luis Lozano <llozano@chromium.org>
Reviewed-by: Yunlian Jiang <yunlian@google.com>
Diffstat (limited to 'crosperf/crosperf_unittest.py')
-rwxr-xr-x | crosperf/crosperf_unittest.py | 28 |
1 files changed, 14 insertions, 14 deletions
diff --git a/crosperf/crosperf_unittest.py b/crosperf/crosperf_unittest.py index 082d8a6c..42a78ef8 100755 --- a/crosperf/crosperf_unittest.py +++ b/crosperf/crosperf_unittest.py @@ -1,14 +1,12 @@ #!/usr/bin/python # # Copyright 2014 Google Inc. All Rights Reserved. - """Unittest for crosperf.""" import atexit import os import optparse import StringIO - import mock import unittest @@ -40,6 +38,7 @@ EXPERIMENT_FILE_1 = """ } """ + class CrosperfTest(unittest.TestCase): def setUp(self): @@ -51,27 +50,28 @@ class CrosperfTest(unittest.TestCase): parser = optparse.OptionParser(usage=Help().GetUsage(), description=Help().GetHelp(), formatter=crosperf.MyIndentedHelpFormatter(), - version="%prog 3.0") - parser.add_option("-l", "--log_dir", - dest="log_dir", - default="", - help="The log_dir, default is under <crosperf_logs>/logs") + version='%prog 3.0') + parser.add_option('-l', + '--log_dir', + dest='log_dir', + default='', + help='The log_dir, default is under <crosperf_logs>/logs') options_before = parser._get_all_options() self.assertEqual(len(options_before), 3) crosperf.SetupParserOptions(parser) options_after = parser._get_all_options() self.assertEqual(len(options_after), 29) - def test_convert_options_to_settings(self): parser = optparse.OptionParser(usage=Help().GetUsage(), description=Help().GetHelp(), formatter=crosperf.MyIndentedHelpFormatter(), - version="%prog 3.0") - parser.add_option("-l", "--log_dir", - dest="log_dir", - default="", - help="The log_dir, default is under <crosperf_logs>/logs") + version='%prog 3.0') + parser.add_option('-l', + '--log_dir', + dest='log_dir', + default='', + help='The log_dir, default is under <crosperf_logs>/logs') crosperf.SetupParserOptions(parser) argv = ['crosperf/crosperf.py', 'temp.exp', '--rerun=True'] options, args = parser.parse_args(argv) @@ -86,5 +86,5 @@ class CrosperfTest(unittest.TestCase): self.assertFalse(settings.GetField('rerun')) -if __name__ == "__main__": +if __name__ == '__main__': unittest.main() |