Changeset 231360 in webkit

Timestamp:

May 4, 2018 10:36:34 AM (6 years ago)

Author:

commit-queue@webkit.org

Message:

Test262-Runner: place the processCLI inside the main call
​https://bugs.webkit.org/show_bug.cgi?id=185304

Patch by Leo Balter <Leo Balter> on 2018-05-04
Reviewed by Michael Saboff.

• Scripts/test262/Runner.pm:

(main):

Location:

trunk/Tools

Files:

• ChangeLog (modified) (1 diff)
• Scripts/test262/Runner.pm (modified) (2 diffs)

trunk/Tools/ChangeLog

@@ +1 @@
+2018-05-04  Leo Balter  <leonardo.balter@gmail.com>
+
+        Test262-Runner: place the processCLI inside the main call
+        https://bugs.webkit.org/show_bug.cgi?id=185304
+
+        Reviewed by Michael Saboff.
+
+        * Scripts/test262/Runner.pm:
+        (main):
+
 2018-05-04  Carlos Garcia Campos  <cgarcia@igalia.com>
 

trunk/Tools/Scripts/test262/Runner.pm

@@ -93 +93 @@
 
 my $tempdir = tempdir();
-
-my @default_harnesses = (
-    "$harnessDir/sta.js",
-    "$harnessDir/assert.js",
-    "$harnessDir/doneprintHandle.js",
-    "$Bin/agent.js"
-);
-
 my ($deffh, $deffile) = getTempFile();
-print $deffh getHarness(<@default_harnesses>);
+
+my @default_harnesses;
 
 my $startTime = time();
 
-processCLI();
 main();
 
@@ -224 +216 @@
 
 sub main {
+    processCLI();
+
+    @default_harnesses = (
+        "$harnessDir/sta.js",
+        "$harnessDir/assert.js",
+        "$harnessDir/doneprintHandle.js",
+        "$Bin/agent.js"
+    );
+    print $deffh getHarness(<@default_harnesses>);
 
     # If not commandline test path supplied, use the root directory of all tests.