Changeset 122548 in webkit

Timestamp:

Jul 13, 2012 12:39:36 AM (12 years ago)

Author:

dpranke@chromium.org

Message:

test-webkitpy: more class renaming cleanup
​https://bugs.webkit.org/show_bug.cgi?id=91182

Reviewed by Adam Barth.

More removing of the unnecessary "Test" prefix.

• Scripts/webkitpy/test/finder.py:

(_DirectoryTree):
(Finder.add_tree):

• Scripts/webkitpy/test/main.py:

(Tester._run_tests):

• Scripts/webkitpy/test/runner.py:

(Runner):

• Scripts/webkitpy/test/runner_unittest.py:

(RunnerTest.test_regular):
(RunnerTest.test_verbose):
(RunnerTest.test_timing):

Location:

trunk/Tools

Files:

• ChangeLog (modified) (1 diff)
• Scripts/webkitpy/test/finder.py (modified) (2 diffs)
• Scripts/webkitpy/test/main.py (modified) (2 diffs)
• Scripts/webkitpy/test/runner.py (modified) (1 diff)
• Scripts/webkitpy/test/runner_unittest.py (modified) (4 diffs)

trunk/Tools/ChangeLog

@@ +1 @@
+2012-07-13  Dirk Pranke  <dpranke@chromium.org>
+
+        test-webkitpy: more class renaming cleanup
+        https://bugs.webkit.org/show_bug.cgi?id=91182
+
+        Reviewed by Adam Barth.
+
+        More removing of the unnecessary "Test" prefix.
+
+        * Scripts/webkitpy/test/finder.py:
+        (_DirectoryTree):
+        (Finder.add_tree):
+        * Scripts/webkitpy/test/main.py:
+        (Tester._run_tests):
+        * Scripts/webkitpy/test/runner.py:
+        (Runner):
+        * Scripts/webkitpy/test/runner_unittest.py:
+        (RunnerTest.test_regular):
+        (RunnerTest.test_verbose):
+        (RunnerTest.test_timing):
+
 2012-07-12  Christophe Dumez  <christophe.dumez@intel.com>
 

trunk/Tools/Scripts/webkitpy/test/finder.py

@@ -32 +32 @@
 
 
-class TestDirectoryTree(object):
+class _DirectoryTree(object):
     def __init__(self, filesystem, top_directory, starting_subdirectory):
         self.filesystem = filesystem
@@ -80 +80 @@
 
     def add_tree(self, top_directory, starting_subdirectory=None):
-        self.trees.append(TestDirectoryTree(self.filesystem, top_directory, starting_subdirectory))
+        self.trees.append(_DirectoryTree(self.filesystem, top_directory, starting_subdirectory))
 
     def additional_paths(self, paths):

trunk/Tools/Scripts/webkitpy/test/main.py

@@ -35 +35 @@
 from webkitpy.common.system import outputcapture
 from webkitpy.test.finder import Finder
-from webkitpy.test.runner import TestRunner
+from webkitpy.test.runner import Runner
 
 _log = logging.getLogger(__name__)
@@ -178 +178 @@
 
         test_suite = unittest.TestSuite(suites)
-        test_runner = TestRunner(self.stream, self._options, loader)
+        test_runner = Runner(self.stream, self._options, loader)
 
         _log.debug("Running the tests.")

trunk/Tools/Scripts/webkitpy/test/runner.py

@@ -32 +32 @@
 
 
-class TestRunner(object):
+class Runner(object):
     def __init__(self, stream, options, loader):
         self.options = options

trunk/Tools/Scripts/webkitpy/test/runner_unittest.py

@@ -26 +26 @@
 
 from webkitpy.tool.mocktool import MockOptions
-from webkitpy.test.runner import TestRunner
+from webkitpy.test.runner import Runner
 
 
@@ -75 +75 @@
                             ('test2 (Foo)', 'F', 'test2\nfailed'),
                             ('test3 (Foo)', 'E', 'test3\nerred'))
-        result = TestRunner(stream, options, loader).run(loader.top_suite())
+        result = Runner(stream, options, loader).run(loader.top_suite())
         self.assertFalse(result.wasSuccessful())
         self.assertEquals(result.testsRun, 3)
@@ -88 +88 @@
                             ('test2 (Foo)', 'F', 'test2\nfailed'),
                             ('test3 (Foo)', 'E', 'test3\nerred'))
-        result = TestRunner(stream, options, loader).run(loader.top_suite())
+        result = Runner(stream, options, loader).run(loader.top_suite())
         self.assertFalse(result.wasSuccessful())
         self.assertEquals(result.testsRun, 3)
@@ -101 +101 @@
                             ('test2 (Foo)', 'F', 'test2\nfailed'),
                             ('test3 (Foo)', 'E', 'test3\nerred'))
-        result = TestRunner(stream, options, loader).run(loader.top_suite())
+        result = Runner(stream, options, loader).run(loader.top_suite())
         self.assertFalse(result.wasSuccessful())
         self.assertEquals(result.testsRun, 3)