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

Unified Diff: tests/standalone/src/ProcessInvalidArgumentsTest.dart

Issue 9024008: Change the process API to be completely asynchronous. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Presubmit fixes Created 9 years 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/src/ProcessInvalidArgumentsTest.dart
diff --git a/tests/standalone/src/ProcessInvalidArgumentsTest.dart b/tests/standalone/src/ProcessInvalidArgumentsTest.dart
index 79c161b9ea7eef8fb0c4316e2979303cb7924bf6..3314de087a338d77a868cb00ef1521a2d12e8d4f 100644
--- a/tests/standalone/src/ProcessInvalidArgumentsTest.dart
+++ b/tests/standalone/src/ProcessInvalidArgumentsTest.dart
@@ -6,7 +6,7 @@
void testNonStringPath() {
Søren Gjesse 2011/12/22 12:49:28 Maybe Expect.throws(( ) => Process p = new Proces
Mads Ager (google) 2011/12/22 12:56:13 Done for all of them and inlined the three cases i
try {
- Process p = new Process(["true"], []);
+ Process p = new Process.start(["true"], []);
Expect.fail("Did not throw exception");
} catch(var e) {
Expect.isTrue(e is ProcessException, "Wrong exception type: $e");
@@ -15,7 +15,7 @@ void testNonStringPath() {
void testNonListArguments() {
try {
- Process p = new Process("true", "asdf");
+ Process p = new Process.start("true", "asdf");
Søren Gjesse 2011/12/22 12:49:28 Ditto.
Mads Ager (google) 2011/12/22 12:56:13 Done.
Expect.fail("Did not throw exception");
} catch(var e) {
Expect.isTrue(e is ProcessException, "Wrong exception type: $e");
@@ -24,7 +24,7 @@ void testNonListArguments() {
void testNonStringArgument() {
try {
- Process p = new Process("true", ["asdf", 1]);
+ Process p = new Process.start("true", ["asdf", 1]);
Søren Gjesse 2011/12/22 12:49:28 Ditto.
Mads Ager (google) 2011/12/22 12:56:13 Done.
Expect.fail("Did not throw exception");
} catch(var e) {
Expect.isTrue(e is ProcessException, "Wrong exception type: $e");

Powered by Google App Engine
This is Rietveld 408576698