Run auto-formatter on files in webkitpy/layout_tests/.

third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/test.py

 # Copyright (C) 2010 Google Inc. All rights reserved.
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
 # met:
 #
 #     * Redistributions of source code must retain the above copyright
 # notice, this list of conditions and the following disclaimer.
 #     * Redistributions in binary form must reproduce the above
 # copyright notice, this list of conditions and the following disclaimer
@@ skipping 91 matching lines @@
 
 #
 # These numbers may need to be updated whenever we add or delete tests. This includes virtual tests.
 #
 TOTAL_TESTS = 114
 TOTAL_SKIPS = 26
 TOTAL_CRASHES = 80
 UNEXPECTED_PASSES = 1
 UNEXPECTED_FAILURES = 26
 
+
 def unit_test_list():
     tests = TestList()
     tests.add('failures/expected/crash.html', crash=True)
     tests.add('failures/expected/exception.html', exception=True)
     tests.add('failures/expected/device_failure.html', device_failure=True)
     tests.add('failures/expected/timeout.html', timeout=True)
     tests.add('failures/expected/leak.html', leak=True)
     tests.add('failures/expected/missing_text.html', expected_text=None)
     tests.add('failures/expected/needsrebaseline.html', actual_text='needsrebaseline text')
     tests.add('failures/expected/needsmanualrebaseline.html', actual_text='needsmanualrebaseline text')
     tests.add('failures/expected/image.html',
               actual_image='image_fail-pngtEXtchecksum\x00checksum_fail',
               expected_image='image-pngtEXtchecksum\x00checksum-png')
     tests.add('failures/expected/image_checksum.html',
               actual_checksum='image_checksum_fail-checksum',
               actual_image='image_checksum_fail-png')
     tests.add('failures/expected/audio.html',
-              actual_audio=base64.b64encode('audio_fail-wav'), expected_audio='audio-wav',
-              actual_text=None, expected_text=None,
-              actual_image=None, expected_image=None,
+              actual_audio=base64.b64encode('audio_fail-wav'),
+              expected_audio='audio-wav',
+              actual_text=None,
+              expected_text=None,
+              actual_image=None,
+              expected_image=None,
               actual_checksum=None)
     tests.add('failures/expected/keyboard.html', keyboard=True)
-    tests.add('failures/expected/missing_check.html',
-              expected_image='missing_check-png')
+    tests.add('failures/expected/missing_check.html', expected_image='missing_check-png')
     tests.add('failures/expected/missing_image.html', expected_image=None)
