Chromium Code Reviews| Index: tools/run-tests.py |
| diff --git a/tools/run-tests.py b/tools/run-tests.py |
| index ae56e756a00c193316cf4f06e68b5c1ca6f5f10a..b0e347da5b326fcf651449d98c1b40af8fc5137f 100755 |
| --- a/tools/run-tests.py |
| +++ b/tools/run-tests.py |
| @@ -301,9 +301,6 @@ def BuildOptions(): |
| result.add_option("--no-sorting", "--nosorting", |
| help="Don't sort tests according to duration of last run.", |
| default=False, dest="no_sorting", action="store_true") |
| - result.add_option("--no-stress", "--nostress", |
| - help="Don't run crankshaft --always-opt --stress-op test", |
| - default=False, dest="no_stress", action="store_true") |
| result.add_option("--no-variants", "--novariants", |
| help="Don't run any testing variants", |
| default=False, dest="no_variants", action="store_true") |
| @@ -349,9 +346,6 @@ def BuildOptions(): |
| help="Don't skip more slow tests when using a simulator.", |
| default=False, action="store_true", |
| dest="dont_skip_simulator_slow_tests") |
| - result.add_option("--stress-only", |
| - help="Only run tests with --always-opt --stress-opt", |
| - default=False, action="store_true") |
| result.add_option("--swarming", |
| help="Indicates running test driver on swarming.", |
| default=False, action="store_true") |
| @@ -545,23 +539,15 @@ def ProcessOptions(options): |
| """Returns true if zero or one of multiple arguments are true.""" |
| return reduce(lambda x, y: x + y, args) <= 1 |
| - if not excl(options.no_stress, options.stress_only, options.no_variants, |
| - bool(options.variants)): |
| - print("Use only one of --no-stress, --stress-only, --no-variants, " |
| - "or --variants.") |
| + if not excl(options.no_variants, bool(options.variants)): |
| + print("Use only one of --no-variants or --variants.") |
| return False |
| if options.quickcheck: |
| VARIANTS = ["default", "stress"] |
|
Michael Achenbach
2016/08/01 15:05:49
I'm also not so happy about this anymore, but I'll
|
| options.slow_tests = "skip" |
| options.pass_fail_tests = "skip" |
| - if options.no_stress: |
| - # FIXME(machenbach): This is not very intuitive anymore. Maybe remove a |
| - # bunch of these shortcuts and require stating the variants explicitly. |
| - VARIANTS = ["default", "nocrankshaft"] |
| if options.no_variants: |
| VARIANTS = ["default"] |
| - if options.stress_only: |
| - VARIANTS = ["stress"] |
| if options.variants: |
| VARIANTS = options.variants.split(",") |