Pass command line args to apptests

mojo/tools/apptest_runner.py

Index: mojo/tools/apptest_runner.py
diff --git a/mojo/tools/apptest_runner.py b/mojo/tools/apptest_runner.py
index d84dd234af44f0eaf755d47211a7f703f39ad357..a0b524570a84c16b2fa714aa692160cc112a991e 100755
--- a/mojo/tools/apptest_runner.py
+++ b/mojo/tools/apptest_runner.py
@@ -64,7 +64,12 @@ def main(argv):
 
     test_list = gtest_list_tests(list_output)
     for test in test_list:
-      command_line = [mojo_shell, apptest + " --gtest_filter=" + test]
+      # TODO(msw): enable passing arguments to tests down from the test harness.
+      command_line = [
+          mojo_shell,
+          "--args-for={0} --example_apptest_arg --gtest_filter={1}".format(

[msw, 2014/11/22 18:23:39]

Is this quoting correct? I think it should be --args-for="..."

[Chris Masone, 2014/11/22 20:14:44]

Since the command isn't being interpreted by a shell when we run it below, I
believe this is the right quoting.

[msw, 2014/11/22 21:46:54]

I think you still want escaped quotes around the value of the args-for switch.

[Chris Masone, 2014/11/24 17:08:22]

No, then you wind up with the quotes embedded in the contents of --args-for, and
mojo_shell complains thusly:

[ERROR:mojo_main.cc(47)] Error: invalid URL: "mojo:example_apptests

and you wind up with no args because of this error.

[msw, 2014/11/24 19:05:20]

Odd, both quoting styles work with direct command line usage, but I confirmed
your findings wrt this invocation.

+              apptest, test),
+          apptest]
 
       _logging.debug("Running %s..." % command_line)
       try: