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).
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.
Comment on attachment 595789 [details] [diff] [review]
disable v1 (all) [landed]
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.
Created attachment 630587 [details] [diff] [review]
Opening the tabzilla panel. There is no more Go button, so using Enter
Comment on attachment 630587 [details] [diff] [review]
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.
Do we have to use the OV cert here? I would say the DV one will also be enough.
Created attachment 649296 [details] [diff] [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
The type of certificate does not matter, as long as I can access the website using https.
Comment on attachment 649296 [details] [diff] [review]
> 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,
>+ 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".
Created attachment 649690 [details] [diff] [review]
addressed requested reviews
Pushed to default:
If everything works fine we should backport the patch to older branches in the next days.
Pushed to older branches:
Alex, please re-enable the Mozmill coverage on Litmus for all branches.
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.