Title: [232343] trunk/Tools
Revision
232343
Author
[email protected]
Date
2018-05-31 03:20:54 -0700 (Thu, 31 May 2018)

Log Message

test262-runner should run in verbose mode in automation
https://bugs.webkit.org/show_bug.cgi?id=186115

Patch by Leo Balter <[email protected]> on 2018-05-31
Reviewed by Yusuke Suzuki.

This patch toggles the verbose mode in the build bot and reports every test file
in order to avoid the '1200 seconds without output' issue.
* BuildSlaveSupport/build.webkit.org-config/steps.py:
(RunTest262Tests):
* Scripts/test262/Runner.pm:
(processResult):

Modified Paths

Diff

Modified: trunk/Tools/BuildSlaveSupport/build.webkit.org-config/steps.py (232342 => 232343)


--- trunk/Tools/BuildSlaveSupport/build.webkit.org-config/steps.py	2018-05-31 07:10:50 UTC (rev 232342)
+++ trunk/Tools/BuildSlaveSupport/build.webkit.org-config/steps.py	2018-05-31 10:20:54 UTC (rev 232343)
@@ -367,7 +367,7 @@
     description = ["test262-tests running"]
     descriptionDone = ["test262-tests"]
     failedTestsFormatString = "%d Test262 test%s failed"
-    command = ["perl", "./Tools/Scripts/test262-runner", WithProperties("--%(configuration)s")]
+    command = ["perl", "./Tools/Scripts/test262-runner", "--verbose", WithProperties("--%(configuration)s")]
 
     def start(self):
         appendCustomBuildFlags(self, self.getProperty('platform'), self.getProperty('fullPlatform'))

Modified: trunk/Tools/ChangeLog (232342 => 232343)


--- trunk/Tools/ChangeLog	2018-05-31 07:10:50 UTC (rev 232342)
+++ trunk/Tools/ChangeLog	2018-05-31 10:20:54 UTC (rev 232343)
@@ -1,3 +1,17 @@
+2018-05-31  Leo Balter  <[email protected]>
+
+        test262-runner should run in verbose mode in automation
+        https://bugs.webkit.org/show_bug.cgi?id=186115
+
+        Reviewed by Yusuke Suzuki.
+
+        This patch toggles the verbose mode in the build bot and reports every test file
+        in order to avoid the '1200 seconds without output' issue.
+        * BuildSlaveSupport/build.webkit.org-config/steps.py:
+        (RunTest262Tests):
+        * Scripts/test262/Runner.pm:
+        (processResult):
+
 2018-05-30  Fujii Hironori  <[email protected]>
 
         [Win][MiniBrowser] Move MainWindow related code from Common.cpp to MainWindow.cpp

Modified: trunk/Tools/Scripts/test262/Runner.pm (232342 => 232343)


--- trunk/Tools/Scripts/test262/Runner.pm	2018-05-31 07:10:50 UTC (rev 232342)
+++ trunk/Tools/Scripts/test262/Runner.pm	2018-05-31 10:20:54 UTC (rev 232343)
@@ -683,7 +683,7 @@
         if ($verbose) {
             my $featuresList = '';
             $featuresList = "\nFeatures: " . join(', ', @{ $data->{features} }) if $data->{features};
-            $suffixMsg = "$result$featuresList\n\n";
+            $suffixMsg = "$result$featuresList\n";
         }
 
         print "$newFail$failMsg$suffixMsg";
@@ -693,11 +693,15 @@
     } elsif ($scenario ne 'skip' && !$currentfailure) {
         if ($expectedfailure) {
             print "NEW PASS $file ($scenario)\n";
-            print "\n" if $verbose;
+        } elsif ($verbose) {
+            print "PASS $file ($scenario)\n";
         }
 
         $resultdata{result} = 'PASS';
     } else {
+        if ($verbose) {
+            print "SKIP $file\n";
+        }
         $resultdata{result} = 'SKIP';
     }
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to