1468634 - Allow running a single APZ mochitest subtest at a time

Status: RESOLVED FIXED

(Core :: Panning and Zooming, enhancement, P3)

Description

[Botond Ballo [:botond]]

For quick iteration on an APZ mochitest subtest, it would be nice if there way a way to just run the one subtest rather than the entire test group (that's more convenient than temporarily modifying the test_group_*.html file to comment out all the other subtests)

Comment 1

[Botond Ballo [:botond]]

Andrew, is there a way for |mach mochitest| to somehow "pass through" an argument to the test being run (i.e., make an argument available in a place that a mochitest's code can read)?

If we had that ability, then wiring this up on the APZ test runner side would be easy.

Comment 2

[Andrew Halberstadt [:ahal]]

Prefs are probably your best bet. You could use --setpref for the time being, or if you want a more descriptive argument we can create one that sets a pref under the hood.

Comment 3

[Botond Ballo [:botond]]

Thanks, that'll be good enough for now.

Comment 4

[Botond Ballo [:botond]]

Attachment: Bug 1468634 - Don't silently swallow exceptions thrown in rubSubtestsSeriallyInFreshWindows().

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

Comment 5

[Botond Ballo [:botond]]

Attachment: Bug 1468634 - Allow running a single APZ mochitest subtest at a time.

This is done by passing "--setpref apz.subtest=<subtest-name>" on the
mochitest command line, where <subtest-name> is the name of the subtest file
(including the extension).

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

Comment 6

[Kartikaya Gupta (email:[e-mail])]

Comment on attachment 8985848 [details]
Bug 1468634 - Don't silently swallow exceptions thrown in rubSubtestsSeriallyInFreshWindows().

https://reviewboard.mozilla.org/r/251328/#review257660

Comment 7

[Kartikaya Gupta (email:[e-mail])]

Comment on attachment 8985849 [details]
Bug 1468634 - Allow running a single APZ mochitest subtest at a time.

https://reviewboard.mozilla.org/r/251330/#review257662

Nice, thanks!

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b42241e5b198
Don't silently swallow exceptions thrown in rubSubtestsSeriallyInFreshWindows(). r=kats
https://hg.mozilla.org/integration/autoland/rev/628bd444ead1
Allow running a single APZ mochitest subtest at a time. r=kats

Comment 9

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/b42241e5b198
https://hg.mozilla.org/mozilla-central/rev/628bd444ead1