Closed Bug 573461 Opened 10 years ago Closed 8 years ago
Test failures due to non-existing window after test
Close Window .js
MODULE: firefox\testPrivateBrowsing\testDisabledElements.js TEST: testDisabledElements FAIL: timeout exceeded for waitForEval('subject.privateBrowsing.enabled == subject.state') BRANCH: 1.9.1, 1.9.2, default (trunk)
That's only happening on OS X or on all platforms?
According to anthony's spreadsheet this is mac only
(In reply to comment #1) > That's only happening on OS X or on all platforms? Locally, I've only seen this error on Mac. On Brasstacks, I can't find a recent non-Mac instance of this failure.
(In reply to comment #3) > (In reply to comment #1) > > That's only happening on OS X or on all platforms? > > Locally, I've only seen this error on Mac. On Brasstacks, I can't find a > recent non-Mac instance of this failure. It's because of the test that runs prior to this testCloseWindow which is mac only in the way it either a) windows or b) the way private browsing starts or stops. Looking into it.
Not sure if you want a patch for a separate test in this bug, but this patch addresses a bug that is the cause of the this reported bug's failure. - Change to the handleWindow call making sure that we don't close the window after the return from the callback handler by passing true.
Comment on attachment 452818 [details] [diff] [review] v1.1 This resolves testDisableElements.js without regressing testCloseWindow.js. r+
Comment on attachment 452818 [details] [diff] [review] v1.1 What a silly mistake. Thanks for spotting this Aaron!
Attachment #452818 - Flags: superreview?(hskupin) → superreview+
Landed on all branches: http://hg.mozilla.org/qa/mozmill-tests/rev/a68b9d3fc719 http://hg.mozilla.org/qa/mozmill-tests/rev/4636315319b8 http://hg.mozilla.org/qa/mozmill-tests/rev/4e76267680e2 This only a bustage fix for the moment. For a real fix we would also have to make sure that a browser window is open when the test fails while staying in private browsing mode. Right now all other tests will fail. This should be part of the teardownModule function.
Summary: [mozmill] Timeout failure in testDisabledElements.js → [mozmill] Test failures due to non-existing window after testCloseWindow.js
Mass move of Mozmill Test related project bugs to newly created components. You can filter out those emails by using "Mozmill-Tests-to-MozillaQA" as criteria.
Component: Private Browsing → Mozmill Tests
Product: Firefox → Mozilla QA
QA Contact: private.browsing → mozmill-tests
Aaron, can you finish this bug up in the near future?
Removing assignee so this can be reassigned.
Assignee: aaron.train → nobody
Status: ASSIGNED → NEW
Summary: [mozmill] Test failures due to non-existing window after testCloseWindow.js → Test failures due to non-existing window after testCloseWindow.js
I don't see it failing any more. I think we can call this a WONTFIX.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
Please see comment 9. It states why it doesn't fail and what has to be done to completely fix this test.
Status: RESOLVED → REOPENED
Resolution: WONTFIX → ---
Can we get some traction on this bug? It's happening in Beta: http://mozmill-ondemand.blargon7.com/#/functional/report/c2b72632f20450b6d99d14c709b017e5
I will have a look at.
Assignee: nobody → hskupin
Status: REOPENED → ASSIGNED
So this one doesn't happen anymore. What you are seeing is a 'Timeout waiting for page loaded.' which is interesting but should be covered by another bug.
Status: ASSIGNED → RESOLVED
Closed: 8 years ago → 8 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.