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

RESOLVED INVALID

Status

RESOLVED INVALID
4 years ago
a year ago

People

(Reporter: mihaelav, Unassigned)

Tracking

unspecified

Firefox Tracking Flags

(firefox36 affected, firefox37 affected, firefox38 affected, firefox-esr31 affected)

Details

(Whiteboard: [mozmill-test-failure], URL)

Attachments

(2 attachments)

(Reporter)

Description

4 years ago
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 4

4 years ago
Created attachment 8559833 [details] [diff] [review]
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
Attachment #8559833 - Flags: review?(mihaela.velimiroviciu)
Attachment #8559833 - Flags: review?(andreea.matei)
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.
Attachment #8559833 - Flags: review?(mihaela.velimiroviciu)
Attachment #8559833 - Flags: review?(andreea.matei)
Attachment #8559833 - Flags: review+

Comment 6

4 years ago
Created attachment 8559901 [details] [diff] [review]
patch_V1_default

created the fix patch for the nightly branch
Attachment #8559901 - Flags: review?(mihaela.velimiroviciu)
Attachment #8559901 - Flags: review?(andreea.matei)
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.
Attachment #8559901 - Flags: review?(mihaela.velimiroviciu)
Attachment #8559901 - Flags: review?(andreea.matei)
Attachment #8559901 - Flags: review-
Mozmill tests have been superseded by Marionette tests.
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.