Closed Bug 1263227 Opened 4 years ago Closed 4 years ago

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

Categories

(Firefox Build System :: General, defect)

defect
Not set

Tracking

(firefox48 fixed)

RESOLVED FIXED
mozilla48
Tracking Status
firefox48 --- fixed

People

(Reporter: mossop, Assigned: chmanchester)

Details

Attachments

(1 file)

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?
Flags: needinfo?(cmanchester)
Yes, I think I see the problem.
Assignee: nobody → cmanchester
Flags: needinfo?(cmanchester)
Attachment #8739515 - Flags: review?(dtownsend) → review+
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
https://hg.mozilla.org/mozilla-central/rev/15f26aca5330
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.