Closed
Bug 1065776
Opened 10 years ago
Closed 10 years ago
Running a single mochitest keeps test browser open after completion.
Categories
(Testing :: Mochitest, defect)
Testing
Mochitest
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 1052211
People
(Reporter: markh, Unassigned)
References
Details
Bug 1014062 is a little confusing, but I'm convinced that in the past, running "mach mochitest-browser path/to/single_test_file" would run the test and automatically terminate the process. Now it acts as if --keep-open was specified, and there seems to be no way to have the process terminate automatically.
CC bz, ted and gijs as they were involved in that discussion in bug 1014062. CC unfocused as he was also wondering why --keep-open seemed to recently become the default.
I can't see a reason for the new behaviour given --keep-open exists and seems to work. Bug 990289 (referenced by bug 1014062) implies that explicitly specifying --keep-open is fine for the people who want this new behaviour.
Can anyone offer insights into why things are now done this way, and a way forward so this behaviour isn't forced on those of us who don't want it?
Comment 1•10 years ago
|
||
> explicitly specifying --keep-open is fine for the people who want this new behaviour
If by "fine" you mean "a huge PITA".... The normal usage for me of running a single mochitest is when debugging/editing it, and having the browser quit is never what I want in that situation, personally.
I agree that there should be a way to get the behavior you want _and_ the behavior I want by default (via a setting in some config file) without having to sprinkle command-line flags all over, though.
Comment 2•10 years ago
|
||
I suspect the default before bug 1014062 was keep-open for mochitest-plain/mochitest-chrome and !keep-open for mochitest-browser. I have no idea why, and I've not verified that, but that would tally with everyone's memories, it seems.
Comment 3•10 years ago
|
||
For mochitest-chrome and mochitest-browser there is not much point to the --keep-open behavior by default, since you have to restart the browser to rerun a modified version of the test.
Comment 4•10 years ago
|
||
Fwiw, I filed a similar bug 1052211. There it was decided to add a --autoclose command-line flag.
Reporter | ||
Comment 5•10 years ago
|
||
(In reply to Martijn Wargers [:mwargers] (QA) from comment #4)
> Fwiw, I filed a similar bug 1052211. There it was decided to add a
> --autoclose command-line flag.
Good enough for me, thanks!
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•