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

Unified Diff: tests/standalone/io/process_path_environment_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_path_environment_test.dart
diff --git a/tests/standalone/io/process_path_environment_test.dart b/tests/standalone/io/process_path_environment_test.dart
index 72f516508fba4180321b03a2abb68355dc5a9d38..80ec89b18b19e3382c17d6909b26bc0e570845ff 100644
--- a/tests/standalone/io/process_path_environment_test.dart
+++ b/tests/standalone/io/process_path_environment_test.dart
@@ -18,11 +18,10 @@ main() {
args = ['/C', 'echo', '"ok"'];
}
- var options = new ProcessOptions();
- options.environment = new Map.from(Platform.environment);
- options.environment['whatever'] = 'something';
+ var environment = new Map.from(Platform.environment);
+ environment['whatever'] = 'something';
- Process.run(executable, args, options).then((result) {
+ Process.run(executable, args, environment: environment).then((result) {
Expect.equals(0, result.exitCode);
});
}

Powered by Google App Engine
This is Rietveld 408576698