1688208 - Android mochitests should log similar to desktop; currently e.g. WARNING - <processID> ERROR TEST-UNEXPECTED-FAIL | ...

Status: RESOLVED FIXED

(Testing :: Mochitest, task)

Description

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

Android mochitests should log similar to desktop; currently they have e.g. WARNING - <processID> ERROR TEST-UNEXPECTED-FAIL | ... (example).
On desktop, the format is ERROR - TEST-UNEXPECTED-FAIL | ... (example).

This causes some differences in tooling like bugs tracking intermittent failures containing <processID> ERROR TEST-UNEXPECTED-FAIL | in the failure summary. There might already be a bug for the issue (didn't find it).

Comment 1

[Geoff Brown [:gbrown]]

It looks like the extra <num> <level> is added by the MochitestFormatter; I think that can be safely removed now. Testing a patch...

Comment 2

[Geoff Brown [:gbrown]]

Attachment: Bug 1688208 - Remove MochitestFormatter to align android and desktop log format; r=

The MochitestFormatter adds an additional count and log level, from the mochitest test
harness. As far as I can tell, the MochitestFormatter is not called during desktop tests
(only Android); this difference appears accidental. Removing this log formatter removes
the extra, undesired fields.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/549540d9f5ef
Remove MochitestFormatter to align android and desktop log format; r=jmaher,aryx

Comment 4

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/549540d9f5ef