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

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py

Issue 2308283002: Allow seeding the random layout test order and write out seed. (Closed)
Patch Set: Created 4 years, 3 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
Index: third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py b/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py
index 7dda2713a136d962d26d30dbdd9d047e1dc6927d..40a29587cf40290f5eddbf56dc8c85a789876f08 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/run_webkit_tests.py
@@ -333,12 +333,12 @@ def parse_args(args):
"--order",
action="store",
default="natural",
- help=("determine the order in which the test cases will be run. "
+ 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-seeded' == randomize the test order using a fixed seed, "
- "'random' == randomize the test order.")),
+ "'random' == pseudorandom order. Seed can be given with "
+ "--seed, otherwise a new seed will be generated.")),
optparse.make_option(
"--profile",
action="store_true",
@@ -382,6 +382,11 @@ def parse_args(args):
default=False,
help="DEPRECATED, same as --batch-size=1 --verbose"),
optparse.make_option(
+ "--seed",
+ type="int",
+ default=None,
+ help="Seed to use for random test order. Only applicable in combination with --order=random."),
mithro 2016/09/06 02:02:25 For the help you should add what the default value
qyearsley 2016/09/06 17:22:48 Agreed - will add this.
qyearsley 2016/09/07 18:07:12 Now done
+ optparse.make_option(
"--skipped",
action="store",
default=None,

Powered by Google App Engine
This is Rietveld 408576698