[Android] Move GTest options to test_options_parser.py.

build/android/pylib/utils/test_options_parser.py

Index: build/android/pylib/utils/test_options_parser.py
diff --git a/build/android/pylib/test_options_parser.py b/build/android/pylib/utils/test_options_parser.py
similarity index 75%
rename from build/android/pylib/test_options_parser.py
rename to build/android/pylib/utils/test_options_parser.py
index 93382645c366a44d1de9cd82f0fef693f5933e60..f9917d9e9eec975a187e6f8c07c279bb7ccabc15 100644
--- a/build/android/pylib/test_options_parser.py
+++ b/build/android/pylib/utils/test_options_parser.py
@@ -4,7 +4,8 @@
 
 """Parses options for the instrumentation tests."""
 
-import constants
+from pylib import constants

[klundberg, 2013/01/03 21:04:07]

Does it make sense to move constants too?

I don't like the idea of having utils depend on pylib and pylib depend on utils.

[craigdh, 2013/01/03 21:13:12]

I don't like it either, but constants.py is used in a lot of places, so I think
that should be its own CL if we decide to move it.

@Frank, any input? Do people outside build/android depend on constants.py?
utils/ seems like it should only have private utilities not used outside of
build/android/.

+
 import optparse
 import os
 import sys
@@ -80,6 +81,57 @@ def AddTestRunnerOptions(option_parser, default_timeout=60):
   AddBuildTypeOption(option_parser)
 
 
+def AddGTestOptions(option_parser):

[frankf, 2013/01/03 21:17:23]

So the reason the other options are here, is that they're used by more than one
test runner. This is only used by run_tests.py. But I see the value of moving it
here for consistancy, and also to be able to see what options can be factored
out into AddTestRunnerOptions.

[craigdh, 2013/01/03 21:44:05]

Improving consistency between the various test runners in general should make
refactoring out common code much easier.

+  """Decorates OptionParser with GTest tests options."""
+
+  AddTestRunnerOptions(option_parser, default_timeout=0)
+  option_parser.add_option('-s', '--suite', dest='test_suite',
+                           help='Executable name of the test suite to run '
+                           '(use -s help to list them)')

[klundberg, 2013/01/03 21:04:07]

nit: period at end of sentence.

[craigdh, 2013/01/03 21:44:05]

Done.

+  option_parser.add_option('--out-directory', dest='out_directory',
+                           help='Path to the out/ directory, irrespective of '
+                           'the build type. Only for non-Chromium uses.')
+  option_parser.add_option('-d', '--device', dest='test_device',
+                           help='Target device the test suite to run ')

[klundberg, 2013/01/03 21:04:07]

Seems like something is missing in this sentence. Maybe it should be:
Target device for the test suite to run on. 

[craigdh, 2013/01/03 21:44:05]

Done.

+  option_parser.add_option('-f', '--gtest_filter', dest='gtest_filter',
+                           help='gtest filter')

[klundberg, 2013/01/03 21:04:07]

nit: period at end of sentence.

[craigdh, 2013/01/03 21:44:05]

Done.

[craigdh, 2013/01/03 21:44:05]

Done.

+  option_parser.add_option('-a', '--test_arguments', dest='test_arguments',
+                           help='Additional arguments to pass to the test')

[klundberg, 2013/01/03 21:04:07]

nit: period at end of sentence.

[craigdh, 2013/01/03 21:44:05]

Done.

+  option_parser.add_option('-L', dest='log_dump',
+                           help='file name of log dump, which will be put in '

[klundberg, 2013/01/03 21:04:07]

file/File

[craigdh, 2013/01/03 21:44:05]

Done.

+                           'subfolder debug_info_dumps under the same '
+                           'directory in where the test_suite exists.')
+  option_parser.add_option('-e', '--emulator', dest='use_emulator',
+                           action='store_true',
+                           help='Run tests in a new instance of emulator')

[klundberg, 2013/01/03 21:04:07]

nit: period at end of sentence.

[craigdh, 2013/01/03 21:44:05]

Done.

+  option_parser.add_option('-n', '--emulator_count',

[frankf, 2013/01/03 21:17:23]

For reference: at some point (not in this CL), it'd be good to make all options
params to use '-' not '_'.

[craigdh, 2013/01/03 21:44:05]

Added a TODO.

+                           type='int', default=1,
+                           help='Number of emulators to launch for running the '
+                           'tests.')
+  option_parser.add_option('-x', '--xvfb', dest='use_xvfb',
+                           action='store_true',
+                           help='Use Xvfb around tests (ignored if not Linux)')

[klundberg, 2013/01/03 21:04:07]

nit: period at end of sentence.

[craigdh, 2013/01/03 21:44:05]

Done.

+  option_parser.add_option('--webkit', action='store_true',
+                           help='Run the tests from a WebKit checkout.')
+  option_parser.add_option('--fast', '--fast_and_loose', dest='fast_and_loose',
+                           action='store_true',
+                           help='Go faster (but be less stable), '
+                           'for quick testing.  Example: when tracking down '
+                           'tests that hang to add to the disabled list, '
+                           'there is no need to redeploy the test binary '
+                           'or data to the device again.  '
+                           'Don\'t use on bots by default!')
+  option_parser.add_option('--repeat', dest='repeat', type='int',
+                           default=2,
+                           help='Repeat count on test timeout')

[klundberg, 2013/01/03 21:04:07]

nit: period at end of sentence.

[craigdh, 2013/01/03 21:44:05]

Done.

+  option_parser.add_option('--exit_code', action='store_true',
+                           help='If set, the exit code will be total number '
+                           'of failures.')
+  option_parser.add_option('--exe', action='store_true',
+                           help='If set, use the exe test runner instead of '
+                           'the APK.')
+
+
 def AddInstrumentationOptions(option_parser):
   """Decorates OptionParser with instrumentation tests options."""