Closed
Bug 1275308
Opened 8 years ago
Closed 8 years ago
Rename marionette.py's run-marionette action to run-tests
Categories
(Release Engineering :: Applications: MozharnessCore, defect)
Release Engineering
Applications: MozharnessCore
Tracking
(firefox49 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox49 | --- | fixed |
People
(Reporter: ahal, Assigned: ahal)
References
Details
Attachments
(1 file)
Armen pointed out that marionette doesn't have a "run-tests" actoin (it's instead called "run-marionette"). This means that some of the tools from bug 1250904 won't work when interactively debugging tests with taskcluster (they assume there is a "run-tests" action). We should just rename run-marionette to run-tests to make this work and to be consistent with every other mozharness script.
Assignee | ||
Comment 1•8 years ago
|
||
In bug 1262260 and 1250904, we are building some tools to make debugging tests in interactive jobs easier. Part of this includes making it easier to run mozharness scripts with various configurations. Some of these tools depend on a "run-tests" action being present, but in marionette.py, this action is called "run-marionette" instead. Rather than make these tools special case marionette, let's just make marionette.py consistent with every other mozharness script. Review commit: https://reviewboard.mozilla.org/r/54884/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/54884/
Attachment #8755916 -
Flags: review?(armenzg)
Comment 2•8 years ago
|
||
Comment on attachment 8755916 [details] MozReview Request: Bug 1275308 - [mozharness] Rename the run-marionette action to run-tests, r?armenzg https://reviewboard.mozilla.org/r/54884/#review51530 Excellent!
Attachment #8755916 -
Flags: review?(armenzg) → review+
Comment 4•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/3449c8ff31c9
You need to log in
before you can comment on or make changes to this bug.
Description
•