1263227 - toolkit/mozapps/extensions/test/browser/head.js is no longer copied to the tests directory

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Dave Townsend [:mossop]]

1. Clobber
2. Do a full build
3. mach mochitest toolkit/mozapps/extensions/test/browser/browser_checkAddonCompatibility.js (or any test in the browser.ini there)

The test will fail because head.js isn't copied along with it.

I suspect this is because head.js is in the browser-common.ini (which is included in browser.ini) but I don't know if it should be in all three browser manifests or what to do here.

Chris, you made some changes to how support-files are processed recently, could that have broken this?

Comment 1

[Chris Manchester (limited bugmail, email directly)]

Yes, I think I see the problem.

Comment 2

[Chris Manchester (limited bugmail, email directly)]

Attachment: MozReview Request: Bug 1263227 - Include head.js in support-files in toolkit/mozapps/extensions/test/browser/browser.ini to avoid failures when running individual tests locally. r=Mossop

Review commit: https://reviewboard.mozilla.org/r/45299/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/45299/

Comment 3

[Dave Townsend [:mossop]]

Comment on attachment 8739515 [details]
MozReview Request: Bug 1263227 - Include head.js in support-files in toolkit/mozapps/extensions/test/browser/browser.ini to avoid failures when running individual tests locally. r=Mossop

https://reviewboard.mozilla.org/r/45299/#review41835

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/15f26aca5330

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/15f26aca5330