Run format-webkitpy and fix long lines.

third_party/WebKit/Tools/Scripts/webkitpy/common/checkout/baselineoptimizer_unittest.py

 # Copyright (C) 2011 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 44 matching lines @@
         return MockSCM.delete_list(self, paths)
 
     def move(self, origin, destination):
         if origin in self._exclusion_list:
             raise Exception("File is not SCM managed: " + origin)
         return MockSCM.move(self, origin, destination)
 
 
 class BaselineOptimizerTest(unittest.TestCase):
 
+    # Protected method _move_baselines is tested below - pylint: disable=protected-access
     def test_move_baselines(self):
         host = MockHost(scm=ExcludingMockSCM(
             ['/mock-checkout/third_party/WebKit/LayoutTests/platform/mac/another/test-expected.txt']))
         host.filesystem.write_text_file('/mock-checkout/third_party/WebKit/LayoutTests/VirtualTestSuites', '[]')
         host.filesystem.write_binary_file(
             '/mock-checkout/third_party/WebKit/LayoutTests/platform/win/another/test-expected.txt', 'result A')
         host.filesystem.write_binary_file(
             '/mock-checkout/third_party/WebKit/LayoutTests/platform/mac/another/test-expected.txt', 'result A')
         host.filesystem.write_binary_file('/mock-checkout/third_party/WebKit/LayoutTests/another/test-expected.txt', 'result B')
-        baseline_optimizer = BaselineOptimizer(host, host.port_factory.get(
-        ), host.port_factory.all_port_names(), skip_scm_commands=False)
-        baseline_optimizer._move_baselines('another/test-expected.txt', {
-            '/mock-checkout/third_party/WebKit/LayoutTests/platform/win': 'aaa',
-            '/mock-checkout/third_party/WebKit/LayoutTests/platform/mac': 'aaa',
-            '/mock-checkout/third_party/WebKit/LayoutTests': 'bbb',
-        }, {
-            '/mock-checkout/third_party/WebKit/LayoutTests': 'aaa',
-        })
+        baseline_optimizer = BaselineOptimizer(
+            host, host.port_factory.get(), host.port_factory.all_port_names(), skip_scm_commands=False)
+        baseline_optimizer._move_baselines(
+            'another/test-expected.txt',
+            {
+                '/mock-checkout/third_party/WebKit/LayoutTests/platform/win': 'aaa',
+                '/mock-checkout/third_party/WebKit/LayoutTests/platform/mac': 'aaa',
+                '/mock-checkout/third_party/WebKit/LayoutTests': 'bbb',
+            },
+            {
+                '/mock-checkout/third_party/WebKit/LayoutTests': 'aaa',
+            })
         self.assertEqual(host.filesystem.read_binary_file(
             '/mock-checkout/third_party/WebKit/LayoutTests/another/test-expected.txt'), 'result A')
 
     def test_move_baselines_skip_scm_commands(self):
         host = MockHost(scm=ExcludingMockSCM(
             ['/mock-checkout/third_party/WebKit/LayoutTests/platform/mac/another/test-expected.txt']))
         host.filesystem.write_text_file('/mock-checkout/third_party/WebKit/LayoutTests/VirtualTestSuites', '[]')
         host.filesystem.write_binary_file(
             '/mock-checkout/third_party/WebKit/LayoutTests/platform/win/another/test-expected.txt', 'result A')
         host.filesystem.write_binary_file(
             '/mock-checkout/third_party/WebKit/LayoutTests/platform/mac/another/test-expected.txt', 'result A')
         host.filesystem.write_binary_file('/mock-checkout/third_party/WebKit/LayoutTests/another/test-expected.txt', 'result B')
         baseline_optimizer = BaselineOptimizer(host, host.port_factory.get(
         ), host.port_factory.all_port_names(), skip_scm_commands=True)
