511314 - [mozmill] Test for verifying BLUE Larry

Status: RESOLVED FIXED

(Mozilla QA Graveyard :: Mozmill Tests, defect)

Description

[u279076]

This is a placeholder bug for creating a Mozmill test script for https://litmus.mozilla.org/show_test.cgi?id=6872

Comment 1

[u279076]

Attachment: Initial patch

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Comment on attachment 395218 [details] [diff] [review]
Initial patch

Most of the comments would also apply to bug 511318 and bug 511208 so I will leave them out here.

>+  var gETLDService = Components.classes["@mozilla.org/network/effective-tld-service;1"]
>+                       .getService(Components.interfaces.nsIEffectiveTLDService);

Sorry, that i haven't mentioned it in the green larry test but you should move this up into setupModule.

Comment 3

[u279076]

Attachment: Rev1

Comment 4

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Comment on attachment 397173 [details] [diff] [review]
Rev1

>+var testLarryBlue = function() {
>+  // Go to a "blue" website
>+  controller.open("https://www.mozilla.org/");

We cannot use this website anymore for Larry tests because it contains unencrypted content and brings up a modal sheet when loading the page. Lets move over to bugzilla.mozilla.org which works fine.

Further I have removed the usage of the string bundle and replaced it by the UtilsAPI.getProperty call.

I will attach a diff to this bug.

Comment 5

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Attachment: Follow-up

Comment 6

[u279076]

Comment on attachment 397830 [details] [diff] [review]
Follow-up

Looks good to me.

Comment 7

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Landed as:
http://hg.mozilla.org/qa/mozmill-tests/rev/85fe3f8ecb63
http://hg.mozilla.org/qa/mozmill-tests/rev/1a2faf42af1b

I have also fixed the closing of the page info window for the grey larry test with this check-in.

Comment 8

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Mass move of Mozmill Test related project bugs to newly created components. You can filter out those emails by using "Mozmill-Tests-to-MozillaQA" as criteria.