1122541 - Test failure " controller.waitForPageLoad(URI=http://ssl-ov.mozqa.com/, readyState=complete) in /testSecurity/testSSLStatusAfterRestart.js

Status: RESOLVED INVALID

(Mozilla QA Graveyard :: Mozmill Tests, defect)

Description

[Mihaela Velimiroviciu (:mihaelav)]

Module:    testDisplayCertificateStatusAfterRestart
Test:      /testSecurity/testSSLStatusAfterRestart.js
Failure:   * controller.waitForPageLoad(URI=http://ssl-ov.mozqa.com/, readyState=complete)
           * controller.waitForPageLoad(URI=https://ssl-ev.mozqa.com/, readyState=complete)
Branches:  mozilla-release (32)
Platforms: All
Report:    http://mozmill-daily.blargon7.com/#/remote/failure?app=Firefox&branch=All&platform=All&from=2014-10-01&test=%2FtestSecurity%2FtestSSLStatusAfterRestart.js&func=testDisplayCertificateStatusAfterRestart

Comment 1

[Daniela Domnici]

Failed once, over the weekend with Aurora 37.0a2 with Ubuntu 14.10.

Reports: http://mozmill-daily.blargon7.com/#/remote/failure?app=Firefox&branch=All&platform=All&from=2015-01-23&to=&test=%2FtestSecurity%2FtestSSLStatusAfterRestart.js&func=testDisplayCertificateStatusAfterRestart

Comment 2

[Daniela Domnici]

Failed again, with the latest beta build.
Reports: 
http://mozmill-release.blargon7.com/#/remote/failure?app=Firefox&branch=All&platform=All&from=2015-01-23&to=&test=%2FtestSecurity%2FtestSSLStatusAfterRestart.js&func=testDisplayCertificateStatusAfterRestart

Comment 3

[Cosmin Malutan, [:cosmin-malutan]]

Failed for 23 times with latest beta spin:
http://mozmill-release.blargon7.com/#/remote/failure?app=Firefox&branch=All&platform=All&from=2015-02-03&to=2015-02-03&test=%2FtestSecurity%2FtestSSLStatusAfterRestart.js&func=testDisplayCertificateStatusAfterRestart

Comment 4

[Daniela Domnici]

Attachment: patch_V1

Used the waitThenClick() function instead of the click() function, because it seems that we weren`t waiting enough to click on that button.
Tested it on ubuntu 14.04 production machine.

Reports: http://mozmill-crowd.blargon7.com/#/remote/reports?app=All&branch=36&platform=All&from=2015-02-05&to=2015-02-05

Comment 5

[Andreea Matei [:AndreeaMatei]]

Comment on attachment 8559833 [details] [diff] [review]
patch_V1

Review of attachment 8559833 [details] [diff] [review]:
-----------------------------------------------------------------

This looks good, but we want it first in nightly. Please add a patch for that - and also unskip since the other failure seems to be fixed too.

Comment 6

[Daniela Domnici]

Attachment: patch_V1_default

created the fix patch for the nightly branch

Comment 7

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Comment on attachment 8559901 [details] [diff] [review]
patch_V1_default

Review of attachment 8559901 [details] [diff] [review]:
-----------------------------------------------------------------

::: firefox/tests/remote/testSecurity/testSSLStatusAfterRestart.js
@@ +118,5 @@
>  
>    // Check the retrieved SSL certificate
>    var moreInfoButton = locationBar.identityPopup.getElement({type: "moreInfoButton"});
>    locationBar.waitForNotificationPanel(() => {
> +    moreInfoButton.waitThenClick();

Do we have a problem with opening the identity popup? Oh wait. I think we have an animation here! That should be the problem and needs a fix on the proper location, but not here.

Comment 8

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Mozmill tests have been superseded by Marionette tests.