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

Unified Diff: compiler/javatests/com/google/dart/compiler/PrettyErrorFormatterTest.java

Issue 11418190: Issue 6969. Fix for reporting machine format for EOS error. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 1 month 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 | « compiler/java/com/google/dart/compiler/resolver/Resolver.java ('k') | tests/language/language.status » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: compiler/javatests/com/google/dart/compiler/PrettyErrorFormatterTest.java
diff --git a/compiler/javatests/com/google/dart/compiler/PrettyErrorFormatterTest.java b/compiler/javatests/com/google/dart/compiler/PrettyErrorFormatterTest.java
index 471654dbdaddc2918d124aa658c8bd710d7cedea..94839a9a8be157c0be1bbe3cccf9176c46c964ca 100644
--- a/compiler/javatests/com/google/dart/compiler/PrettyErrorFormatterTest.java
+++ b/compiler/javatests/com/google/dart/compiler/PrettyErrorFormatterTest.java
@@ -104,7 +104,7 @@ public class PrettyErrorFormatterTest extends TestCase {
String errorString = getErrorString(error, false, false);
assertEquals(
Joiner.on("\n").join(
- "my/path/Test.dart:1: no such type \"Foo\" (sourced from Test_app)",
+ "my/path/Test.dart:1:3: no such type \"Foo\" (sourced from Test_app)",
" 1: lineAAA",
" ~~~",
""),
@@ -122,7 +122,7 @@ public class PrettyErrorFormatterTest extends TestCase {
String errorString = getErrorString(error, false, false);
assertEquals(
Joiner.on("\n").join(
- "my/path/Test.dart:2: no such type \"Foo\" (sourced from Test_app)",
+ "my/path/Test.dart:2:3: no such type \"Foo\" (sourced from Test_app)",
" 1: lineAAA",
" 2: lineBBB",
" ~~~~~",
@@ -141,7 +141,7 @@ public class PrettyErrorFormatterTest extends TestCase {
assertEquals(
Joiner.on("\n").join(
WARNING_BOLD_COLOR
- + "my/path/Test.dart:2: no such type \"Foo\" (sourced from Test_app)"
+ + "my/path/Test.dart:2:3: no such type \"Foo\" (sourced from Test_app)"
+ NO_COLOR,
" 1: lineAAA",
" 2: li" + WARNING_COLOR + "neBBB" + NO_COLOR,
@@ -153,7 +153,7 @@ public class PrettyErrorFormatterTest extends TestCase {
assertEquals(
Joiner.on("\n").join(
ERROR_BOLD_COLOR
- + "my/path/Test.dart:2: no such type \"Foo\" (sourced from Test_app)"
+ + "my/path/Test.dart:2:3: no such type \"Foo\" (sourced from Test_app)"
+ NO_COLOR,
" 1: lineAAA",
" 2: li" + ERROR_COLOR + "neBBB" + NO_COLOR,
@@ -172,7 +172,7 @@ public class PrettyErrorFormatterTest extends TestCase {
String errorString = getErrorString(error, false, false);
assertEquals(
Joiner.on("\n").join(
- "my/path/Test.dart:2: no such type \"Foo\" (sourced from Test_app)",
+ "my/path/Test.dart:2:3: no such type \"Foo\" (sourced from Test_app)",
" 1: lineAAA",
" 2: lineBBB",
" ~~~",
@@ -191,7 +191,7 @@ public class PrettyErrorFormatterTest extends TestCase {
assertEquals(
Joiner.on("\n").join(
WARNING_BOLD_COLOR
- + "my/path/Test.dart:2: no such type \"Foo\" (sourced from Test_app)"
+ + "my/path/Test.dart:2:3: no such type \"Foo\" (sourced from Test_app)"
+ NO_COLOR,
" 1: lineAAA",
" 2: li" + WARNING_COLOR + "neB" + NO_COLOR + "BB",
@@ -203,7 +203,7 @@ public class PrettyErrorFormatterTest extends TestCase {
assertEquals(
Joiner.on("\n").join(
ERROR_BOLD_COLOR
- + "my/path/Test.dart:2: no such type \"Foo\" (sourced from Test_app)"
+ + "my/path/Test.dart:2:3: no such type \"Foo\" (sourced from Test_app)"
+ NO_COLOR,
" 1: lineAAA",
" 2: li" + ERROR_COLOR + "neB" + NO_COLOR + "BB",
« no previous file with comments | « compiler/java/com/google/dart/compiler/resolver/Resolver.java ('k') | tests/language/language.status » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698