Reformat webkitpy.layout_tests w/ format-webkitpy.

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 24 matching lines @@
 from webkitpy.layout_tests.port.base import VirtualTestSuite
 from webkitpy.layout_tests.models.test_configuration import TestConfiguration
 from webkitpy.layout_tests.models import test_run_results
 from webkitpy.common.system.filesystem_mock import MockFileSystem
 from webkitpy.common.system.crashlogs import CrashLogs
 
 
 # This sets basic expectations for a test. Each individual expectation
 # can be overridden by a keyword argument in TestList.add().
 class TestInstance(object):
+
     def __init__(self, name):
         self.name = name
-        self.base = name[(name.rfind("/") + 1):name.rfind(".")]
+        self.base = name[(name.rfind('/') + 1):name.rfind('.')]
         self.crash = False
         self.web_process_crash = False
         self.exception = False
         self.keyboard = False
         self.error = ''
         self.timeout = False
         self.is_reftest = False
         self.device_failure = False
         self.leak = False
 
@@ skipping 10 matching lines @@
         self.expected_text = self.actual_text
         self.expected_image = self.actual_image
 
         self.actual_audio = None
         self.expected_audio = None
 
 
 # This is an in-memory list of tests, what we want them to produce, and
 # what we want to claim are the expected results.
 class TestList(object):
+
     def __init__(self):
         self.tests = {}
 
     def add(self, name, **kwargs):
         test = TestInstance(name)
         for key, value in kwargs.items():
             test.__dict__[key] = value
         self.tests[name] = test
 
     def add_reftest(self, name, reference_name, same_image, crash=False):
@@ skipping 14 matching lines @@
 
 #
 # These numbers may need to be updated whenever we add or delete tests. This includes virtual tests.
 #
 TOTAL_TESTS = 113
 TOTAL_SKIPS = 29
 
 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')
@@ skipping 11 matching lines @@
     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_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,
               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")
+              expected_text='\nfoo\n', actual_text='foo\n')
     tests.add('failures/expected/newlines_trailing.html',
-              expected_text="foo\n\n", actual_text="foo\n")
+              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")
+              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
   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)
     tests.add('failures/unexpected/crash.html', crash=True)
     tests.add('failures/unexpected/crash-with-stderr.html', crash=True,
-              error="mock-std-error-output")
+              error='mock-std-error-output')
     tests.add('failures/unexpected/web-process-crash-with-stderr.html', web_process_crash=True,
-              error="mock-std-error-output")
+              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')
@@ skipping 32 matching lines @@
 
     # This adds a different virtual reference to ensure that that also works.
     tests.add('virtual/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-leopard/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/passes/test-virtual-passes.html')
 
     return tests
 
 
 # Here we use a non-standard location for the layout tests, to ensure that
@@ skipping 92 matching lines @@
     default_port_name = 'test-mac-leopard'
 
     """Test implementation of the Port interface."""
     ALL_BASELINE_VARIANTS = (
         'test-linux-x86_64',
         'test-mac-snowleopard', 'test-mac-leopard',
         'test-win-win7', 'test-win-xp',
     )
 
     FALLBACK_PATHS = {
-        'xp':          ['test-win-win7', 'test-win-xp'],
-        'win7':        ['test-win-win7'],
-        'leopard':     ['test-mac-leopard', 'test-mac-snowleopard'],
+        'xp': ['test-win-win7', 'test-win-xp'],
+        'win7': ['test-win-win7'],
+        'leopard': ['test-mac-leopard', 'test-mac-snowleopard'],
         'snowleopard': ['test-mac-snowleopard'],
-        'lucid':       ['test-linux-x86_64', 'test-win-win7'],
+        'lucid': ['test-linux-x86_64', '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 55 matching lines @@
     def default_configuration(self):
         return 'Release'
 
     def diff_image(self, expected_contents, actual_contents):
         diffed = actual_contents != expected_contents
         if not actual_contents and not expected_contents:
             return (None, None)
         if not actual_contents or not expected_contents:
             return (True, None)
         if diffed:
-            return ("< %s\n---\n> %s\n" % (expected_contents, actual_contents), None)
+            return ('< %s\n---\n> %s\n' % (expected_contents, actual_contents), None)
         return (None, None)
 
     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'
@@ skipping 33 matching lines @@
     def stop_http_server(self):
         pass
 
     def stop_websocket_server(self):
         pass
 
     def release_http_lock(self):
         pass
 
     def path_to_apache(self):
-        return "/usr/sbin/httpd"
+        return '/usr/sbin/httpd'
 
     def path_to_apache_config_file(self):
         return self._filesystem.join(self.layout_tests_dir(), 'http', 'conf', 'httpd.conf')
 
     def path_to_generic_test_expectations_file(self):
         return self._generic_expectations_path
 
     def _port_specific_expectations_files(self):
         return [self._filesystem.join(self._webkit_baseline_path(d), 'TestExpectations') for d in ['test', 'test-win-xp']]
 
@@ skipping 29 matching lines @@
         return self.ALL_BASELINE_VARIANTS
 
     def virtual_test_suites(self):
         return [
             VirtualTestSuite('passes', 'passes', ['--virtual-arg'], use_legacy_naming=True),
             VirtualTestSuite('skipped', 'failures/expected', ['--virtual-arg2'], use_legacy_naming=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_drt_flag', []) + per_test_args
+        return [self._port._path_to_driver()] + [pixel_tests_flag] + \
+            self._port.get_option('additional_drt_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 52 matching lines @@
             crash_log = crash_logs.find_newest_log(crashed_process_name, None) or ''
 
         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)
+                            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