Add a verbose flag to binding tests
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 3 Jun 2014 06:51:26 +0000 (06:51 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 3 Jun 2014 06:51:26 +0000 (06:51 +0000)
https://bugs.webkit.org/show_bug.cgi?id=122303

Patch by Tibor Meszaros <tmeszaros.u-szeged@partner.samsung.com> on 2014-06-02
Reviewed by Csaba Osztrogon√°c.

This patch will:
- Add a verbose keyword argument to the BindingsTests constructor,
- Add --no-verbose option to the run-bindigs-test.
Verbose mode, which is on by default, prints positive results to stdout as before.
When the --no-verbose option added to run-bindings-tests, positive results are not printed at all.

Original patch from Matthew Hanson <matthew_hanson@apple.com>

* Scripts/run-bindings-tests:
(main):
* Scripts/webkitpy/bindings/main.py:
(BindingsTests.__init__):
(BindingsTests.detect_changes):

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

Tools/ChangeLog
Tools/Scripts/run-bindings-tests
Tools/Scripts/webkitpy/bindings/main.py

index 07d7aa0..c7c811f 100644 (file)
@@ -1,3 +1,24 @@
+2014-06-02  Tibor Meszaros  <tmeszaros.u-szeged@partner.samsung.com>
+
+        Add a verbose flag to binding tests
+        https://bugs.webkit.org/show_bug.cgi?id=122303
+
+        Reviewed by Csaba Osztrogon√°c.
+
+        This patch will:
+        - Add a verbose keyword argument to the BindingsTests constructor,
+        - Add --no-verbose option to the run-bindigs-test.
+        Verbose mode, which is on by default, prints positive results to stdout as before.
+        When the --no-verbose option added to run-bindings-tests, positive results are not printed at all.
+
+        Original patch from Matthew Hanson <matthew_hanson@apple.com>
+
+        * Scripts/run-bindings-tests:
+        (main):
+        * Scripts/webkitpy/bindings/main.py:
+        (BindingsTests.__init__):
+        (BindingsTests.detect_changes):
+
 2014-06-01  Ryosuke Niwa  <rniwa@webkit.org>
 
         Rename DoYouEvenBench 0.17 to Speedometer 1.0 and add a new look.
 2014-06-01  Ryosuke Niwa  <rniwa@webkit.org>
 
         Rename DoYouEvenBench 0.17 to Speedometer 1.0 and add a new look.
index e70839d..d316fe2 100755 (executable)
@@ -40,6 +40,7 @@ def main(argv):
 
     """
     reset_results = "--reset-results" in argv
 
     """
     reset_results = "--reset-results" in argv
+    verbose = not "--no-verbose" in argv
 
     generators = [
         'JS',
 
     generators = [
         'JS',
@@ -49,7 +50,7 @@ def main(argv):
 
     from webkitpy.bindings.main import BindingsTests
 
 
     from webkitpy.bindings.main import BindingsTests
 
-    return BindingsTests(reset_results, generators, executive.Executive()).main()
+    return BindingsTests(reset_results, generators, executive.Executive(), verbose).main()
 
 
 if __name__ == '__main__':
 
 
 if __name__ == '__main__':
index 94b825c..b4a8bcb 100644 (file)
@@ -34,10 +34,11 @@ from webkitpy.common.system.executive import ScriptError
 
 class BindingsTests:
 
 
 class BindingsTests:
 
-    def __init__(self, reset_results, generators, executive):
+    def __init__(self, reset_results, generators, executive, verbose):
         self.reset_results = reset_results
         self.generators = generators
         self.executive = executive
         self.reset_results = reset_results
         self.generators = generators
         self.executive = executive
+        self.verbose = verbose
 
     def generate_from_idl(self, generator, idl_file, output_directory, supplemental_dependency_file):
         cmd = ['perl', '-w',
 
     def generate_from_idl(self, generator, idl_file, output_directory, supplemental_dependency_file):
         cmd = ['perl', '-w',
@@ -112,7 +113,7 @@ class BindingsTests:
                 print 'FAIL: (%s) %s' % (generator, output_file)
                 print output
                 changes_found = True
                 print 'FAIL: (%s) %s' % (generator, output_file)
                 print output
                 changes_found = True
-            else:
+            elif self.verbose:
                 print 'PASS: (%s) %s' % (generator, output_file)
         return changes_found
 
                 print 'PASS: (%s) %s' % (generator, output_file)
         return changes_found