test262/Runner.pm: minor fixes
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 3 May 2018 20:27:43 +0000 (20:27 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 3 May 2018 20:27:43 +0000 (20:27 +0000)
https://bugs.webkit.org/show_bug.cgi?id=185250

Patch by Valerie R Young <valerie@bocoup.com> on 2018-05-03
Reviewed by Michael Saboff.

Add comments, change output and fix bug: unintended interpolation
of filename when running jsc.

* Scripts/test262/Runner.pm:
(processCLI):
(main):
(runTest):

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

Tools/ChangeLog
Tools/Scripts/test262/Runner.pm

index c83bd12..7b8c63a 100644 (file)
@@ -1,3 +1,18 @@
+2018-05-03  Valerie R Young  <valerie@bocoup.com>
+
+        test262/Runner.pm: minor fixes
+        https://bugs.webkit.org/show_bug.cgi?id=185250
+
+        Reviewed by Michael Saboff.
+
+        Add comments, change output and fix bug: unintended interpolation
+        of filename when running jsc.
+
+        * Scripts/test262/Runner.pm:
+        (processCLI):
+        (main):
+        (runTest):
+
 2018-05-03  Chris Dumez  <cdumez@apple.com>
 
         Load hangs if the WebProcess fails to launch
index 8d3d19f..67cd8ba 100755 (executable)
@@ -61,9 +61,8 @@ use YAML qw(Load LoadFile Dump DumpFile Bless);
 use Parallel::ForkManager;
 use Getopt::Long qw(GetOptions);
 use Pod::Usage;
-use Term::ANSIColor;
 
-# Commandline args
+# Commandline settings
 my $max_process;
 my @cliTestDirs;
 my $verbose;
@@ -79,7 +78,11 @@ my $saveExpectations;
 my $failingOnly;
 my $latestImport;
 my $runningAllTests;
+
+processCLI();
+
 my @results;
+my @files;
 
 my $expectationsFile = abs_path("$Bin/expectations.yaml");
 my $configFile = abs_path("$Bin/config.yaml");
@@ -87,8 +90,6 @@ my $resultsFile = abs_path("$Bin/results.yaml");
 my $summaryTxtFile = abs_path("$Bin/results-summary.txt");
 my $summaryFile = abs_path("$Bin/results-summary.yaml");
 
-processCLI();
-
 my $tempdir = tempdir();
 
 my @default_harnesses = (
@@ -98,8 +99,6 @@ my @default_harnesses = (
     "$Bin/agent.js"
 );
 
-my @files;
-
 my ($deffh, $deffile) = getTempFile();
 print $deffh getHarness(<@default_harnesses>);
 
@@ -156,8 +155,6 @@ sub processCLI {
         }
     } else {
         $JSC = getBuildPath($debug);
-
-        print("Using the following jsc path: $JSC\n");
     }
 
     if ($latestImport) {
@@ -219,6 +216,8 @@ sub processCLI {
 }
 
 sub main {
+
+    # If not commandline test path supplied, use the root directory of all tests.
     push(@cliTestDirs, 'test') if not @cliTestDirs;
 
     if ($latestImport) {
@@ -242,7 +241,7 @@ sub main {
 
 
     # If we are processing many files, fork process
-    if (scalar @files > $max_process * 10) {
+    if (scalar @files > $max_process * 5) {
 
         # Make temporary files to record results
         my @resultsfhs;
@@ -578,7 +577,7 @@ sub runTest {
     my $defaultHarness = '';
     $defaultHarness = $deffile if $scenario ne 'raw';
 
-    my $result = qx/$JSC $args $defaultHarness $includesfile $prefixFile$filename/;
+    my $result = qx/$JSC $args $defaultHarness $includesfile '$prefixFile$filename'/;
 
     chomp $result;
 
@@ -857,7 +856,7 @@ Specify one or more specific test262 directory of test to run, relative to the r
 
 =item B<--save, -s>
 
-Overwrites the test262-expectations.yaml and test262-results.yaml file with the current list of test262 files and test results.
+Overwrites the test262-expectations.yaml file with the current list of test262 files and test results.
 
 =item B<--ignore-expectations, -x>