Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1628)

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/style/optparser_unittest.py

Issue 1839193004: Run auto-formatter (autopep8) on webkitpy. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Tools/Scripts/webkitpy/style/optparser_unittest.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/style/optparser_unittest.py b/third_party/WebKit/Tools/Scripts/webkitpy/style/optparser_unittest.py
index d72479fc5e487a410b4f007a0aaba483a1f426cc..713a62e614d40de6ff9b7d8193257e39023b9f7c 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/style/optparser_unittest.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/style/optparser_unittest.py
@@ -50,14 +50,14 @@ class ArgumentPrinterTest(unittest.TestCase):
def test_to_flag_string(self):
options = self._create_options('vs7', 5, ['+foo', '-bar'], 'git')
self.assertEqual('--filter=+foo,-bar --git-commit=git '
- '--min-confidence=5 --output=vs7',
- self._printer.to_flag_string(options))
+ '--min-confidence=5 --output=vs7',
+ self._printer.to_flag_string(options))
# This is to check that --filter and --git-commit do not
# show up when not user-specified.
options = self._create_options()
self.assertEqual('--min-confidence=3 --output=emacs',
- self._printer.to_flag_string(options))
+ self._printer.to_flag_string(options))
class ArgumentParserTest(LoggingTestCase):
@@ -86,7 +86,7 @@ class ArgumentParserTest(LoggingTestCase):
"""Return an ArgumentParser instance for testing."""
default_options = self._create_defaults()
- all_categories = ["build" ,"whitespace"]
+ all_categories = ["build", "whitespace"]
mock_stderr = self._MockStdErr()
@@ -123,19 +123,19 @@ class ArgumentParserTest(LoggingTestCase):
self.assertRaises(SystemExit, parse, ['--min-confidence=6'])
self.assertLog(['ERROR: option --min-confidence: invalid integer: 6: '
'value must be between 1 and 5\n'])
- parse(['--min-confidence=1']) # works
- parse(['--min-confidence=5']) # works
+ parse(['--min-confidence=1']) # works
+ parse(['--min-confidence=5']) # works
self.assertRaises(SystemExit, parse, ['--output=bad'])
self.assertLog(['ERROR: option --output-format: invalid choice: '
"'bad' (choose from 'emacs', 'vs7')\n"])
- parse(['--output=vs7']) # works
+ parse(['--output=vs7']) # works
# Pass a filter rule not beginning with + or -.
self.assertRaises(SystemExit, parse, ['--filter=build'])
self.assertLog(['ERROR: Invalid filter rule "build": '
'every rule must start with + or -.\n'])
- parse(['--filter=+build']) # works
+ parse(['--filter=+build']) # works
def test_parse_default_arguments(self):
parse = self._parse
@@ -173,12 +173,12 @@ class ArgumentParserTest(LoggingTestCase):
# Pass user_rules.
(files, options) = parse(['--filter=+build,-whitespace'])
self.assertEqual(options.filter_rules,
- ["+build", "-whitespace"])
+ ["+build", "-whitespace"])
# Pass spurious white space in user rules.
(files, options) = parse(['--filter=+build, -whitespace'])
self.assertEqual(options.filter_rules,
- ["+build", "-whitespace"])
+ ["+build", "-whitespace"])
def test_parse_files(self):
parse = self._parse
@@ -207,12 +207,12 @@ class CommandOptionValuesTest(unittest.TestCase):
# Check argument validation.
self.assertRaises(ValueError, ProcessorOptions, output_format="bad")
- ProcessorOptions(output_format="emacs") # No ValueError: works
- ProcessorOptions(output_format="vs7") # works
+ ProcessorOptions(output_format="emacs") # No ValueError: works
+ ProcessorOptions(output_format="vs7") # works
self.assertRaises(ValueError, ProcessorOptions, min_confidence=0)
self.assertRaises(ValueError, ProcessorOptions, min_confidence=6)
- ProcessorOptions(min_confidence=1) # works
- ProcessorOptions(min_confidence=5) # works
+ ProcessorOptions(min_confidence=1) # works
+ ProcessorOptions(min_confidence=5) # works
# Check attributes.
options = ProcessorOptions(filter_rules=["+"],
@@ -255,4 +255,3 @@ class CommandOptionValuesTest(unittest.TestCase):
# Thus, just check the distinguishing case to verify that the
# code defines __ne__.
self.assertFalse(ProcessorOptions().__ne__(ProcessorOptions()))
-

Powered by Google App Engine
This is Rietveld 408576698