806081 - Remove references from mozilla.com in our mozmill tests

Status: RESOLVED FIXED

(Mozilla QA Graveyard :: Mozmill Tests, defect)

Description

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

To lower the amount of hosts we connect to for our mozmill tests we should get rid of mozilla.com. Right now only the testStopReloadButtons.js test makes use of it. We can easily port this to mozilla.org until we have a working testcase which works locally.

Comment 1

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

Attachment: Patch v1

Comment 2

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Comment on attachment 675832 [details] [diff] [review]
Patch v1

Review of attachment 675832 [details] [diff] [review]:
-----------------------------------------------------------------

r=me with the whitespace nit addressed.

::: tests/functional/testToolbar/testStopReloadButtons.js
@@ +9,4 @@
>  
>  var setupModule = function(module) {
>    module.controller = mozmill.getBrowserController();
> +  locationBar =  new toolbars.locationBar(controller);

nit: Extra whitespace.

Comment 3

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

http://hg.mozilla.org/qa/mozmill-tests/rev/8ae83bc17b52 (default)

Comment 4

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

No test failures so lets get this patch backported:

http://hg.mozilla.org/qa/mozmill-tests/rev/e9bea4e2f8ee (aurora)
http://hg.mozilla.org/qa/mozmill-tests/rev/0a14d82c63d3 (beta)
http://hg.mozilla.org/qa/mozmill-tests/rev/16f4876f7d86 (release)
http://hg.mozilla.org/qa/mozmill-tests/rev/4e732c0cff4e (esr10)