Closed
Bug 668063
Opened 13 years ago
Closed 12 years ago
Allow testruns to specify a list of tags
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, 2 obsolete files)
3.32 KB,
patch
|
whimboo
:
review+
|
Details | Diff | Splinter Review |
For endurance tests we are going to need to filter results run from a specific machine so as to limit the variables when running tests with multiple Firefox builds. I suggest adding a --tags command line option to specify one of more tags. We should then be able to use these to filter results in the dashboard.
A Pivotal Tracker story has been created for this Bug: https://www.pivotaltracker.com/story/show/15491557
Assignee | ||
Comment 2•12 years ago
|
||
Comment 3•12 years ago
|
||
Comment on attachment 589890 [details] [diff] [review] Allow testruns to specify a list of tags. v1.0 Looks good. One thing which we better should do is to not use a comma separated list for the command line. Add-ons are getting specified with separate options so we should probably do the same here for tags. Otherwise we have an inconsistency. Sorry that I missed that yesterday in the call.
Attachment #589890 -
Flags: review?(hskupin) → review-
Assignee | ||
Comment 4•12 years ago
|
||
Changed to match behavior of addons. Now each time the --tag command line argument is used, the tag is appended to a list, which is attached to the report.
Attachment #589890 -
Attachment is obsolete: true
Attachment #590682 -
Flags: review?(hskupin)
Comment 5•12 years ago
|
||
Comment on attachment 590682 [details] [diff] [review] Allow testruns to specify a list of tags. v1.1 >+ ("--tag",): dict(dest="tags", >+ action="append", >+ default=[ ], This has to default to None. Same as for addons in the mozrunner.__init.py__ file. >+ report['tags'] = self.options.tags We should only add the property if tags have been specified.
Attachment #590682 -
Flags: review?(hskupin) → review-
Assignee | ||
Comment 6•12 years ago
|
||
As discussed on IRC changing default to None is not needed. Added if statement to only add tags section to report if there's at least one tag.
Attachment #590682 -
Attachment is obsolete: true
Attachment #590691 -
Flags: review?(hskupin)
Comment 7•12 years ago
|
||
Comment on attachment 590691 [details] [diff] [review] Allow testruns to specify a list of tags. v1.2 >+ ("--tag",): dict(dest="tags", >+ action="append", >+ default=[ ], >+ metavar="TAG", >+ help="Tag to apply to the report") Can you please fix the indentation before the check-in? Otherwise looks good. r=me.
Attachment #590691 -
Flags: review?(hskupin) → review+
Assignee | ||
Comment 8•12 years ago
|
||
Landed as: http://hg.mozilla.org/qa/mozmill-automation/rev/2b525372b8ca
Comment 9•12 years ago
|
||
So this is fixed now. Lets close it out. Thanks Dave!
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•10 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
•