Closed Bug 777412 Opened 9 years ago Closed 9 years ago

Support --setenv in Reftests

Categories

(Testing :: Reftest, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla17

People

(Reporter: decoder, Assigned: decoder)

Details

(Keywords: sec-want)

Attachments

(1 file)

Mochitests support the --setenv option to add additional environment variables during testing. One of my tools needs this during the automated tests but reftests don't seem to support this. I'll attach a patch that adds the same support to reftests (just the way Mochitests do).
Attached patch PatchSplinter Review
Assignee: nobody → choller
Status: NEW → ASSIGNED
Attachment #645807 - Flags: review?(ted.mielczarek)
Comment on attachment 645807 [details] [diff] [review]
Patch

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

I would normally ask that you refactor this out to somewhere common, but we're going to rework all of these test suites to use mozbase soon anyway.
Attachment #645807 - Flags: review?(ted.mielczarek) → review+
https://hg.mozilla.org/mozilla-central/rev/e0779a303e31
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in before you can comment on or make changes to this bug.