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

Unified Diff: tests/standalone/io/process_run_output_test.dart

Issue 15883003: Remove ProcessOptions and make the options named arguments. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Comments cleanup. Created 7 years, 7 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 side-by-side diff with in-line comments
Download patch
Index: tests/standalone/io/process_run_output_test.dart
diff --git a/tests/standalone/io/process_run_output_test.dart b/tests/standalone/io/process_run_output_test.dart
index 5dd731bb3723272803621f3c57f68539aec4bc47..6bdbf447212e086e5426aeb14427e8ac32d9e610 100644
--- a/tests/standalone/io/process_run_output_test.dart
+++ b/tests/standalone/io/process_run_output_test.dart
@@ -29,21 +29,18 @@ test(scriptFile, encoding, stream) {
enc = Encoding.UTF_8;
}
- var options = new ProcessOptions();
if (stream == 'stdout') {
- options.stdoutEncoding = enc;
Process.run(new Options().executable,
[scriptFile, encoding, stream],
- options). then((result) {
+ stdoutEncoding: enc). then((result) {
Expect.equals(result.exitCode, 0);
Expect.equals(result.stderr, '');
checkOutput(encoding, result.stdout);
});
} else {
- options.stderrEncoding = enc;
Process.run(new Options().executable,
[scriptFile, encoding, stream],
- options).then((result) {
+ stderrEncoding: enc).then((result) {
Expect.equals(result.exitCode, 0);
Expect.equals(result.stdout, '');
checkOutput(encoding, result.stderr);

Powered by Google App Engine
This is Rietveld 408576698