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

Unified Diff: tests/standalone/debugger/debug_lib.dart

Issue 13724021: Remove deprecated Expect from the libraries. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 8 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
Index: tests/standalone/debugger/debug_lib.dart
diff --git a/tests/standalone/debugger/debug_lib.dart b/tests/standalone/debugger/debug_lib.dart
index 26f372508ee2d5b9a101cadc5a41b3f99ab6d6cd..76b6899774f685fa8b1694ddd4a637952127e93a 100644
--- a/tests/standalone/debugger/debug_lib.dart
+++ b/tests/standalone/debugger/debug_lib.dart
@@ -455,7 +455,7 @@ class Debugger {
targetProcess.kill();
print("Target process killed");
}
- Expect.isTrue(!errorsDetected);
+ if (errorsDetected) throw new Exception("Errors detected");
Lasse Reichstein Nielsen 2013/04/08 11:27:49 Why use "Exception" and not "Error"? Or perhaps ju
floitsch 2013/04/08 12:06:47 I wanted to use the Error class, but Error doesn't
floitsch 2013/04/08 16:06:59 Done.
exit(errors.length);
}
}
@@ -487,7 +487,7 @@ bool RunScript(List script) {
print("Debug target process started");
process.stdin.close();
process.exitCode.then((int exitCode) {
- Expect.equals(0, exitCode);
+ if (exitCode != 0) throw new Exception("bad exit code: $exitCode");
print("Debug target process exited with exit code $exitCode");
});
var debugger =

Powered by Google App Engine
This is Rietveld 408576698