-        baseline_optimizer._move_baselines('another/test-expected.txt', {
-            '/mock-checkout/third_party/WebKit/LayoutTests/platform/win': 'aaa',
-            '/mock-checkout/third_party/WebKit/LayoutTests/platform/mac': 'aaa',
-            '/mock-checkout/third_party/WebKit/LayoutTests': 'bbb',
-        }, {
-            '/mock-checkout/third_party/WebKit/LayoutTests/platform/linux': 'bbb',
-            '/mock-checkout/third_party/WebKit/LayoutTests': 'aaa',
-        })
-        self.assertEqual(host.filesystem.read_binary_file(
-            '/mock-checkout/third_party/WebKit/LayoutTests/another/test-expected.txt'), 'result A')
+        baseline_optimizer._move_baselines(
+            'another/test-expected.txt',
+            {
+                '/mock-checkout/third_party/WebKit/LayoutTests/platform/win': 'aaa',
+                '/mock-checkout/third_party/WebKit/LayoutTests/platform/mac': 'aaa',
+                '/mock-checkout/third_party/WebKit/LayoutTests': 'bbb',
+            },
+            {
+                '/mock-checkout/third_party/WebKit/LayoutTests/platform/linux': 'bbb',
+                '/mock-checkout/third_party/WebKit/LayoutTests': 'aaa',
+            })
+        self.assertEqual(
+            host.filesystem.read_binary_file(
+                '/mock-checkout/third_party/WebKit/LayoutTests/another/test-expected.txt'),
+            'result A')
 
-        self.assertEqual(baseline_optimizer._files_to_delete, [
-            '/mock-checkout/third_party/WebKit/LayoutTests/platform/win/another/test-expected.txt',
-        ])
+        self.assertEqual(
+            baseline_optimizer._files_to_delete,
+            [
+                '/mock-checkout/third_party/WebKit/LayoutTests/platform/win/another/test-expected.txt',
+            ])
 
-        self.assertEqual(baseline_optimizer._files_to_add, [
-            '/mock-checkout/third_party/WebKit/LayoutTests/another/test-expected.txt',
-            '/mock-checkout/third_party/WebKit/LayoutTests/platform/linux/another/test-expected.txt',
-        ])
+        self.assertEqual(
+            baseline_optimizer._files_to_add,
+            [
+                '/mock-checkout/third_party/WebKit/LayoutTests/another/test-expected.txt',
+                '/mock-checkout/third_party/WebKit/LayoutTests/platform/linux/another/test-expected.txt',
+            ])
 
     def _assertOptimization(self, results_by_directory, expected_new_results_by_directory,
                             baseline_dirname='', expected_files_to_delete=None, host=None):
         if not host:
             host = MockHost()
         fs = host.filesystem
         webkit_base = WebKitFinder(fs).webkit_base()
         baseline_name = 'mock-baseline-expected.txt'
         fs.write_text_file(fs.join(webkit_base, 'LayoutTests', 'VirtualTestSuites'),
                            '[{"prefix": "gpu", "base": "fast/canvas", "args": ["--foo"]}]')
@@ skipping 16 matching lines @@
         # Check that the files that were in the original set have been deleted where necessary.
         for dirname in results_by_directory:
             path = fs.join(webkit_base, 'LayoutTests', dirname, baseline_name)
             if not dirname in expected_new_results_by_directory:
                 self.assertTrue(not fs.exists(path) or path in baseline_optimizer._files_to_delete)
 
         if expected_files_to_delete:
             self.assertEqual(sorted(baseline_optimizer._files_to_delete), sorted(expected_files_to_delete))
 
     def test_linux_redundant_with_win(self):
-        self._assertOptimization({
-            'platform/win': '1',
-            'platform/linux': '1',
-        }, {
-            'platform/win': '1',
-        })
+        self._assertOptimization(
+            {
+                'platform/win': '1',
+                'platform/linux': '1',
+            },
+            {
+                'platform/win': '1',
+            })
 
     def test_covers_mac_win_linux(self):
-        self._assertOptimization({
-            'platform/mac': '1',
-            'platform/win': '1',
-            'platform/linux': '1',
-            '': None,
-        }, {
-            '': '1',
-        })
+        self._assertOptimization(
+            {
+                'platform/mac': '1',
+                'platform/win': '1',
+                'platform/linux': '1',
+                '': None,
+            },
+            {
+                '': '1',
+            })
 
     def test_overwrites_root(self):
-        self._assertOptimization({
-            'platform/mac': '1',
-            'platform/win': '1',
-            'platform/linux': '1',
-            '': '2',
-        }, {
-            '': '1',
-        })
+        self._assertOptimization(
+            {
+                'platform/mac': '1',
+                'platform/win': '1',
+                'platform/linux': '1',
+                '': '2',
+            },
+            {
+                '': '1',
+            })
 
     def test_no_new_common_directory(self):
