Remove ProcessOptions and make the options named arguments.

tests/standalone/io/process_shell_test.dart

 // Copyright (c) 2013, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 import "dart:io";
+import "dart:isolate";
 
 void testRunShell() {
   test(args) {
     var options = new Options();
     var path = new Path(options.script);
     path = path.directoryPath.join(new Path("process_echo_util.dart"));
-    Process.runShell(options.executable, [path.toString()]..addAll(args))
+    Process.run(options.executable,
+                [path.toString()]..addAll(args),
+                runInShell: true)
         .then((result) {
           if (Platform.operatingSystem == "windows") {
             result = result.stdout.split("\r\n");
           } else {
             result = result.stdout.split("\n");
           }
           if (result.length - 1 != args.length) {
             throw "wrong number of args: $args vs $result";
           }
           for (int i = 0; i < args.length; i++) {
             if (args[i] != result[i]) {
               throw "bad result at $i: '${args[i]}' vs '${result[i]}'";
             }
           }
         });
   }
   test(["\""]);
   test(["a b"]);
   test(["'"]);
+  test(["'", "'"]);
   test(["'\"\"'\"'\"'"]);
   test(["'\"\"'", "\"'\"'"]);
+  test(["'\\\"\\\"'\\", "\"\\'\"'"]);
   test(["'\$HOME'"]);
   test(["'\$tmp'"]);
+  test(["arg'"]);
+  test(["arg\\'", "'\\arg"]);
 }
 
-void testShell() {
-  test(args, expected) {
+void testBadRunShell() {

[Bill Hesse, 2013/07/22 15:36:29]

This change to testBadRunShell seems totally broken, and I'm not sure what it is
even supposed to do.  PTAL.

[Anders Johnsen, 2013/07/22 15:44:22]

It's supposed to test that you can't run the program '"' and '$HOME'. Nothing
fancy, just to make sure that the escaping goes through and that e.g. it's not
resolve to the empty string, that will complete with exitCode==0 on some
platforms.

[Bill Hesse, 2013/07/22 15:50:22]

OK, so it has nothing to do with the testShell() function it replaces.  It still
is dirty code, with dead variables, unused arguments, etc.  Is it really what
was intended?

I'm removing the path variable as part of removing Path, since it is dead code.

+  test(exe, [args = const []]) {
     var options = new Options();
     var path = new Path(options.script);
     path = path.directoryPath.join(new Path("process_echo_util.dart"));
-    var command = "${options.executable} $path $args";
-    Process.runShell(command, [])
+    Process.run(exe, args, runInShell: true)
         .then((result) {
-          if (Platform.operatingSystem == "windows") {
-            result = result.stdout.split("\r\n");
-          } else {
-            result = result.stdout.split("\n");
-          }
-          if (result.length - 1 != expected.length) {
-            throw "wrong number of args: $expected vs $result";
-          }
-          for (int i = 0; i < expected.length; i++) {
-            if (expected[i] != result[i]) {
-              throw "bad result at $i: ${expected[i]} vs ${result[i]}";
-            }
+          port.close();
+          if (result.exitCode == 0) {
+            throw "error expected";
           }
         });
   }
-  test("arg", ["arg"]);
-  test("arg1 arg2", ["arg1", "arg2"]);
-  if (Platform.operatingSystem != 'windows') {
-    test("arg1 arg2 > /dev/null", []);
-  }
+  test("\"");
+  test("'\$HOME'");
 }
 
 void main() {
   testRunShell();
-  testShell();
+  testBadRunShell();
 }