725486 - Failure in testSecurity/testSubmitUnencryptedInfoWarning.js | The value in the search field should equal 'mozilla'

Status: RESOLVED FIXED

(Mozilla QA Graveyard :: Mozmill Tests, defect)

Description

[u279076]

testSecurity/testSubmitUnencryptedInfoWarning.js is failing with the following message:

The value in the search field is the expected search term - '' should equal 'mozilla' 

This failure appears across all branches and platforms, and appears to have just started today (Feb 8, 2012).

Comment 1

[Alex Lakatos[:AlexLakatos]]

Attachment: disable v1 (all) [landed]

The test can't type in the search box. It can type in other boxes, but not in the search box. Investigating further and skipping the test for now.

Comment 2

[u279076]

Comment on attachment 595789 [details] [diff] [review]
disable v1 (all) [landed]

Landed:
http://hg.mozilla.org/qa/mozmill-tests/rev/a277f3190db6 (default)
http://hg.mozilla.org/qa/mozmill-tests/rev/be1372e06884 (mozilla-aurora)
http://hg.mozilla.org/qa/mozmill-tests/rev/3988868680b0 (mozilla-beta)
http://hg.mozilla.org/qa/mozmill-tests/rev/5519e6aad0d5 (mozilla-release)
http://hg.mozilla.org/qa/mozmill-tests/rev/41a34631ffe0 (mozilla-esr10)
http://hg.mozilla.org/qa/mozmill-tests/rev/c8fd2e85db00 (mozilla-1.9.2)

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+1] (away from Feb 12th - Feb 22nd)]

Alex, any update on this bug?

Comment 4

[Alex Lakatos[:AlexLakatos]]

(In reply to Henrik Skupin (:whimboo) from comment #3)
> Alex, any update on this bug?
The Mozilla main page changed in the meantime. The test needs to be updated, so I'm doing that.

Comment 5

[Alex Lakatos[:AlexLakatos]]

Attachment: patch v1.0

Opening the tabzilla panel. There is no more Go button, so using Enter

Comment 6

[Henrik Skupin [:whimboo][⌚️UTC+1] (away from Feb 12th - Feb 22nd)]

Comment on attachment 630587 [details] [diff] [review]
patch v1.0

We now have SSL certs on mozqa.com and can make use of it. Please find a page or create a new one in the litmus-data repository which we can make use of.

Comment 7

[Henrik Skupin [:whimboo][⌚️UTC+1] (away from Feb 12th - Feb 22nd)]

Do we have to use the OV cert here? I would say the DV one will also be enough.

Comment 8

[Alex Lakatos[:AlexLakatos]]

Attachment: patch v2.0

(In reply to Henrik Skupin (:whimboo) from comment #7)
> Do we have to use the OV cert here? I would say the DV one will also be
> enough.
The type of certificate does not matter, as long as I can access the website using https.

Comment 9

[Henrik Skupin [:whimboo][⌚️UTC+1] (away from Feb 12th - Feb 22nd)]

Comment on attachment 649296 [details] [diff] [review]
patch v2.0

> const TIMEOUT_MODAL_DIALOG = 30000;
>-const TEST_SITE = "https://www.mozilla.org/";
>+const TEST_SITE = "https://ssl-dv.mozqa.com/"

Please separate those two blocks by an empty line.

>+  var submitButton = new elementslib.ID(controller.tabs.activeTab,
>+                                        "submit");
>+  controller.waitForElement(submitButton, gTimeout);

No gTimeout needed here. Lets get rid of this variable. 

>+  expect.equal(searchTerm.getNode().textContent, "mozilla",
>+               "The value bolded is the expected search term");

There is no need to mention the styling of the search term. It's not relevant here. It just should say: "Search term correctly submitted".

Comment 10

[Alex Lakatos[:AlexLakatos]]

Attachment: patch v3.0

addressed requested reviews

Comment 11

[Henrik Skupin [:whimboo][⌚️UTC+1] (away from Feb 12th - Feb 22nd)]

Pushed to default:
http://hg.mozilla.org/qa/mozmill-tests/rev/d687a328f751

If everything works fine we should backport the patch to older branches in the next days.

Comment 12

[Henrik Skupin [:whimboo][⌚️UTC+1] (away from Feb 12th - Feb 22nd)]

Pushed to older branches:
http://hg.mozilla.org/qa/mozmill-tests/rev/9a3e54819764 (aurora)
http://hg.mozilla.org/qa/mozmill-tests/rev/3fde9d85516b (beta)
http://hg.mozilla.org/qa/mozmill-tests/rev/069a7d7125f3 (release)
http://hg.mozilla.org/qa/mozmill-tests/rev/df3c7bf14728 (esr10)

Alex, please re-enable the Mozmill coverage on Litmus for all branches.

Comment 13

[Alex Lakatos[:AlexLakatos]]

https://litmus.mozilla.org/show_test.cgi?id=16752 - aurora
https://litmus.mozilla.org/show_test.cgi?id=41850 - release
https://litmus.mozilla.org/show_test.cgi?id=65099 - esr
There is no Beta or Default branch in litmus.