-        self._assertOptimization({
-            'platform/mac': '1',
-            'platform/linux': '1',
-            '': '2',
-        }, {
-            'platform/mac': '1',
-            'platform/linux': '1',
-            '': '2',
-        })
+        self._assertOptimization(
+            {
+                'platform/mac': '1',
+                'platform/linux': '1',
+                '': '2',
+            },
+            {
+                'platform/mac': '1',
+                'platform/linux': '1',
+                '': '2',
+            })
 
     def test_local_optimization(self):
-        self._assertOptimization({
-            'platform/mac': '1',
-            'platform/linux': '1',
-            'platform/linux-precise': '1',
-        }, {
-            'platform/mac': '1',
-            'platform/linux': '1',
-        })
+        self._assertOptimization(
+            {
+                'platform/mac': '1',
+                'platform/linux': '1',
+                'platform/linux-precise': '1',
+            },
+            {
+                'platform/mac': '1',
+                'platform/linux': '1',
+            })
 
     def test_local_optimization_skipping_a_port_in_the_middle(self):
-        self._assertOptimization({
-            'platform/mac-snowleopard': '1',
-            'platform/win': '1',
-            'platform/linux': '1',
-            'platform/linux-precise': '1',
-        }, {
-            'platform/mac-snowleopard': '1',
-            'platform/win': '1',
-        })
+        self._assertOptimization(
+            {
+                'platform/mac-snowleopard': '1',
+                'platform/win': '1',
+                'platform/linux': '1',
+                'platform/linux-precise': '1',
+            },
+            {
+                'platform/mac-snowleopard': '1',
+                'platform/win': '1',
+            })
 
     def test_baseline_redundant_with_root(self):
-        self._assertOptimization({
-            'platform/mac': '1',
-            'platform/win': '2',
-            '': '2',
-        }, {
-            'platform/mac': '1',
-            '': '2',
-        })
+        self._assertOptimization(
+            {
+                'platform/mac': '1',
+                'platform/win': '2',
+                '': '2',
+            },
+            {
+                'platform/mac': '1',
+                '': '2',
+            })
 
     def test_root_baseline_unused(self):
-        self._assertOptimization({
-            'platform/mac': '1',
-            'platform/win': '2',
-            '': '3',
-        }, {
-            'platform/mac': '1',
-            'platform/win': '2',
-        })
+        self._assertOptimization(
+            {
+                'platform/mac': '1',
+                'platform/win': '2',
+                '': '3',
+            },
+            {
+                'platform/mac': '1',
+                'platform/win': '2',
+            })
 
     def test_root_baseline_unused_and_non_existant(self):
-        self._assertOptimization({
-            'platform/mac': '1',
-            'platform/win': '2',
-        }, {
-            'platform/mac': '1',
-            'platform/win': '2',
-        })
+        self._assertOptimization(
+            {
+                'platform/mac': '1',
+                'platform/win': '2',
+            },
+            {
+                'platform/mac': '1',
+                'platform/win': '2',
+            })
 
     def test_virtual_root_redundant_with_actual_root(self):
-        self._assertOptimization({
-            'virtual/gpu/fast/canvas': '2',
-            'fast/canvas': '2',
-        }, {
-            'virtual/gpu/fast/canvas': None,
-            'fast/canvas': '2',
-        }, baseline_dirname='virtual/gpu/fast/canvas')
+        self._assertOptimization(
+            {
+                'virtual/gpu/fast/canvas': '2',
+                'fast/canvas': '2',
+            },
+            {
+                'virtual/gpu/fast/canvas': None,
+                'fast/canvas': '2',
+            },
+            baseline_dirname='virtual/gpu/fast/canvas')
 
     def test_virtual_root_redundant_with_ancestors(self):
-        self._assertOptimization({
-            'virtual/gpu/fast/canvas': '2',
-            'platform/mac/fast/canvas': '2',
-            'platform/win/fast/canvas': '2',
-        }, {
-            'virtual/gpu/fast/canvas': None,
-            'fast/canvas': '2',
-        }, baseline_dirname='virtual/gpu/fast/canvas')
+        self._assertOptimization(
+            {
+                'virtual/gpu/fast/canvas': '2',
+                'platform/mac/fast/canvas': '2',
+                'platform/win/fast/canvas': '2',
+            },
+            {
+                'virtual/gpu/fast/canvas': None,
+                'fast/canvas': '2',
+            },
+            baseline_dirname='virtual/gpu/fast/canvas')
 
     def test_virtual_root_redundant_with_ancestors_skip_scm_commands(self):
