933270 - Allow some xpcshell-tests to run in parallel

Status: RESOLVED FIXED

(MailNews Core :: Testing Infrastructure, defect)

Description

[Mark Banner (:standard8)]

Attachment: The fix

The attached patch removes some of the forced run-sequentially that were added to avoid bustage when parallel xpcshell-tests were enabled.

The tests I'm enabling to run in parallel are all ones that don't use the fake servers.

On my system, it saves about 5% on 5 minutes when running the xpcshell-tests specific to mailnews, so not much, but still an improvement.

I've only touched this at the directory level for now, we could probably do more later for being explicit about which tests do/don't need to be run in sequence (or just fix the fake servers), but we can leave them to other bugs or bug 905910.

Comment 1

[Joshua Cranmer [:jcranmer]]

Comment on attachment 825310 [details] [diff] [review]
The fix

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

I've tested a similar patch locally (minus the mail/* tests), and IIRC, all the other directories break when global parallellization is reenabled, so this looks good to me.

Comment 2

[Mark Banner (:standard8)]

https://hg.mozilla.org/comm-central/rev/4a83d411ad62