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

Unified Diff: tests/standalone/io/process_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_environment_test.dart
diff --git a/tests/standalone/io/process_environment_test.dart b/tests/standalone/io/process_environment_test.dart
index 30e523b7a9aeec2457c73f30ed9d30aa9a09bc54..24cc73d326576fe4dfee0d53038029952dafe6bf 100644
--- a/tests/standalone/io/process_environment_test.dart
+++ b/tests/standalone/io/process_environment_test.dart
@@ -9,16 +9,17 @@ import "process_test_util.dart";
runEnvironmentProcess(Map environment, name, callback) {
var dartExecutable = new Options().executable;
- var options = new ProcessOptions();
- options.environment = environment;
var printEnv = 'tests/standalone/io/print_env.dart';
if (!new File(printEnv).existsSync()) {
printEnv = '../$printEnv';
}
- Process.run(dartExecutable, [printEnv, name], options).then((result) {
- Expect.equals(0, result.exitCode);
- callback(result.stdout);
- });
+ Process.run(dartExecutable,
+ [printEnv, name],
+ environment: environment)
+ .then((result) {
+ Expect.equals(0, result.exitCode);
+ callback(result.stdout);
+ });
}
testEnvironment() {

Powered by Google App Engine
This is Rietveld 408576698