1146885 - Make marionette use manifestparser's chunking algorithm

Status: NEW

(Testing :: Marionette Client and Harness, defect, P5)

Description

[Andrew Halberstadt [:ahal]]

We recently moved all chunking logic out of mochitest and into manifestparser, time to do the same for marionette.

Comment 1

[Andrew Halberstadt [:ahal]]

This is slightly more complicated than mochitest and xpcshell because marionette allows arbitrary test inputs, i.e you can pass in a combination of test files, directories and manifest files all at the same time. Obviously the manifestparser chunking will only apply to manifest files and not directories/test files. One possible way to get around this (which is what the mochitest mach command does) is to manually append the tests passed in via file/directory to manifest.tests, and then do the filtering.

Fwiw cppunittests also mixes test sources together.

Comment 2

[(not currently active) Ted Mielczarek]

Adding everything into manifest.tests seems fine. In reality the only situation we care about is chunking from a single manifest input, so as long as that works well and passing arbitrary stuff on the commandline works at all it should be fine.