Closed
Bug 678183
Opened 13 years ago
Closed 13 years ago
Add command line argument to allow running a reserved endurance test
Categories
(Mozilla QA Graveyard :: Mozmill Automation, defect)
Mozilla QA Graveyard
Mozmill Automation
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: davehunt, Assigned: davehunt)
References
Details
Attachments
(1 file)
|
5.12 KB,
patch
|
u279076
:
review+
whimboo
:
review+
|
Details | Diff | Splinter Review |
The Mem Buster endurance test is a reserved test, only run if specifically needed. We need to add a command line argument to allow running such tests.
|
Assignee | |
Comment 1•13 years ago
|
||
Assignee: nobody → dave.hunt
Attachment #552405 -
Flags: review?(anthony.s.hughes)
Attachment #552405 -
Flags: review?(anthony.s.hughes) → review+
|
|
Assignee | |
Updated•13 years ago
|
Attachment #552405 -
Flags: review?(hskupin)
|
||
Comment 2•13 years ago
|
||
Comment on attachment 552405 [details] [diff] [review]
Add command line argument to allow running a reserved endurance test. 1.0
>- self.test_path = os.path.join('tests','endurance')
>+ if self.reserved:
>+ self.test_path = os.path.join('tests','endurance','reserved',self.reserved)
>+ else:
>+ self.test_path = os.path.join('tests','endurance')
Can you keep the former line and only call join again if self.reserved has been specified? That lowers the amount of constants we have to update once something changes.
r=me with those changes.
Attachment #552405 -
Flags: review?(hskupin) → review+
|
|
Assignee | |
Comment 3•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
|
||
Updated•11 years ago
|
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•