Closed
Bug 1032530
Opened 10 years ago
Closed 10 years ago
mach mochitest commands should take --setpref argument like runtests.py does
Categories
(Testing :: Mochitest, defect)
Testing
Mochitest
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla33
People
(Reporter: dbaron, Assigned: agi)
Details
Attachments
(1 file)
3.22 KB,
patch
|
gps
:
review+
|
Details | Diff | Splinter Review |
The mach mochitest commands should have a --setpref argument like mochitest runtests.py does (or like mach reftest comands do). (I'm not planning to patch this since I don't follow all the python magic right now; it's easier to just modify greprefs.js.)
Updated•10 years ago
|
Component: Build Config → Mochitest
Product: Core → Testing
Assignee | ||
Comment 1•10 years ago
|
||
I think this should do. dbaron do you know who I can ask for review for this patch? I'm new in this component :) Thanks!
Flags: needinfo?(dbaron)
Reporter | ||
Comment 2•10 years ago
|
||
Based on https://hg.mozilla.org/mozilla-central/filelog/161fadfcbadc/testing/mochitest/mach_commands.py I'd suggest :gps.
Flags: needinfo?(dbaron)
Assignee | ||
Updated•10 years ago
|
Attachment #8455031 -
Flags: review?(gps)
Comment 3•10 years ago
|
||
Comment on attachment 8455031 [details] [diff] [review] mach mochitest takes --setpref argument Review of attachment 8455031 [details] [diff] [review]: ----------------------------------------------------------------- LGTM. I assume you tested this.
Attachment #8455031 -
Flags: review?(gps) → review+
Reporter | ||
Comment 4•10 years ago
|
||
If you have in fact tested this, you should add the checkin-needed keyword.
Flags: needinfo?(agi.novanta)
Assignee | ||
Comment 5•10 years ago
|
||
Yup. I was planning on doing a little bit more of testing tonight before asking to check-in the code. Sorry that was probably unclear.
Flags: needinfo?(agi.novanta)
Comment 7•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/816b270f3a2e
Assignee: nobody → agi.novanta
Keywords: checkin-needed
Comment 8•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/816b270f3a2e
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
You need to log in
before you can comment on or make changes to this bug.
Description
•