Closed Bug 837278 Opened 8 years ago Closed 6 years ago

Support selecting tests to run in in-tree mach and Makefile test commands

Categories

(Add-on SDK Graveyard :: General, defect, P3)

x86_64
Windows 8
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mossop, Unassigned)

References

Details

(Whiteboard: [mach])

Attachments

(1 file)

Bug 793928 added a way to run the jetpack tests from mach and make but it just runs all the tests. It would be good to add a way to select an individual test
Whiteboard: [mach]
So, this seems to let me filter tests when using | mach jetpack-test |. (Or at least, | mach jetpack-test -f context | only runs the contextMenu tests, and | mach jetpack-test -f context:testUnload | only runs the contextMenu tests that include "testUnload" in the test name.)

It isn't working for me with make, though.
Attachment #734317 - Flags: feedback?(dtownsend+bugmail)
Comment on attachment 734317 [details] [diff] [review]
Add ability to filter tests to mach jetpack-test

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

I don't especially like the regexp filter method of selecting tests and would really like us to just pass the path to a test file to run, but I think that needs cfx changes and pain so I guess this is good for the interim.

gps should review this really but it looks fine for me. Mach just calls make so not sure why make isn't working. How are you running it, with something like |EXTRA_TEST_ARGS="-f context" make jetpack-tests| ?
Attachment #734317 - Flags: feedback?(dtownsend+bugmail) → feedback+
Fixed by bug 1035512
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.