Closed Bug 1117731 Opened 11 years ago Closed 10 years ago

Test failure 'Notification popup state has been opened' in testSecurity/testSSLStatusAfterRestart.js

Categories

(Mozilla QA Graveyard :: Mozmill Tests, defect)

Version 2
x86
All
defect
Not set
normal

Tracking

(firefox34 unaffected, firefox35 unaffected, firefox36 fixed, firefox37 fixed, firefox38 fixed)

RESOLVED FIXED
Tracking Status
firefox34 --- unaffected
firefox35 --- unaffected
firefox36 --- fixed
firefox37 --- fixed
firefox38 --- fixed

People

(Reporter: teodruta, Assigned: daniela.domnici)

References

()

Details

(Whiteboard: [mozmill-test-failure])

Attachments

(2 files)

Module: testDisplayCertificateStatusAfterRestart Test: testSecurity/testSSLStatusAfterRestart.js Failure: Notification popup state has been closed Branches: mozilla-aurora Platforms: Linux x86 Locale: All Report: http://mozmill-daily.blargon7.com/#/remote/report/5c7e7a22f9984b46554bc3b9fc44c734 Fails intermittently on Ubuntu 14.10, 14.04 x86.
Just a small note I have mistakenly reported the failure message wrong, it should be: "Notification popup state has been opened"
Summary: Test failure 'Notification popup state has been closed' in testSecurity/testSSLStatusAfterRestart.js → Test failure 'Notification popup state has been opened' in testSecurity/testSSLStatusAfterRestart.js
This failed on Nightly, in the testDisplayCertificateStatus module, most probably it's the same issue. http://mozmill-daily.blargon7.com/#/remote/report/04e54d608fc589a28217304fe674870e
Created the skip patch for beta branch.
Attachment #8558377 - Flags: review?(mihaela.velimiroviciu)
Attachment #8558377 - Flags: review?(andreea.matei)
Attachment #8558377 - Flags: review?(mihaela.velimiroviciu)
Attachment #8558377 - Flags: review?(andreea.matei)
Attachment #8558377 - Flags: review+
Whiteboard: [mozmill-test-failure] → [mozmill-test-failure][mozmill-test-skipped]
I`ve investigated today this bug and I fixed it, tomorrow I `ll run more testruns on ubuntu 14.04 production machine.Today I`ve did ~30 testruns on ubuntu 14.04 staging machine and ~20 testruns on ubuntu 14.04 production machine. Here are the results: http://mozmill-crowd.blargon7.com/#/remote/reports?app=All&branch=36&platform=Linux&from=2015-02-03&to=2015-02-03
Created a fix patch for this test.This patch also fix the bug 1122541. I`ve did ~150 testruns on ubuntu 14.04-32-2 production machine. Reports:http://mozmill-crowd.blargon7.com/#/remote/reports?app=All&branch=All&platform=All&from=2015-02-04&to=2015-02-04
Attachment #8559115 - Flags: review?(mihaela.velimiroviciu)
Attachment #8559115 - Flags: review?(andreea.matei)
Comment on attachment 8559115 [details] [diff] [review] fix testSSLStatusAfterRestart.js Review of attachment 8559115 [details] [diff] [review]: ----------------------------------------------------------------- ::: firefox/tests/remote/testSecurity/testSSLStatusAfterRestart.js @@ +51,5 @@ > function teardownTest(aModule) { > if (persisted.nextTest) { > controller.restartApplication(persisted.nextTest); > } > + aModule.tabBrowser.closeAllTabs(); Why do we need to close tabs after each test?
Comment on attachment 8559115 [details] [diff] [review] fix testSSLStatusAfterRestart.js Review of attachment 8559115 [details] [diff] [review]: ----------------------------------------------------------------- ::: firefox/tests/remote/testSecurity/testSSLStatusAfterRestart.js @@ +51,5 @@ > function teardownTest(aModule) { > if (persisted.nextTest) { > controller.restartApplication(persisted.nextTest); > } > + aModule.tabBrowser.closeAllTabs(); This line of code is not reached @@ -55,4 @@ > } > > function teardownModule(aModule) { > - aModule.tabBrowser.closeAllTabs(); Please add this line back
Attachment #8559115 - Flags: review?(mihaela.velimiroviciu)
Attachment #8559115 - Flags: review?(andreea.matei)
Attachment #8559115 - Flags: review-
Please ignore the patch above.It seems that Cosmin`s closeTabs refactor solve this problem. Today, I`ve did 100 remote - testruns using 14.04-32-2 production machine and it didn`t fail at all. Here are the results: http://mozmill-crowd.blargon7.com/#/remote/reports?app=All&branch=36&platform=All&from=2015-02-05&to=2015-02-05.
Great. Then we can mark this as fixed for now. Hopefully won't fail again :) https://hg.mozilla.org/qa/mozmill-tests/rev/17fa1493b539 (beta)
Assignee: nobody → daniela.domnici
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [mozmill-test-failure][mozmill-test-skipped] → [mozmill-test-failure]
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: