Update tests and ensure that always a window is selected after closing the active window

RESOLVED WONTFIX

Status

RESOLVED WONTFIX
4 years ago
3 years ago

People

(Reporter: whimboo, Assigned: whimboo)

Tracking

41 Branch
Points:
---

Firefox Tracking Flags

(firefox41 wontfix)

Details

Seen this lately in our tests only on Windows while running them in Mozmill CI. Especially test_windows.py does not call switch_to() for the browser window when another before opened window gets closed. That means Marionette hangs around with a reference to a non-existent window and reports a failure that this.browser cannot be found.

Not sure why this only happens on Windows. David, do you have an idea?

I think we should not do magic calls to switch_to() in our BaseWindow.close() method, but request the test to explicitly call this method.
Flags: needinfo?(dburns)
Most likely we don't need that cause it smells like a regression from bug 1169798 too. I will stop investigating possible fixes until the patch on the mentioned bug has been landed and tested.
Status: ASSIGNED → NEW
Depends on: 1169798
Flags: needinfo?(dburns)
This was clearly the regression in Marionette which is causing this failure. The code we currently have does not require a change to work. Once the patch on bug 1169798 landed on all affected branches, our tests will pass again. This is a WONTFIX.
Status: NEW → RESOLVED
Last Resolved: 4 years ago
status-firefox41: affected → wontfix
Resolution: --- → WONTFIX
Product: Mozilla QA → Testing
You need to log in before you can comment on or make changes to this bug.