-    tests.add('failures/expected/missing_audio.html', expected_audio=None,
-              actual_text=None, expected_text=None,
-              actual_image=None, expected_image=None,
+    tests.add('failures/expected/missing_audio.html',
+              expected_audio=None,
+              actual_text=None,
+              expected_text=None,
+              actual_image=None,
+              expected_image=None,
               actual_checksum=None)
     tests.add('failures/expected/missing_text.html', expected_text=None)
-    tests.add('failures/expected/newlines_leading.html',
-              expected_text="\nfoo\n", actual_text="foo\n")
-    tests.add('failures/expected/newlines_trailing.html',
-              expected_text="foo\n\n", actual_text="foo\n")
-    tests.add('failures/expected/newlines_with_excess_CR.html',
-              expected_text="foo\r\r\r\n", actual_text="foo\n")
+    tests.add('failures/expected/newlines_leading.html', expected_text="\nfoo\n", actual_text="foo\n")
+    tests.add('failures/expected/newlines_trailing.html', expected_text="foo\n\n", actual_text="foo\n")
+    tests.add('failures/expected/newlines_with_excess_CR.html', expected_text="foo\r\r\r\n", actual_text="foo\n")
     tests.add('failures/expected/text.html', actual_text='text_fail-png')
     tests.add('failures/expected/crash_then_text.html')
     tests.add('failures/expected/skip_text.html', actual_text='text diff')
     tests.add('failures/flaky/text.html')
     tests.add('failures/unexpected/missing_text.html', expected_text=None)
     tests.add('failures/unexpected/missing_check.html', expected_image='missing-check-png')
     tests.add('failures/unexpected/missing_image.html', expected_image=None)
-    tests.add('failures/unexpected/missing_render_tree_dump.html', actual_text="""layer at (0,0) size 800x600
+    tests.add('failures/unexpected/missing_render_tree_dump.html',
+              actual_text="""layer at (0,0) size 800x600
   RenderView at (0,0) size 800x600
 layer at (0,0) size 800x34
   RenderBlock {HTML} at (0,0) size 800x34
     RenderBody {BODY} at (8,8) size 784x18
       RenderText {#text} at (0,0) size 133x18
         text run at (0,0) width 133: "This is an image test!"
-""", expected_text=None)
+""",
+              expected_text=None)
     tests.add('failures/unexpected/crash.html', crash=True)
-    tests.add('failures/unexpected/crash-with-stderr.html', crash=True,
-              error="mock-std-error-output")
-    tests.add('failures/unexpected/web-process-crash-with-stderr.html', web_process_crash=True,
-              error="mock-std-error-output")
+    tests.add('failures/unexpected/crash-with-stderr.html', crash=True, error="mock-std-error-output")
+    tests.add('failures/unexpected/web-process-crash-with-stderr.html', web_process_crash=True, error="mock-std-error-output")
     tests.add('failures/unexpected/pass.html')
     tests.add('failures/unexpected/text-checksum.html',
               actual_text='text-checksum_fail-txt',
               actual_checksum='text-checksum_fail-checksum')
     tests.add('failures/unexpected/text-image-checksum.html',
               actual_text='text-image-checksum_fail-txt',
               actual_image='text-image-checksum_fail-pngtEXtchecksum\x00checksum_fail',
               actual_checksum='text-image-checksum_fail-checksum')
-    tests.add('failures/unexpected/checksum-with-matching-image.html',
-              actual_checksum='text-image-checksum_fail-checksum')
+    tests.add('failures/unexpected/checksum-with-matching-image.html', actual_checksum='text-image-checksum_fail-checksum')
     tests.add('failures/unexpected/skip_pass.html')
     tests.add('failures/unexpected/text.html', actual_text='text_fail-txt')
     tests.add('failures/unexpected/text_then_crash.html')
     tests.add('failures/unexpected/timeout.html', timeout=True)
     tests.add('failures/unexpected/leak.html', leak=True)
     tests.add('http/tests/passes/text.html')
     tests.add('http/tests/passes/image.html')
     tests.add('http/tests/ssl/text.html')
     tests.add('passes/args.html')
     tests.add('passes/error.html', error='stuff going to stderr')
     tests.add('passes/image.html')
     tests.add('passes/audio.html',
-              actual_audio=base64.b64encode('audio-wav'), expected_audio='audio-wav',
-              actual_text=None, expected_text=None,
-              actual_image=None, expected_image=None,
+              actual_audio=base64.b64encode('audio-wav'),
+              expected_audio='audio-wav',
+              actual_text=None,
+              expected_text=None,
+              actual_image=None,
+              expected_image=None,
               actual_checksum=None)

[qyearsley, 2016/03/18 00:33:39]

Yapf also reformats arguments to functions like this so that there's one per
line (although in this case, there was a readability reason for putting two per
line).

     tests.add('passes/platform_image.html')
-    tests.add('passes/checksum_in_image.html',
-              expected_image='tEXtchecksum\x00checksum_in_image-checksum')
+    tests.add('passes/checksum_in_image.html', expected_image='tEXtchecksum\x00checksum_in_image-checksum')
     tests.add('passes/skipped/skip.html')
 
     # Note that here the checksums don't match but the images do, so this test passes "unexpectedly".
     # See https://bugs.webkit.org/show_bug.cgi?id=69444 .
     tests.add('failures/unexpected/checksum.html', actual_checksum='checksum_fail-checksum')
 
     # Text output files contain "\r\n" on Windows.  This may be
     # helpfully filtered to "\r\r\n" by our Python/Cygwin tooling.
-    tests.add('passes/text.html',
-              expected_text='\nfoo\n\n', actual_text='\nfoo\r\n\r\r\n')
+    tests.add('passes/text.html', expected_text='\nfoo\n\n', actual_text='\nfoo\r\n\r\r\n')
 
     # For reftests.
     tests.add_reftest('passes/reftest.html', 'passes/reftest-expected.html', same_image=True)
 
     # This adds a different virtual reference to ensure that that also works.
     tests.add('virtual/virtual_passes/passes/reftest-expected.html', actual_checksum='xxx', actual_image='XXX', is_reftest=True)
 
     tests.add_reftest('passes/mismatch.html', 'passes/mismatch-expected-mismatch.html', same_image=False)
     tests.add_reftest('passes/svgreftest.svg', 'passes/svgreftest-expected.svg', same_image=True)
     tests.add_reftest('passes/xhtreftest.xht', 'passes/xhtreftest-expected.html', same_image=True)
     tests.add_reftest('passes/phpreftest.php', 'passes/phpreftest-expected-mismatch.svg', same_image=False)
     tests.add_reftest('failures/expected/reftest.html', 'failures/expected/reftest-expected.html', same_image=False)
     tests.add_reftest('failures/expected/mismatch.html', 'failures/expected/mismatch-expected-mismatch.html', same_image=True)
-    tests.add_reftest('failures/unexpected/crash-reftest.html', 'failures/unexpected/crash-reftest-expected.html', same_image=True, crash=True)
+    tests.add_reftest('failures/unexpected/crash-reftest.html',
+                      'failures/unexpected/crash-reftest-expected.html',
+                      same_image=True,
+                      crash=True)
     tests.add_reftest('failures/unexpected/reftest.html', 'failures/unexpected/reftest-expected.html', same_image=False)
     tests.add_reftest('failures/unexpected/mismatch.html', 'failures/unexpected/mismatch-expected-mismatch.html', same_image=True)
     tests.add('failures/unexpected/reftest-nopixel.html', actual_checksum=None, actual_image=None, is_reftest=True)
     tests.add('failures/unexpected/reftest-nopixel-expected.html', actual_checksum=None, actual_image=None, is_reftest=True)
     tests.add('reftests/foo/test.html')
     tests.add('reftests/foo/test-ref.html')
 
     tests.add('reftests/foo/multiple-match-success.html', actual_checksum='abc', actual_image='abc')
     tests.add('reftests/foo/multiple-match-failure.html', actual_checksum='abc', actual_image='abc')
     tests.add('reftests/foo/multiple-mismatch-success.html', actual_checksum='abc', actual_image='abc')
@@ skipping 15 matching lines @@
     # For testing that we don't run tests under platform/. Note that these don't contribute to TOTAL_TESTS.
     tests.add('platform/test-mac-10.10/http/test.html')
     tests.add('platform/test-win-win7/http/test.html')
 
     # For testing if perf tests are running in a locked shard.
     tests.add('perf/foo/test.html')
     tests.add('perf/foo/test-ref.html')
 
     # For testing --pixel-test-directories.
     tests.add('failures/unexpected/pixeldir/image_in_pixeldir.html',
-        actual_image='image_in_pixeldir-pngtEXtchecksum\x00checksum_fail',
-        expected_image='image_in_pixeldir-pngtEXtchecksum\x00checksum-png')
+              actual_image='image_in_pixeldir-pngtEXtchecksum\x00checksum_fail',
+              expected_image='image_in_pixeldir-pngtEXtchecksum\x00checksum-png')
     tests.add('failures/unexpected/image_not_in_pixeldir.html',
-        actual_image='image_not_in_pixeldir-pngtEXtchecksum\x00checksum_fail',
-        expected_image='image_not_in_pixeldir-pngtEXtchecksum\x00checksum-png')
+              actual_image='image_not_in_pixeldir-pngtEXtchecksum\x00checksum_fail',
+              expected_image='image_not_in_pixeldir-pngtEXtchecksum\x00checksum-png')
 
     # For testing that virtual test suites don't expand names containing themselves
     # See webkit.org/b/97925 and base_unittest.PortTest.test_tests().
     tests.add('passes/test-virtual-passes.html')
     tests.add('passes/virtual_passes/test-virtual-passes.html')
 
     return tests
 
-
 # Here we use a non-standard location for the layout tests, to ensure that
 # this works. The path contains a '.' in the name because we've seen bugs
 # related to this before.
 
 LAYOUT_TEST_DIR = '/test.checkout/LayoutTests'
 PERF_TEST_DIR = '/test.checkout/PerformanceTests'
 
 
 # Here we synthesize an in-memory filesystem from the test list
 # in order to fully control the test output and to demonstrate that
@@ skipping 66 matching lines @@
     for test in test_list.tests.values():
         add_file(test, test.name[test.name.rfind('.'):], '')
         if test.is_reftest:
             continue
         if test.actual_audio:
             add_file(test, '-expected.wav', test.expected_audio)
             continue
         add_file(test, '-expected.txt', test.expected_text)
         add_file(test, '-expected.png', test.expected_image)
 
-    filesystem.write_text_file(filesystem.join(LAYOUT_TEST_DIR, 'virtual', 'virtual_passes', 'passes', 'args-expected.txt'), 'args-txt --virtual-arg')
+    filesystem.write_text_file(
+        filesystem.join(LAYOUT_TEST_DIR, 'virtual', 'virtual_passes', 'passes', 'args-expected.txt'), 'args-txt --virtual-arg')
     # Clear the list of written files so that we can watch what happens during testing.
     filesystem.clear_written_files()
 
 
 class TestPort(Port):
     port_name = 'test'
     default_port_name = 'test-mac-mac10.10'
 
     # TODO(wkorman): The below constant is legacy code and is only referenced by a unit test. Find the modern way to do
     # the same thing that test is doing and delete this.
     #
     # A list of platform names sufficient to cover all the baselines.
     # The list should be sorted so that a later platform  will reuse
     # an earlier platform's baselines if they are the same (e.g.,
     # 'mac10.10' should precede 'mac10.9').
