Closed Bug 927989 Opened 8 years ago Closed 8 years ago

Add TestStartupCacheTelemetry js and manifest files to test package

Categories

(Testing :: General, defect)

Product:
Testing
Component:
General
Platform:
x86_64
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
mozilla27

People

(Reporter: dminor, Assigned: dminor)

Details

Attachments

(1 file)

Add TestStartupCacheTelemetry manifest and js files to test package
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.
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: nobody → dminor
Status: NEW → ASSIGNED
Attachment #820545 - Flags: review?(ted)
Attachment #820545 - Flags: review?(ted) → review+
https://hg.mozilla.org/mozilla-central/rev/376956b7585a
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
You need to log in before you can comment on or make changes to this bug.