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

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

Issue 2688243003: Add call to flush() to avoid flakiness (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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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
11 11
12 import "package:expect/expect.dart"; 12 import "package:expect/expect.dart";
13 import 'dart:async'; 13 import 'dart:async';
14 import 'dart:io'; 14 import 'dart:io';
15 import 'dart:math'; 15 import 'dart:math';
16 16
17 import "process_test_util.dart"; 17 import "process_test_util.dart";
18 18
19 void test(Future<Process> future, int expectedExitCode) { 19 void test(Future<Process> future, int expectedExitCode) {
20 future.then((process) { 20 future.then((process) {
21 process.exitCode.then((exitCode) { 21 process.exitCode.then((exitCode) {
22 Expect.equals(expectedExitCode, exitCode); 22 Expect.equals(expectedExitCode, exitCode);
23 }); 23 });
24 24
25 process.stdout.listen((_) {}); 25 process.stdout.listen((_) {});
26 process.stderr.listen((_) {}); 26 process.stderr.listen((_) {});
27 process.stdin.writeln("Line1"); 27 process.stdin.writeln("Line1");
28 process.stdin.flush().then((_) {
29 print("flush completed");
30 });
28 }); 31 });
29 } 32 }
30 33
31 main() { 34 main() {
32 var scriptName = "process_stdin_transform_unsubscribe_script.dart"; 35 var scriptName = "process_stdin_transform_unsubscribe_script.dart";
33 var scriptFile = new File("tests/standalone/io/$scriptName"); 36 var scriptFile = new File("tests/standalone/io/$scriptName");
34 if (!scriptFile.existsSync()) { 37 if (!scriptFile.existsSync()) {
35 scriptFile = new File("../tests/standalone/io/$scriptName"); 38 scriptFile = new File("../tests/standalone/io/$scriptName");
36 } 39 }
37 Expect.isTrue(scriptFile.existsSync()); 40 Expect.isTrue(scriptFile.existsSync());
38 test(Process.start(Platform.executable, [scriptFile.path]), 0); 41 test(Process.start(Platform.executable, [scriptFile.path]), 0);
39 } 42 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698