Closed Bug 1275308 Opened 4 years ago Closed 4 years ago

Rename marionette.py's run-marionette action to run-tests

Categories

(Release Engineering :: Applications: MozharnessCore, defect)

defect
Not set
normal

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.
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 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+
https://hg.mozilla.org/mozilla-central/rev/3449c8ff31c9
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.