Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(89)

Unified Diff: compiler/javatests/com/google/dart/corelib/SharedTestCase.java

Issue 9315036: Change output of test.dart --list option, so it can be parsed automatically. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: more changes Created 8 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | compiler/javatests/com/google/dart/corelib/SharedTests.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: compiler/javatests/com/google/dart/corelib/SharedTestCase.java
diff --git a/compiler/javatests/com/google/dart/corelib/SharedTestCase.java b/compiler/javatests/com/google/dart/corelib/SharedTestCase.java
index 81d745ae234a64a73e815a0badae3c8289fd3226..1f0a68470071a67e7f27a89ee2eef920111b909e 100644
--- a/compiler/javatests/com/google/dart/corelib/SharedTestCase.java
+++ b/compiler/javatests/com/google/dart/corelib/SharedTestCase.java
@@ -279,7 +279,9 @@ public abstract class SharedTestCase extends TestCase {
static Test getInstance(String line, boolean regularCompile) {
String[] fields = SEPARATOR.split(line);
- assertTrue(line, fields.length > 3);
+ if (fields.length <= 3) {
+ fail("SharedTestCase.java(284): Line is not parsable: " + line);
ahe 2012/02/03 11:57:36 No need to insert the filename and line number. Th
+ }
String name = fields[0];
Set<String> outcomes = new HashSet<String>(Arrays.<String>asList(fields[1].split(",")));
boolean isNegative = fields[2].equals("True");
« no previous file with comments | « no previous file | compiler/javatests/com/google/dart/corelib/SharedTests.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698