run-api-tests: inline runAllTestsInSuite() into runAllTests()
authorddkilzer@apple.com <ddkilzer@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 22 Dec 2011 17:19:49 +0000 (17:19 +0000)
committerddkilzer@apple.com <ddkilzer@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 22 Dec 2011 17:19:49 +0000 (17:19 +0000)
Reviewed by Adam Roben.

Part of: <http://webkit.org/b/75065> run-api-tests should be able to run individual suites and tests

In order to make it easy to run one or more individual suites,
or one more individual tests, it's easier to call a single
method with different data structures than individual methods.

* Scripts/run-api-tests:
(runAllTests): Inline runAllTestsInSuite().

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@103549 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/Scripts/run-api-tests

index 29bce7d..a34efa8 100644 (file)
@@ -1,3 +1,18 @@
+2011-12-22  David Kilzer  <ddkilzer@apple.com>
+
+        run-api-tests: inline runAllTestsInSuite() into runAllTests()
+
+        Reviewed by Adam Roben.
+
+        Part of: <http://webkit.org/b/75065> run-api-tests should be able to run individual suites and tests
+
+        In order to make it easy to run one or more individual suites,
+        or one more individual tests, it's easier to call a single
+        method with different data structures than individual methods.
+
+        * Scripts/run-api-tests:
+        (runAllTests): Inline runAllTestsInSuite().
+
 2011-12-22  Adam Roben  <aroben@apple.com>
 
         Add a --xml flag to test-webkitpy
 2011-12-22  Adam Roben  <aroben@apple.com>
 
         Add a --xml flag to test-webkitpy
index cfc4e6a..1ee8dd5 100755 (executable)
@@ -42,7 +42,6 @@ sub buildTestTool();
 sub dumpTestsBySuite(\%);
 sub populateTests();
 sub runAllTests();
 sub dumpTestsBySuite(\%);
 sub populateTests();
 sub runAllTests();
-sub runAllTestsInSuite($);
 sub runTest($$);
 sub prepareEnvironmentForRunningTestTool();
 sub testToolPath();
 sub runTest($$);
 sub prepareEnvironmentForRunningTestTool();
 sub testToolPath();
@@ -118,9 +117,12 @@ sub runAllTests()
 {
     my $anyFailures = 0;
     for my $suite (sort keys %testsToRun) {
 {
     my $anyFailures = 0;
     for my $suite (sort keys %testsToRun) {
-        my $failed = runAllTestsInSuite($suite);
-        if ($failed) {
-            $anyFailures = 1;
+        print "Suite: $suite\n" unless $verbose;
+        for my $test (sort @{$testsToRun{$suite}}) {
+            my $failed = runTest($suite, $test);
+            if ($failed) {
+                $anyFailures = 1;
+            }
         }
     }
     
         }
     }
     
@@ -141,22 +143,6 @@ sub runAllTests()
     return $anyFailures;
 }
 
     return $anyFailures;
 }
 
-sub runAllTestsInSuite($)
-{
-    my ($suite) = @_;
-    print "Suite: $suite\n" unless $verbose;
-
-    my $anyFailures = 0;
-    for my $test (sort @{$testsToRun{$suite}}) {
-        my $failed = runTest($suite, $test);
-        if ($failed) {
-            $anyFailures = 1;
-        }
-    }
-   
-    return $anyFailures;
-}
-
 sub runTest($$)
 {
     my ($suite, $testName) = @_;
 sub runTest($$)
 {
     my ($suite, $testName) = @_;