Closed Bug 1275308 Opened 4 years ago Closed 4 years ago
.py's run-marionette action to run-tests
MozReview Request: Bug 1275308 - [mozharness] Rename the run-marionette action to run-tests, r?armenzg
58 bytes, text/x-review-board-request
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+
You need to log in before you can comment on or make changes to this bug.