1301334 - Package Marionette tests via root manifests

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Henrik Skupin [:whimboo][⌚️UTC+1]]

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.

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+1]]

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.

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Attachment: Bug 1301334 - Package Marionette tests via moz.build entries

Move packaging for Marionette from make to test_archiver by using root manifest files.

**

Review commit: https://reviewboard.mozilla.org/r/79778/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/79778/

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+1]]

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 4

[Gregory Szorc [:gps]]

Comment on attachment 8792948 [details]
Bug 1301334 - Package Marionette tests via moz.build entries

https://reviewboard.mozilla.org/r/79778/#review79232

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f9edea7fac2d
Package Marionette tests via moz.build entries r=gps

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/f9edea7fac2d