Closed
Bug 615144
Opened 14 years ago
Closed 14 years ago
mozmill currently only runs a single test file or directory; should support multiples
Categories
(Testing Graveyard :: Mozmill, defect)
Testing Graveyard
Mozmill
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: k0scist, Assigned: k0scist)
References
Details
(Whiteboard: [mozmill-1.5.2+])
Attachments
(2 files)
(deleted),
patch
|
harth
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
harth
:
review+
|
Details | Diff | Splinter Review |
Currently mozmill only has the ability to deal with a single test file or directory. This should be extended to support multiple test files and directories.
https://github.com/mozautomation/mozmill/blob/master/mozmill/mozmill/__init__.py#L200
Tackling this may be different for the 1.5 branch vs the master
Comment 1•14 years ago
|
||
Hm, I thought that I have already filed this bug. But doesn't seem so. Talked with Clint during our Automation work week and we both agreed on the usefulness of this feature. Something which I would also need for my crowd extension.
Assignee | ||
Updated•14 years ago
|
Assignee: nobody → jhammel
Assignee | ||
Comment 2•14 years ago
|
||
Probably should interpolate the directory stuff in python. There's a bunch of directory weirdness for the restart tests too. If this looks good, a parallel patch could be done on master
Tested, seems to work
Attachment #493846 -
Flags: review?(fayearthur+bugs)
Comment 3•14 years ago
|
||
Comment on attachment 493846 [details] [diff] [review]
patch for multiple tests vs 1.5.2
looks good, works great.
Attachment #493846 -
Flags: review?(fayearthur+bugs) → review+
Assignee | ||
Comment 4•14 years ago
|
||
flagging for possible 1.5.2 inclusion. It would be nice to get this into the tree soon as manifests depend on it
Whiteboard: [mozmill-1.5.2?]
Assignee | ||
Comment 5•14 years ago
|
||
landed https://github.com/mozautomation/mozmill/commit/4bc0962d8acaaead0222661ea630d5a7d6f1e475
There should be a patch for master too, TBD
Assignee | ||
Comment 6•14 years ago
|
||
Attachment #494750 -
Flags: review?(fayearthur+bugs)
Comment 7•14 years ago
|
||
Comment on attachment 494750 [details] [diff] [review]
patch for multiple tests vs master
A+
Attachment #494750 -
Flags: review?(fayearthur+bugs) → review+
Assignee | ||
Comment 8•14 years ago
|
||
Assignee | ||
Updated•14 years ago
|
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Comment 9•14 years ago
|
||
Works fantastic with a 1.5.2pre build of mozmill for non-restart and restart-tests. Marking as verified fixed.
Status: RESOLVED → VERIFIED
Updated•8 years ago
|
Product: Testing → Testing Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•