Performance tester bots should report the number of failing tests

https://bugs.webkit.org/show_bug.cgi?id=120462

Patch by Gabor Abraham <abrhm@inf.u-szeged.hu> on 2013-09-11
Reviewed by Csaba Osztrogonác.

* BuildSlaveSupport/build.webkit.org-config/master.cfg:
(RunAndUploadPerfTests.start.getText):
(RunAndUploadPerfTests.start):
(RunAndUploadPerfTests):
(RunAndUploadPerfTests.start.getText2):

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

diff --git a/Tools/BuildSlaveSupport/build.webkit.org-config/master.cfg b/Tools/BuildSlaveSupport/build.webkit.org-config/master.cfg
index 1ed72f0e4214670dfc4c4eb64489991ea31db83d..d177c3d13e8992dc6c667e7a0b08628512286dd1 100644 (file)
--- a/Tools/BuildSlaveSupport/build.webkit.org-config/master.cfg
+++ b/Tools/BuildSlaveSupport/build.webkit.org-config/master.cfg
@@ -671,6 +671,15 @@ class RunAndUploadPerfTests(shell.Test):
        self.setCommand(self.command)
        return shell.Test.start(self)
 
+    def getText(self, cmd, results):
+        return self.getText2(cmd, results)
+
+    def getText2(self, cmd, results):
+        if results != SUCCESS
+            return ["%d perf tests failed" % cmd.rc]
+
+        return [self.name]
+
 
 class RunAndUploadPerfTestsWebKit2(RunAndUploadPerfTests):
     def start(self):

diff --git a/Tools/ChangeLog b/Tools/ChangeLog
index 7897e53756b994286cc15a9dafa17215b340a322..15668f14a259f6a32f213ddf2cd971587d4321f0 100644 (file)
--- a/Tools/ChangeLog
+++ b/Tools/ChangeLog
@@ -1,3 +1,16 @@
+2013-09-11  Gabor Abraham  <abrhm@inf.u-szeged.hu>
+
+        Performance tester bots should report the number of failing tests
+        https://bugs.webkit.org/show_bug.cgi?id=120462
+
+        Reviewed by Csaba Osztrogonác.
+
+        * BuildSlaveSupport/build.webkit.org-config/master.cfg:
+        (RunAndUploadPerfTests.start.getText):
+        (RunAndUploadPerfTests.start):
+        (RunAndUploadPerfTests):
+        (RunAndUploadPerfTests.start.getText2):
+
 2013-09-11  Krzysztof Czech  <k.czech@samsung.com>
 
         [ATK] Adds mapping MenuItemRadioRole to ATK