Bug 1286900 - Reformat WPT command argument additions; r=ted

list.extend() is favored over +=. Also use single quotes, fixup
indentation, and factor out a common variable.

MozReview-Commit-ID: 3qVDGrkYhVe

--HG--
extra : rebase_source : 0c5faa30f5be09165f61943300ecfb2e9c7080f4
This commit is contained in:
Gregory Szorc 2016-09-20 12:59:19 -07:00
parent 8c38bb4182
commit bf8b5f0282

View File

@ -131,14 +131,17 @@ class WebPlatformTest(TestingMixin, MercurialScript, BlobUploadMixin):
self.fatal("Could not create blobber upload directory")
# Exit
cmd += ["--log-raw=-",
"--log-raw=%s" % os.path.join(dirs["abs_blob_upload_dir"],
"wpt_raw.log"),
"--log-errorsummary=%s" % os.path.join(dirs["abs_blob_upload_dir"],
"wpt_errorsummary.log"),
"--binary=%s" % self.binary_path,
"--symbols-path=%s" % self.query_symbols_url(),
"--stackwalk-binary=%s" % self.query_minidump_stackwalk()]
blob_upload_dir = dirs['abs_blob_upload_dir']
cmd.extend([
'--log-raw=-',
'--log-raw=%s' % os.path.join(blob_upload_dir, 'wpt_raw.log'),
'--log-errorsummary=%s' % os.path.join(blob_upload_dir,
'wpt_errorsummary.log'),
'--binary=%s' % self.binary_path,
'--symbols-path=%s' % self.query_symbols_url(),
'--stackwalk-binary=%s' % self.query_minidump_stackwalk(),
])
for test_type in c.get("test_type", []):
cmd.append("--test-type=%s" % test_type)