Bug 1567264 - [mochitest] Add a convenience --enable-fission argument r=gbrown

This argument is an alias for '--setpref="fission.autostart=1"'.

Differential Revision: https://phabricator.services.mozilla.com/D39548

--HG--
extra : moz-landing-system : lando
This commit is contained in:
Andrew Halberstadt 2019-07-29 17:49:33 +00:00
parent e3e6ea6d8e
commit 429fb9a444
2 changed files with 15 additions and 0 deletions

View File

@ -406,6 +406,11 @@ class MochitestArguments(ArgumentContainer):
"dest": "e10s",
"help": "Run tests with electrolysis preferences and test filtering disabled.",
}],
[["--enable-fission"],
{"action": "store_true",
"default": False,
"help": "Run tests with fission (site isolation) enabled.",
}],
[["--store-chrome-manifest"],
{"action": "store",
"help": "Destination path to write a copy of any chrome manifest "
@ -824,6 +829,9 @@ class MochitestArguments(ArgumentContainer):
parser.error("mochitest-{} does not support e10s, try again with "
"--disable-e10s.".format(options.flavor))
if options.enable_fission:
options.extraPrefs.append("fission.autostart=1")
options.leakThresholds = {
"default": options.defaultLeakThreshold,
"tab": options.defaultLeakThreshold,

View File

@ -2614,6 +2614,13 @@ toolbar#nav-bar {
tests = self.getActiveTests(options)
self.logPreamble(tests)
if mozinfo.info['fission'] and not mozinfo.info['e10s']:
# Make sure this is logged *after* suite_start so it gets associated with the
# current suite in the summary formatters.
self.log.error("Fission is not supported without e10s.")
return 1
tests = [t for t in tests if 'disabled' not in t]
# Until we have all green, this does not run on a11y (for perf reasons)