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

RESOLVED FIXED

Status

Mozilla QA
Mozmill Tests
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: ashughes, Assigned: AlexLakatos)

Tracking

unspecified
Bug Flags:
in-litmus +

Firefox Tracking Flags

(firefox14 fixed, firefox15 fixed, firefox16 fixed, firefox17 fixed, firefox-esr10 fixed)

Details

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

Attachments

(2 attachments, 2 obsolete attachments)

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).
(Assignee)

Updated

5 years ago
Assignee: nobody → alex.lakatos
Status: NEW → ASSIGNED
(Assignee)

Comment 1

5 years ago
Created attachment 595789 [details] [diff] [review]
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.
Attachment #595789 - Flags: review?(anthony.s.hughes)
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)
Attachment #595789 - Attachment description: skipPatch v1.0 → disable v1 (all) [landed]
Attachment #595789 - Flags: review?(anthony.s.hughes) → review+
Whiteboard: [mozmill-test-failure] → [mozmill-test-failure][mozmill-test-skipped]
Alex, any update on this bug?
(Assignee)

Comment 4

5 years ago
(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.
(Assignee)

Comment 5

5 years ago
Created attachment 630587 [details] [diff] [review]
patch v1.0

Opening the tabzilla panel. There is no more Go button, so using Enter
Attachment #630587 - Flags: review?(hskupin)
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.
Attachment #630587 - Flags: review?(hskupin) → review-
(Assignee)

Updated

5 years ago
Depends on: 762964
(Assignee)

Updated

5 years ago
Depends on: 768358
Do we have to use the OV cert here? I would say the DV one will also be enough.
(Assignee)

Comment 8

5 years ago
Created attachment 649296 [details] [diff] [review]
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.
Attachment #630587 - Attachment is obsolete: true
Attachment #649296 - Flags: review?(hskupin)
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".
Attachment #649296 - Flags: review?(hskupin) → review-
No longer depends on: 768358
(Assignee)

Comment 10

5 years ago
Created attachment 649690 [details] [diff] [review]
patch v3.0

addressed requested reviews
Attachment #649296 - Attachment is obsolete: true
Attachment #649690 - Flags: review?(hskupin)
Attachment #649690 - Flags: review?(hskupin) → review+
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.
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
status-firefox-esr10: --- → affected
status-firefox14: --- → affected
status-firefox15: --- → affected
status-firefox16: --- → affected
status-firefox17: --- → fixed
Resolution: --- → FIXED
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.
status-firefox-esr10: affected → fixed
status-firefox14: affected → fixed
status-firefox15: affected → fixed
status-firefox16: affected → fixed
Flags: in-litmus?(alex.lakatos)
Whiteboard: [mozmill-test-failure][mozmill-test-skipped] → [mozmill-test-failure]
(Assignee)

Comment 13

5 years ago
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.
Flags: in-litmus?(alex.lakatos) → in-litmus+
Whiteboard: [mozmill-test-failure] → [mozmill-test-failure][qa-]
You need to log in before you can comment on or make changes to this bug.