1147104 - Add running e10s based mochitests to run-all-loop-tests.sh

Status: RESOLVED FIXED

(Hello (Loop) :: Client, defect, P2)

Description

[Mark Banner (:standard8)]

We keep hitting e10s failures when we land things on fx-team.

We should extend run-all-loop-tests.sh to run the e10s versions of tests as well, as several of us rely on this to make sure we've run the relevant tests.

Patch coming up

Comment 1

[Mark Banner (:standard8)]

Attachment: Add running e10s based mochitests to run-all-loop-tests.sh to help catch Loop failures before they land. NPOTB DONTBUILD

Runs e10s by default, but also provides a simple option to skip them. I've not done any validation on the arguments supplied, I don't think its worth it. Hopefully in a few cycles we'll be able to drop the non-e10s stuff.

Comment 2

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 8582667 [details] [diff] [review]
Add running e10s based mochitests to run-all-loop-tests.sh to help catch Loop failures before they land. NPOTB DONTBUILD

Review of attachment 8582667 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/components/loop/run-all-loop-tests.sh
@@ +1,4 @@
>  #!/bin/sh
>  # Run from topsrcdir, no args
>  
> +if [ "$1" == "-h" ]; then

Can we use --help here instead of -h?

Comment 3

[Mark Banner (:standard8)]

https://hg.mozilla.org/integration/fx-team/rev/e03fe4951391

Comment 4

[Ryan VanderMeulen [:RyanVM]]

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