Closed Bug 725486 Opened 10 years ago Closed 9 years ago

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

Categories

(Mozilla QA Graveyard :: Mozmill Tests, defect)

defect
Not set
normal

Tracking

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

RESOLVED FIXED
Tracking Status
firefox14 --- fixed
firefox15 --- fixed
firefox16 --- fixed
firefox17 --- fixed
firefox-esr10 --- fixed

People

(Reporter: u279076, Assigned: AlexLakatos)

References

()

Details

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

Attachments

(2 files, 2 obsolete files)

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: nobody → alex.lakatos
Status: NEW → ASSIGNED
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?
(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.
Attached patch patch v1.0 (obsolete) — Splinter Review
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-
Depends on: 762964
Depends on: 768358
Do we have to use the OV cert here? I would say the DV one will also be enough.
Attached patch patch v2.0 (obsolete) — Splinter Review
(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
Attached patch patch v3.0Splinter Review
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
Closed: 9 years ago
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.
Flags: in-litmus?(alex.lakatos)
Whiteboard: [mozmill-test-failure][mozmill-test-skipped] → [mozmill-test-failure]
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-]
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in before you can comment on or make changes to this bug.