Chromium Code Reviews| 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 = |