-    ALL_BASELINE_VARIANTS = (
-        'test-linux-trusty', 'test-linux-precise', 'test-linux-x86',
-        'test-mac-mac10.11', 'test-mac-mac10.10',
-        'test-win-win10', 'test-win-win7'
-    )
+    ALL_BASELINE_VARIANTS = ('test-linux-trusty', 'test-linux-precise', 'test-linux-x86', 'test-mac-mac10.11', 'test-mac-mac10.10',
+                             'test-win-win10', 'test-win-win7')
 
     FALLBACK_PATHS = {
-        'win7':        ['test-win-win7', 'test-win-win10'],
-        'win10':       ['test-win-win10'],
-        'mac10.10':    ['test-mac-mac10.10', 'test-mac-mac10.11'],
-        'mac10.11':    ['test-mac-mac10.11'],
-        'trusty':      ['test-linux-trusty', 'test-win-win7'],
-        'precise':     ['test-linux-precise', 'test-linux-trusty', 'test-win-win7'],
-        'linux32':     ['test-linux-x86', 'test-linux-precise', 'test-linux-trusty', 'test-win-win7'],
+        'win7': ['test-win-win7', 'test-win-win10'],
+        'win10': ['test-win-win10'],
+        'mac10.10': ['test-mac-mac10.10', 'test-mac-mac10.11'],
+        'mac10.11': ['test-mac-mac10.11'],
+        'trusty': ['test-linux-trusty', 'test-win-win7'],
+        'precise': ['test-linux-precise', 'test-linux-trusty', 'test-win-win7'],
+        'linux32': ['test-linux-x86', 'test-linux-precise', 'test-linux-trusty', 'test-win-win7'],
     }
 
     @classmethod
     def determine_full_port_name(cls, host, options, port_name):
         if port_name == 'test':
             return TestPort.default_port_name
         return port_name
 
     def __init__(self, host, port_name=None, **kwargs):
         Port.__init__(self, host, port_name or TestPort.default_port_name, **kwargs)
@@ skipping 68 matching lines @@
     def layout_tests_dir(self):
         return LAYOUT_TEST_DIR
 
     def perf_tests_dir(self):
         return PERF_TEST_DIR
 
     def webkit_base(self):
         return '/test.checkout'
 
     def _skipped_tests_for_unsupported_features(self, test_list):
-        return set(['failures/expected/skip_text.html',
-                    'failures/unexpected/skip_pass.html',
-                    'virtual/skipped/failures/expected'])
+        return set(['failures/expected/skip_text.html', 'failures/unexpected/skip_pass.html', 'virtual/skipped/failures/expected'])
 
     def name(self):
         return self._name
 
     def operating_system(self):
         return self._operating_system
 
     def _path_to_wdiff(self):
         return None
 
@@ skipping 36 matching lines @@
     def _port_specific_expectations_files(self):
         return [self._filesystem.join(self._webkit_baseline_path(d), 'TestExpectations') for d in ['test', 'test-win-win7']]
 
     def all_test_configurations(self):
         """Returns a sequence of the TestConfigurations the port supports."""
         # By default, we assume we want to test every graphics type in
         # every configuration on every system.
         test_configurations = []
         for version, architecture in self._all_systems():
             for build_type in self._all_build_types():
