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

Side by Side 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2011, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 // 4 //
5 // Process test program to test that invalid arguments throw exceptions. 5 // Process test program to test that invalid arguments throw exceptions.
6 6
7 void testNonStringPath() { 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
8 try { 8 try {
9 Process p = new Process(["true"], []); 9 Process p = new Process.start(["true"], []);
10 Expect.fail("Did not throw exception"); 10 Expect.fail("Did not throw exception");
11 } catch(var e) { 11 } catch(var e) {
12 Expect.isTrue(e is ProcessException, "Wrong exception type: $e"); 12 Expect.isTrue(e is ProcessException, "Wrong exception type: $e");
13 } 13 }
14 } 14 }
15 15
16 void testNonListArguments() { 16 void testNonListArguments() {
17 try { 17 try {
18 Process p = new Process("true", "asdf"); 18 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.
19 Expect.fail("Did not throw exception"); 19 Expect.fail("Did not throw exception");
20 } catch(var e) { 20 } catch(var e) {
21 Expect.isTrue(e is ProcessException, "Wrong exception type: $e"); 21 Expect.isTrue(e is ProcessException, "Wrong exception type: $e");
22 } 22 }
23 } 23 }
24 24
25 void testNonStringArgument() { 25 void testNonStringArgument() {
26 try { 26 try {
27 Process p = new Process("true", ["asdf", 1]); 27 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.
28 Expect.fail("Did not throw exception"); 28 Expect.fail("Did not throw exception");
29 } catch(var e) { 29 } catch(var e) {
30 Expect.isTrue(e is ProcessException, "Wrong exception type: $e"); 30 Expect.isTrue(e is ProcessException, "Wrong exception type: $e");
31 } 31 }
32 } 32 }
33 33
34 void main() { 34 void main() {
35 testNonStringPath(); 35 testNonStringPath();
36 testNonListArguments(); 36 testNonListArguments();
37 testNonStringArgument(); 37 testNonStringArgument();
38 } 38 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698