Chromium Code Reviews| Index: tools/testing/dart/test_suite.dart |
| diff --git a/tools/testing/dart/test_suite.dart b/tools/testing/dart/test_suite.dart |
| index c47a60867276c3c753d3e31cc6554c45ab0ce3d4..d99ab213e29f7dcf73ffb14c3e96baf79f9a570e 100644 |
| --- a/tools/testing/dart/test_suite.dart |
| +++ b/tools/testing/dart/test_suite.dart |
| @@ -407,6 +407,11 @@ class StandardTestSuite implements TestSuite { |
| } |
| Set<String> expectations = testExpectations.expectations(testName); |
| + if (info.hasCompileError && |
| + TestUtils.isBrowserRuntime(configuration['runtime'])) { |
| + SummaryReport.addCompileErrorSkipTest(); |
| + return; |
| + } |
| if (configuration['report']) { |
| // Tests with multiple VMOptions are counted more than once. |
| for (var dummy in getVmOptions(optionsFromFile)) { |
| @@ -1446,6 +1451,7 @@ class SummaryReport { |
| static int fail = 0; |
| static int crash = 0; |
| static int timeout = 0; |
| + static int compileErrorSkip = 0; |
| static void add(Set<String> expectations) { |
| ++total; |
| @@ -1474,6 +1480,11 @@ class SummaryReport { |
| } |
| } |
| + static void addCompileErrorSkipTest() { |
| + total++; |
| + compileErrorSkip++; |
| + } |
| + |
| static void printReport() { |
| if (total == 0) return; |
| String report = """Total: $total tests |
| @@ -1484,6 +1495,7 @@ class SummaryReport { |
| * $fail tests are expected to fail that we should fix |
| * $crash tests are expected to crash that we should fix |
| * $timeout tests are allowed to timeout |
| + * $compileErrorSkip tests are skipped on compile-time error |
|
ahe
2012/11/06 14:25:57
"tests are skipped on browsers due to compile-time
kustermann
2012/11/06 14:43:06
Done.
|
| """; |
| print(report); |
| } |