-                test_configurations.append(TestConfiguration(
-                    version=version,
-                    architecture=architecture,
-                    build_type=build_type))
+                test_configurations.append(TestConfiguration(version=version, architecture=architecture, build_type=build_type))
         return test_configurations
 
     def _all_systems(self):
-        return (('mac10.10', 'x86'),
-                ('mac10.11', 'x86'),
-                ('win7', 'x86'),
-                ('win10', 'x86'),
-                ('linux32', 'x86'),
-                ('precise', 'x86_64'),
-                ('trusty', 'x86_64'))
+        return (('mac10.10', 'x86'), ('mac10.11', 'x86'), ('win7', 'x86'), ('win10', 'x86'), ('linux32', 'x86'),
+                ('precise', 'x86_64'), ('trusty', 'x86_64'))
 
     def _all_build_types(self):
         return ('debug', 'release')
 
     def configuration_specifier_macros(self):
         """To avoid surprises when introducing new macros, these are intentionally fixed in time."""
-        return {
-            'mac': ['mac10.10', 'mac10.11'],
-            'win': ['win7', 'win10'],
-            'linux': ['linux32', 'precise', 'trusty']
-        }
+        return {'mac': ['mac10.10', 'mac10.11'], 'win': ['win7', 'win10'], 'linux': ['linux32', 'precise', 'trusty']}
 
     def virtual_test_suites(self):
         return [
-            VirtualTestSuite(prefix='virtual_passes', base='passes', args=['--virtual-arg']),
-            VirtualTestSuite(prefix='skipped', base='failures/expected', args=['--virtual-arg2']),
-            VirtualTestSuite(prefix='references_use_default_args', base='passes/reftest.html',
-                             args=['--virtual-arg'], references_use_default_args=True),
+            VirtualTestSuite(prefix='virtual_passes',
+                             base='passes',
+                             args=['--virtual-arg']),
+            VirtualTestSuite(prefix='skipped',
+                             base='failures/expected',
+                             args=['--virtual-arg2']),
+            VirtualTestSuite(prefix='references_use_default_args',
+                             base='passes/reftest.html',
+                             args=['--virtual-arg'],
+                             references_use_default_args=True),
         ]
 
 
 class TestDriver(Driver):
     """Test/Dummy implementation of the driver interface."""
     next_pid = 1
 
     def __init__(self, *args, **kwargs):
         super(TestDriver, self).__init__(*args, **kwargs)
         self.started = False
         self.pid = 0
 
     def cmd_line(self, pixel_tests, per_test_args):
         pixel_tests_flag = '-p' if pixel_tests else ''
-        return [self._port._path_to_driver()] + [pixel_tests_flag] + self._port.get_option('additional_driver_flag', []) + per_test_args
+        return [self._port._path_to_driver()] + [pixel_tests_flag] + self._port.get_option('additional_driver_flag',
+                                                                                           []) + per_test_args
 
     def run_test(self, driver_input, stop_when_done):
         if not self.started:
             self.started = True
             self.pid = TestDriver.next_pid
             TestDriver.next_pid += 1
 
         start_time = time.time()
         test_name = driver_input.test_name
         test_args = driver_input.args or []
@@ skipping 57 matching lines @@
             crash = True
             crash_log = 'reftest crash log'
 
         if stop_when_done:
             self.stop()
 
         if test.actual_checksum == driver_input.image_hash:
             image = None
         else:
             image = test.actual_image
-        return DriverOutput(actual_text, image, test.actual_checksum, audio,
-            crash=(crash or web_process_crash), crashed_process_name=crashed_process_name,
-            crashed_pid=crashed_pid, crash_log=crash_log,
-            test_time=time.time() - start_time, timeout=test.timeout, error=test.error, pid=self.pid,
-            leak=test.leak)
+        return DriverOutput(actual_text,
+                            image,
+                            test.actual_checksum,
+                            audio,
+                            crash=(crash or web_process_crash),
+                            crashed_process_name=crashed_process_name,
+                            crashed_pid=crashed_pid,
+                            crash_log=crash_log,
+                            test_time=time.time() - start_time,
+                            timeout=test.timeout,
+                            error=test.error,
+                            pid=self.pid,
+                            leak=test.leak)
 
     def stop(self):
         self.started = False