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

Side by Side Diff: tests/standalone/io/process_stderr_test.dart

Issue 2687963003: [dart:io] flush() stdin before close()ing (Closed)
Patch Set: Created 3 years, 10 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2013, 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 process communication. 5 // Process test program to test process communication.
6 // 6 //
7 // VMOptions= 7 // VMOptions=
8 // VMOptions=--short_socket_read 8 // VMOptions=--short_socket_read
9 // VMOptions=--short_socket_write 9 // VMOptions=--short_socket_write
10 // VMOptions=--short_socket_read --short_socket_write 10 // VMOptions=--short_socket_read --short_socket_write
(...skipping 30 matching lines...) Expand all
41 if (13 == buffer[dataSize - 1] && dataSize + 1 == received) { 41 if (13 == buffer[dataSize - 1] && dataSize + 1 == received) {
42 Expect.equals(13, buffer[dataSize - 1]); 42 Expect.equals(13, buffer[dataSize - 1]);
43 Expect.equals(10, buffer[dataSize]); 43 Expect.equals(10, buffer[dataSize]);
44 buffer.removeLast(); 44 buffer.removeLast();
45 } 45 }
46 } 46 }
47 } 47 }
48 48
49 process.stdout.listen((_) {}); 49 process.stdout.listen((_) {});
50 process.stdin.add(data); 50 process.stdin.add(data);
51 process.stdin.close(); 51 process.stdin.flush().then((_) => process.stdin.close());
52 process.stderr.listen(readData); 52 process.stderr.listen(readData);
53 }); 53 });
54 } 54 }
55 55
56 main() { 56 main() {
57 // Run the test using the process_test binary. 57 // Run the test using the process_test binary.
58 test(Process.start(getProcessTestFileName(), 58 test(Process.start(getProcessTestFileName(),
59 const ["1", "1", "99", "0"]), 99); 59 const ["1", "1", "99", "0"]), 99);
60 60
61 // Run the test using the dart binary with an echo script. 61 // Run the test using the dart binary with an echo script.
62 // The test runner can be run from either the root or from runtime. 62 // The test runner can be run from either the root or from runtime.
63 var scriptFile = new File("tests/standalone/io/process_std_io_script.dart"); 63 var scriptFile = new File("tests/standalone/io/process_std_io_script.dart");
64 if (!scriptFile.existsSync()) { 64 if (!scriptFile.existsSync()) {
65 scriptFile = new File("../tests/standalone/io/process_std_io_script.dart"); 65 scriptFile = new File("../tests/standalone/io/process_std_io_script.dart");
66 } 66 }
67 Expect.isTrue(scriptFile.existsSync()); 67 Expect.isTrue(scriptFile.existsSync());
68 test(Process.start(Platform.executable, [scriptFile.path, "1"]), 0); 68 test(Process.start(Platform.executable, [scriptFile.path, "1"]), 0);
69 } 69 }
OLDNEW
« no previous file with comments | « tests/standalone/io/process_detached_test.dart ('k') | tests/standalone/io/process_stdout_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698