[GTK] WTF unit tests are timing out in the bots
authorcarlosgc@webkit.org <carlosgc@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 7 Aug 2015 15:28:01 +0000 (15:28 +0000)
committercarlosgc@webkit.org <carlosgc@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 7 Aug 2015 15:28:01 +0000 (15:28 +0000)
https://bugs.webkit.org/show_bug.cgi?id=147777

Reviewed by Filip Pizlo.

Add a way to mark google unit tests as slow and add
WTF_Lock.ContentedShortSection to the list. In case of slow test
we use the double of the given timeout for that particular test.

* Scripts/run-gtk-tests:
(TestRunner):
(TestRunner._run_google_test):

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

Tools/ChangeLog
Tools/Scripts/run-gtk-tests

index 96ec63e..66c95be 100644 (file)
@@ -1,5 +1,20 @@
 2015-08-07  Carlos Garcia Campos  <cgarcia@igalia.com>
 
+        [GTK] WTF unit tests are timing out in the bots
+        https://bugs.webkit.org/show_bug.cgi?id=147777
+
+        Reviewed by Filip Pizlo.
+
+        Add a way to mark google unit tests as slow and add
+        WTF_Lock.ContentedShortSection to the list. In case of slow test
+        we use the double of the given timeout for that particular test.
+
+        * Scripts/run-gtk-tests:
+        (TestRunner):
+        (TestRunner._run_google_test):
+
+2015-08-07  Carlos Garcia Campos  <cgarcia@igalia.com>
+
         [GTK] run-gtk-tests should not stop when a google test case fails
         https://bugs.webkit.org/show_bug.cgi?id=147778
 
index 7205e44..22d9ad2 100755 (executable)
@@ -80,6 +80,10 @@ class TestRunner:
         SkippedTest("WebKit2/TestWebKit2", "WebKit2.GeolocationTransitionToLowAccuracy", "Test causes crash on the next test", 125068),
     ]
 
+    SLOW = [
+        "WTF_Lock.ContentedShortSection",
+    ]
+
     def __init__(self, options, tests=[]):
         self._options = options
 
@@ -340,7 +344,10 @@ class TestRunner:
 
     def _run_google_test(self, test_program, subtest):
         test_command = [test_program, '--gtest_filter=%s' % (subtest)]
-        return self._run_test_command(test_command, self._options.timeout)
+        timeout = self._options.timeout
+        if subtest in TestRunner.SLOW:
+            timeout *= 2
+        return self._run_test_command(test_command, timeout)
 
     def _run_google_test_suite(self, test_program):
         retcode = 0