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

Unified Diff: tests/standalone/io/process_non_ascii_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_non_ascii_test.dart
diff --git a/tests/standalone/io/process_non_ascii_test.dart b/tests/standalone/io/process_non_ascii_test.dart
index 020b07e075f390490dfa14b3bafcdb3209a6d468..92e3a8bb031abdff959f23abe400030b73a2b09f 100644
--- a/tests/standalone/io/process_non_ascii_test.dart
+++ b/tests/standalone/io/process_non_ascii_test.dart
@@ -25,12 +25,11 @@ main() {
""");
var nonAsciiTxtFile = new File('${nonAsciiDir.path}/æøå.txt');
nonAsciiTxtFile.writeAsStringSync('æøå');
- var options = new ProcessOptions();
- options.workingDirectory = nonAsciiDir.path;
var script = nonAsciiFile.path;
- Process.run(executable, [script], options).then((result) {
- Expect.equals(0, result.exitCode);
- tempDir.deleteSync(recursive: true);
- port.close();
- });
+ Process.run(executable, [script], workingDirectory: nonAsciiDir.path)
+ .then((result) {
+ Expect.equals(0, result.exitCode);
+ tempDir.deleteSync(recursive: true);
+ port.close();
+ });
}

Powered by Google App Engine
This is Rietveld 408576698