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

Unified Diff: scripts/slave/chromium/layout_test_wrapper.py

Issue 2459983002: Reland of Add initial CL for RandomOrder chromium.fyi bot (Closed)
Patch Set: Make --order default None Created 4 years, 2 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
« no previous file with comments | « masters/master.chromium.fyi/slaves.cfg ('k') | scripts/slave/recipe_modules/chromium_tests/chromium_fyi.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/slave/chromium/layout_test_wrapper.py
diff --git a/scripts/slave/chromium/layout_test_wrapper.py b/scripts/slave/chromium/layout_test_wrapper.py
index 7bf498e2a22c80164a9efea09c20039930881823..157079316d4ef2bd1ad01cef6e2a4b7921833e5b 100755
--- a/scripts/slave/chromium/layout_test_wrapper.py
+++ b/scripts/slave/chromium/layout_test_wrapper.py
@@ -96,6 +96,10 @@ def layout_test(options, args):
command.append('--enable-wptserve')
if options.batch_size:
command.extend(['--batch-size', options.batch_size])
+ if options.order:
+ command.extend(['--order', options.order])
+ if options.seed:
+ command.extend(['--seed', options.seed])
if options.run_part:
command.extend(['--run-part', options.run_part])
if options.builder_name:
@@ -202,6 +206,16 @@ def main():
default=None,
help=('Run a the tests in batches (n), after every '
'n tests, the test shell is relaunched.'))
+ option_parser.add_option('--order', action='store', default=None,
+ help=('Determine the order in which the test cases will be run. '
+ '"none" == use the order in which the tests were listed '
+ 'either in arguments or test list, '
+ '"natural" == use the natural order (default), '
+ '"random" == pseudo-random order. Seed can be specified '
+ 'via --seed, otherwise a default seed will be used.')),
+ option_parser.add_option('--seed', type='int', default=None,
+ help=('Seed to use for random test order. '
+ 'Only applicable in combination with --order=random.')),
qyearsley 2016/10/28 22:41:05 Note, fix is here
option_parser.add_option('--run-part',
default=None,
help=('Run a specified part (n:l), the nth of lth'
« no previous file with comments | « masters/master.chromium.fyi/slaves.cfg ('k') | scripts/slave/recipe_modules/chromium_tests/chromium_fyi.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698