If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Add support for cppunittest js and manifest files to mozharness

RESOLVED FIXED

Status

Release Engineering
Mozharness
RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: dminor, Assigned: dminor)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
The TestStartupCache cpp unittest currently fails everywhere because its manifest and js files are not packaged.

Bug 927989 will add these to the test package, but the test itself looks for the files in application binary directory. A preflight method can be added to move these from the test directory to the appropriate location.
(Assignee)

Comment 1

4 years ago
Created attachment 820324 [details] [diff] [review]
Patch to copy manifest and js files to abs_app_dir
Assignee: nobody → dminor
Status: NEW → ASSIGNED
Attachment #820324 - Flags: review?(aki)

Comment 2

4 years ago
Comment on attachment 820324 [details] [diff] [review]
Patch to copy manifest and js files to abs_app_dir

>+        for f in files:
>+            shutil.move(f, abs_app_dir)

self.move() will add logging.
Attachment #820324 - Flags: review?(aki) → review+
(Assignee)

Comment 3

4 years ago
Thanks, with comment addressed: https://hg.mozilla.org/build/mozharness/rev/ea5514077c82
merged to production mozharness
(Assignee)

Updated

4 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED

Updated

3 years ago
Component: General Automation → Mozharness
You need to log in before you can comment on or make changes to this bug.