Closed
Bug 1104742
Opened 10 years ago
Closed 10 years ago
Stopping Marionette with Ctrl-C does not quit the application
Categories
(Remote Protocol :: Marionette, defect)
Remote Protocol
Marionette
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla37
People
(Reporter: whimboo, Assigned: ahal)
References
Details
(Keywords: pi-marionette-runner)
Attachments
(1 file, 1 obsolete file)
When Marionette is running tests in the target application, and you issue ctrl-c in the terminal, Marionette quits but does not close the application it has started.
Marionette should be made failure-proof so that the application under test is always closed whatever happens to Marionette. Otherwise instances of the application will continue to exist, and can badly interact with following tests.
Comment 1•10 years ago
|
||
is this in general or e10s specific?
Reporter | ||
Comment 2•10 years ago
|
||
I don't have underlined tab names when running the tests from the firefox-greenlight repository. So the browser is not running in e10s mode, and this applies in general.
Updated•10 years ago
|
Keywords: ateam-marionette-runner
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → ahalberstadt
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•10 years ago
|
||
Attachment #8532095 -
Flags: review?(dburns)
Assignee | ||
Comment 4•10 years ago
|
||
/r/1147 - Bug 1104742 - Marionette runner should cleanup marionette objects when gc'ed, r=AutomatedTester
Pull down this commit:
hg pull review -r 8a869167794d3bb439d167f4d29ceb8c820a5ebf
Updated•10 years ago
|
Attachment #8532095 -
Flags: review?(dburns) → review+
Comment 5•10 years ago
|
||
Assignee | ||
Comment 6•10 years ago
|
||
Comment 7•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
Assignee | ||
Comment 8•9 years ago
|
||
Attachment #8532095 -
Attachment is obsolete: true
Attachment #8618720 -
Flags: review+
Assignee | ||
Comment 9•9 years ago
|
||
Updated•2 years ago
|
Product: Testing → Remote Protocol
You need to log in
before you can comment on or make changes to this bug.
Description
•