[GTK] run-gtk-tests should not stop when a google test case fails
authorcarlosgc@webkit.org <carlosgc@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 7 Aug 2015 10:27:27 +0000 (10:27 +0000)
committercarlosgc@webkit.org <carlosgc@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 7 Aug 2015 10:27:27 +0000 (10:27 +0000)
https://bugs.webkit.org/show_bug.cgi?id=147778

Reviewed by Philippe Normand.

For glib based tests we run the test runner with the -k option,
but for google tests we are aborting as soon as we find a failure.

* Scripts/run-gtk-tests:
(TestRunner._run_google_test_suite): Use a global return code that
is set to 1 when any test has failed.

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

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

index 6dd59ff..96ec63e 100644 (file)
@@ -1,5 +1,19 @@
 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
+
+        Reviewed by Philippe Normand.
+
+        For glib based tests we run the test runner with the -k option,
+        but for google tests we are aborting as soon as we find a failure.
+
+        * Scripts/run-gtk-tests:
+        (TestRunner._run_google_test_suite): Use a global return code that
+        is set to 1 when any test has failed.
+
+2015-08-07  Carlos Garcia Campos  <cgarcia@igalia.com>
+
         [GStreamer] Do not automatically show PackageKit codec installation notifications
         https://bugs.webkit.org/show_bug.cgi?id=135973
 
index 9aa5252..7205e44 100755 (executable)
@@ -343,11 +343,11 @@ class TestRunner:
         return self._run_test_command(test_command, self._options.timeout)
 
     def _run_google_test_suite(self, test_program):
+        retcode = 0
         for subtest in self._get_tests_from_google_test_suite(test_program):
-            retcode = self._run_google_test(test_program, subtest)
-            if retcode:
-                return retcode
-        return 0
+            if self._run_google_test(test_program, subtest):
+                retcode = 1
+        return retcode
 
     def _run_test(self, test_program):
         basedir = os.path.basename(os.path.dirname(test_program))