Chromium Code Reviews| OLD | NEW |
|---|---|
| 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 } |
| OLD | NEW |