Closed Bug 867831 Opened 11 years ago Closed 7 years ago

Need multi-process unit tests for Marionette

Categories

(Remote Protocol :: Marionette, defect)

Product:
Remote Protocol
Component:
Marionette
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX

People

(Reporter: jgriffin, Unassigned)

Details

We're adding a lot of code to handle multi-process use cases, but we don't have any multi-process unit tests in Marionette.

Mochitest does this, so I assume we could do something similar.
[Mass closure] Closing as these features are no longer needed or wanted
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INCOMPLETE
I would wait with closing this. Given that this feature hasn't been fully rolled out, I would like to request feedback from Gabor first. If we don't need those tests we can clearly close this bug.
Status: RESOLVED → REOPENED
Flags: needinfo?(gkrizsanits)
Resolution: INCOMPLETE → ---
What kind of tests are we missing exactly? If marionette tests open multiple tabs/windows it will use multiple content processes already, isn't that enough? Are there any specific scenarios we want to test?
Flags: needinfo?(gkrizsanits)
Ah ok. So if you cannot think of any testcase which might require a restart of the application, we could close this bug. I assume lots of specific tests have already been implemented via other test suites.
Flags: needinfo?(gkrizsanits)
Yes, I think it's safe to close it.
Flags: needinfo?(gkrizsanits)
Thanks.
Status: REOPENED → RESOLVED
Closed: 7 years ago → 7 years ago
Resolution: --- → WONTFIX
Product: Testing → Remote Protocol
You need to log in before you can comment on or make changes to this bug.