Closed Bug 999407 Opened 10 years ago Closed 10 years ago

add --setenv option for mochitests run through mach

Categories

(Testing :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla31

People

(Reporter: froydnj, Assigned: froydnj)

Details

Attachments

(1 file)

runtests.py supports this, and it's handy for setting up NSPR logging on only
the main browser process but not the xpcshell HTTP server process.
We stuff the option's value into |environment| to keep it consistent with
MochitestOptions.
Attachment #8410241 - Flags: review?(jmaher)
Comment on attachment 8410241 [details] [diff] [review]
add --setenv option for mochitests run through mach

Review of attachment 8410241 [details] [diff] [review]:
-----------------------------------------------------------------

at first I thought- don't we have this, then I realized it was for mach.  It is too bad mach cannot use mochitest_options.py.
Attachment #8410241 - Flags: review?(jmaher) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/4d232ce4ffb5
Assignee: nobody → nfroyd
Flags: in-testsuite-
(In reply to Joel Maher (:jmaher) from comment #2)
> at first I thought- don't we have this, then I realized it was for mach.  It
> is too bad mach cannot use mochitest_options.py.

Well, mach_commands.py clearly can, because it instantiates MochitestOptions, and then fills in various bits and pieces for the moral equivalent of runtests.py.  So really, we should be able to use MochitestOptions to construct the command decorators that mach needs.  I will file a bug.
https://hg.mozilla.org/mozilla-central/rev/4d232ce4ffb5
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: