1167412 - Install testing modules in all mochitest suites

Status: RESOLVED FIXED

(Testing :: Mochitest, defect)

Description

[Blake Kaplan (:mrbkap) (inactive)]

Currently modules like ContentTaskUtils.jsm are only installed in browser-chrome tests. We should install them in all of the suites.

Comment 1

[Blake Kaplan (:mrbkap) (inactive)]

Attachment: patch v1

I *think* that we always register the specialpowers extension, though I can't acutally find the makefile/moz.build file that is responsible for doing so.

Comment 2

[(not currently active) Ted Mielczarek]

Comment on attachment 8609033 [details] [diff] [review]
patch v1

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

I was judging by the comment here guessing that we wouldn't hit this code in chrome/browser-chrome tests:
https://dxr.mozilla.org/mozilla-central/source/testing/specialpowers/components/SpecialPowersObserver.js#23

...but it might be a lie. In any event, mochitest-chrome/browser-chrome would blow up pretty quickly if this doesn't work, so a Try push should prove it out.

Comment 3

[Blake Kaplan (:mrbkap) (inactive)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=a6849e6a96f4

Comment 4

[Blake Kaplan (:mrbkap) (inactive)]

I got worried when I saw your comment, but I had actually checked locally with a couple of browser-chrome tests to make sure they still worked. This is good to go.

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/f886fbfcd550

Comment 6

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/f886fbfcd550