-        self._assertOptimization({
-            'virtual/gpu/fast/canvas': '2',
-            'platform/mac/fast/canvas': '2',
-            'platform/win/fast/canvas': '2',
-        }, {
-            'virtual/gpu/fast/canvas': None,
-            'fast/canvas': '2',
-        },
+        self._assertOptimization(
+            {
+                'virtual/gpu/fast/canvas': '2',
+                'platform/mac/fast/canvas': '2',
+                'platform/win/fast/canvas': '2',
+            },
+            {
+                'virtual/gpu/fast/canvas': None,
+                'fast/canvas': '2',
+            },
             baseline_dirname='virtual/gpu/fast/canvas',
             expected_files_to_delete=[
-            '/mock-checkout/third_party/WebKit/LayoutTests/virtual/gpu/fast/canvas/mock-baseline-expected.txt',
-            '/mock-checkout/third_party/WebKit/LayoutTests/platform/mac/fast/canvas/mock-baseline-expected.txt',
-            '/mock-checkout/third_party/WebKit/LayoutTests/platform/win/fast/canvas/mock-baseline-expected.txt',
-        ])
+                '/mock-checkout/third_party/WebKit/LayoutTests/virtual/gpu/fast/canvas/mock-baseline-expected.txt',
+                '/mock-checkout/third_party/WebKit/LayoutTests/platform/mac/fast/canvas/mock-baseline-expected.txt',
+                '/mock-checkout/third_party/WebKit/LayoutTests/platform/win/fast/canvas/mock-baseline-expected.txt',
+            ])
 
     def test_virtual_root_redundant_with_ancestors_skip_scm_commands_with_file_not_in_scm(self):
-        self._assertOptimization({
-            'virtual/gpu/fast/canvas': '2',
-            'platform/mac/fast/canvas': '2',
-            'platform/win/fast/canvas': '2',
-        }, {
-            'virtual/gpu/fast/canvas': None,
-            'fast/canvas': '2',
-        },
+        self._assertOptimization(
+            {
+                'virtual/gpu/fast/canvas': '2',
+                'platform/mac/fast/canvas': '2',
+                'platform/win/fast/canvas': '2',
+            },
+            {
+                'virtual/gpu/fast/canvas': None,
+                'fast/canvas': '2',
+            },
             baseline_dirname='virtual/gpu/fast/canvas',
             expected_files_to_delete=[
-            '/mock-checkout/third_party/WebKit/LayoutTests/platform/mac/fast/canvas/mock-baseline-expected.txt',
-            '/mock-checkout/third_party/WebKit/LayoutTests/platform/win/fast/canvas/mock-baseline-expected.txt',
-        ],
-            host=MockHost(scm=ExcludingMockSCM(['/mock-checkout/third_party/WebKit/LayoutTests/virtual/gpu/fast/canvas/mock-baseline-expected.txt'])))
+                '/mock-checkout/third_party/WebKit/LayoutTests/platform/mac/fast/canvas/mock-baseline-expected.txt',
+                '/mock-checkout/third_party/WebKit/LayoutTests/platform/win/fast/canvas/mock-baseline-expected.txt',
+            ],
+            host=MockHost(scm=ExcludingMockSCM(
+                ['/mock-checkout/third_party/WebKit/LayoutTests/virtual/gpu/fast/canvas/mock-baseline-expected.txt'])))
 
     def test_virtual_root_not_redundant_with_ancestors(self):
-        self._assertOptimization({
-            'virtual/gpu/fast/canvas': '2',
-            'platform/mac/fast/canvas': '1',
-        }, {
-            'virtual/gpu/fast/canvas': '2',
-            'platform/mac/fast/canvas': '1',
-        }, baseline_dirname='virtual/gpu/fast/canvas')
+        self._assertOptimization(
+            {
+                'virtual/gpu/fast/canvas': '2',
+                'platform/mac/fast/canvas': '1',
+            },
+            {
+                'virtual/gpu/fast/canvas': '2',
+                'platform/mac/fast/canvas': '1',
+            },
+            baseline_dirname='virtual/gpu/fast/canvas')