Closed
Bug 927989
Opened 9 years ago
Closed 9 years ago
Add TestStartupCacheTelemetry js and manifest files to test package
Categories
(Testing :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla27
People
(Reporter: dminor, Assigned: dminor)
Details
Attachments
(1 file)
1.31 KB,
patch
|
ted
:
review+
|
Details | Diff | Splinter Review |
This test relies upon TestStartupCacheTelemetry.js and TestStartupCacheTelemetry.manifest which aren't currently packaged, and so fails across the board on Cedar. It looks for these files under /application/firefox which is awkward for packaging. The easiest approach would be to either move these to a location that is packaged in the test zip, or check both the current location and one that is packaged. Packaging these could potentially also require some small modifications to the test harness logic to run all files in a directory so that it does not try to execute the .manifest and .js files.
Assignee | ||
Comment 1•9 years ago
|
||
It looks like this can be fixed by adding these to test package and moving the files to appropriate location from the mozharness script.
Summary: cpp unittest TestStartupCache fails when run from test package → Add TestStartupCacheTelemetry js and manifest files to test package
Assignee | ||
Comment 2•9 years ago
|
||
Updated•9 years ago
|
Attachment #820545 -
Flags: review?(ted) → review+
Assignee | ||
Comment 3•9 years ago
|
||
Thanks, https://hg.mozilla.org/integration/mozilla-inbound/rev/376956b7585a
Comment 4•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/376956b7585a
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
You need to log in
before you can comment on or make changes to this bug.
Description
•