Revision: 12632
Author:   [email protected]
Date:     Fri Sep 28 07:59:00 2012
Log:      test runner: fix wildcard handling in test status files

Review URL: https://codereview.chromium.org/10986086
http://code.google.com/p/v8/source/detail?r=12632

Modified:
 /branches/bleeding_edge/tools/testrunner/local/testsuite.py

=======================================
--- /branches/bleeding_edge/tools/testrunner/local/testsuite.py Mon Sep 24 02:38:46 2012 +++ /branches/bleeding_edge/tools/testrunner/local/testsuite.py Fri Sep 28 07:59:00 2012
@@ -96,14 +96,17 @@
         if statusfile.DoSkip(outcomes):
           continue  # Don't add skipped tests to |filtered|.
       if len(self.wildcards) != 0:
+        skip = False
         for rule in self.wildcards:
           assert rule[-1] == '*'
           if testname.startswith(rule[:-1]):
             used_rules.add(rule)
             outcomes = self.wildcards[rule]
-            if statusfile.DoSkip(outcomes):
-              continue
             t.outcomes = outcomes
+            if statusfile.DoSkip(outcomes):
+              skip = True
+              break  # "for rule in self.wildcards"
+        if skip: continue  # "for t in self.tests"
       filtered.append(t)
     self.tests = filtered

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to