Closed Bug 1301334 Opened 3 years ago Closed 3 years ago

Package Marionette tests via root manifests

Categories

(Firefox Build System :: General, defect)

49 Branch
defect
Not set

Tracking

(firefox52 fixed)

RESOLVED FIXED
mozilla52
Tracking Status
firefox52 --- fixed

People

(Reporter: whimboo, Assigned: whimboo)

References

(Blocks 2 open bugs)

Details

(Whiteboard: [test-run-speed-up])

Attachments

(1 file)

With bug 1283919 landed we can now package tests via root manifests. We could do the same for Marionette but it would be better to directly make use of the moz.build file entries for related manifest files.
As discovered while working on bug 1302364 we might have a problem if we do not include the root certificates in the test archive. Not sure how mozharness scripts will pick all those tests.

So I would propose we move out the packaging via moz.build to a follow-up bug, and get my patch similar to the one as formerly contained on bug 1283919 - just without the removal of moz.build entries - landed.
Summary: Package Marionette tests via moz.build entries → Package Marionette tests via root manifests
Whiteboard: [test-run-speed-up]
Gregory, the patch is basically the same as it has been landed formerly via bug 1283919 but without the removal of the moz.build entries. I only removed the one for loop tests, which do no longer exist.
Comment on attachment 8792948 [details]
Bug 1301334 - Package Marionette tests via moz.build entries

https://reviewboard.mozilla.org/r/79778/#review79232
Attachment #8792948 - Flags: review?(gps) → review+
Pushed by hskupin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f9edea7fac2d
Package Marionette tests via moz.build entries r=gps
https://hg.mozilla.org/mozilla-central/rev/f9edea7fac2d
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.