Title: [231874] trunk/Tools
Revision
231874
Author
commit-qu...@webkit.org
Date
2018-05-16 15:15:49 -0700 (Wed, 16 May 2018)

Log Message

Test262-Runner: Adds a --timeout option
https://bugs.webkit.org/show_bug.cgi?id=185696

Patch by Leo Balter <leonardo.bal...@gmail.com> on 2018-05-16
Reviewed by Michael Saboff.

The timeout option is used to set a timeout execution in ms for each test run in JSC.
The passed value - if defined - will be used as the value for the JSC --watchdog arg.
* Scripts/test262/Runner.pm:
(processCLI):
(runTest):

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (231873 => 231874)


--- trunk/Tools/ChangeLog	2018-05-16 21:46:36 UTC (rev 231873)
+++ trunk/Tools/ChangeLog	2018-05-16 22:15:49 UTC (rev 231874)
@@ -1,5 +1,18 @@
 2018-05-16  Leo Balter  <leonardo.bal...@gmail.com>
 
+        Test262-Runner: Adds a --timeout option
+        https://bugs.webkit.org/show_bug.cgi?id=185696
+
+        Reviewed by Michael Saboff.
+
+        The timeout option is used to set a timeout execution in ms for each test run in JSC.
+        The passed value - if defined - will be used as the value for the JSC --watchdog arg.
+        * Scripts/test262/Runner.pm:
+        (processCLI):
+        (runTest):
+
+2018-05-16  Leo Balter  <leonardo.bal...@gmail.com>
+
         Test262-Runner: Set timer for each test run
         https://bugs.webkit.org/show_bug.cgi?id=185692
 

Modified: trunk/Tools/Scripts/test262/Runner.pm (231873 => 231874)


--- trunk/Tools/Scripts/test262/Runner.pm	2018-05-16 21:46:36 UTC (rev 231873)
+++ trunk/Tools/Scripts/test262/Runner.pm	2018-05-16 22:15:49 UTC (rev 231874)
@@ -83,6 +83,7 @@
 my $failingOnly;
 my $latestImport;
 my $runningAllTests;
+my $timeout;
 
 my $expectationsFile = abs_path("$Bin/../../../JSTests/test262/expectations.yaml");
 my $configFile = abs_path("$Bin/../../../JSTests/test262/config.yaml");
@@ -135,6 +136,7 @@
         'l|latest-import' => \$latestImport,
         'stats' => \$stats,
         'r|results=s' => \$specifiedResultsFile,
+        'timeout=i' => \$timeout,
     );
 
     if ($help) {
@@ -234,8 +236,9 @@
         . "DYLD_FRAMEWORK_PATH: $DYLD_FRAMEWORK_PATH\n"
         . "Child Processes: $max_process\n";
 
+    print "Test timeout: $timeout\n" if $timeout;
     print "Features to include: " . join(', ', @features) . "\n" if @features;
-    print "Paths:  " . join(', ', @cliTestDirs) . "\n" if @cliTestDirs;
+    print "Paths: " . join(', ', @cliTestDirs) . "\n" if @cliTestDirs;
     print "Config file: $configFile\n" if $config;
     print "Expectations file: $expectationsFile\n" if $expect;
     print "Results file: $resultsFile\n" if $stats || $failingOnly;
@@ -597,6 +600,10 @@
 
     my $args = '';
 
+    if ($timeout) {
+        $args .= " --watchdog=$timeout ";
+    }
+
     if (exists $data->{negative}) {
         my $type = $data->{negative}->{type};
         $args .=  " --exception=$type ";
@@ -957,6 +964,10 @@
 
 Specifies a results file for the --stats or --failing-files options.
 
+=item B<--timeout>
+
+Specifies a timeout execution in ms for each test. Defers the value to the jsc --watchdog argument. Disabled by default.
+
 =back
 
 =cut
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to