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

Unified Diff: tests/standalone/src/io/ProcessStartExceptionTest.dart

Issue 9863015: This is a request for comments. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Address comments. Created 8 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
« no previous file with comments | « tests/standalone/src/io/ProcessSegfaultTest.dart ('k') | tests/standalone/src/io/ProcessStdIOScript2.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/standalone/src/io/ProcessStartExceptionTest.dart
diff --git a/tests/standalone/src/io/ProcessStartExceptionTest.dart b/tests/standalone/src/io/ProcessStartExceptionTest.dart
index 886f6ba76e32d0cbf41107baef4b7587e81f7982..1843c431e6c03abaa3a8214be5bd8098768a9309 100644
--- a/tests/standalone/src/io/ProcessStartExceptionTest.dart
+++ b/tests/standalone/src/io/ProcessStartExceptionTest.dart
@@ -2,27 +2,40 @@
// for details. All rights reserved. Use of this source code is governed by a
// BSD-style license that can be found in the LICENSE file.
//
-// Process test program to test process communication.
+// Process test program to errors during startup of the process.
#import("dart:io");
-class ProcessStartExceptionTest {
+testStartError() {
+ Process process =
+ new Process.start("__path_to_something_that_should_not_exist__",
+ const []);
- static void testStartError() {
- Process process =
- new Process.start("__path_to_something_that_should_not_exist__",
- const []);
+ process.onExit = (int exitCode) {
+ Expect.fail("exit handler called");
+ };
- process.onExit = (int exitCode) {
- Expect.fail("exit handler called");
- };
+ process.onError = (ProcessException e) {
+ Expect.equals(2, e.errorCode, e.toString());
+ };
+}
+
+
+testRunError() {
+ Process process =
+ new Process.run("__path_to_something_that_should_not_exist__",
+ const [],
+ null,
+ (exit, out, err) {
+ Expect.fail("exit handler called");
+ });
- process.onError = (ProcessException e) {
- Expect.equals(2, e.errorCode, e.toString());
- };
- }
+ process.onError = (ProcessException e) {
+ Expect.equals(2, e.errorCode, e.toString());
+ };
}
main() {
- ProcessStartExceptionTest.testStartError();
+ testStartError();
+ testRunError();
}
« no previous file with comments | « tests/standalone/src/io/ProcessSegfaultTest.dart ('k') | tests/standalone/src/io/ProcessStdIOScript2.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698