473506 - Synchronize BrowserChrome and Mochitest log formats

Status: RESOLVED DUPLICATE of bug 600413

(Testing :: Mochitest, defect)

Description

[Serge Gautherie (:sgautherie)]

At least, I'd like the summary formats to be synchronized.

browser-harness.xul: (BrowserChrome)
{
Browser Chrome Test Summary
	Pass: 1272
	Fail: 0
	Todo: 5
}

SimpleTest.js: (MochiTest, MochiChrome, A11y)
{
1327 INFO Passed: 1177
1328 INFO Failed: 0
1329 INFO Todo:   79
}

Comment 1

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) Mochitest: remove '*** ' console output [Checkin: Comment 2 & 3]

Bug 364043 added this to new console output but not to existing file output.

This difference broke my bug 473259 patch Ev1,
because I test locally from the file, whereas buildbot uses the console :-/

Comment 2

[Serge Gautherie (:sgautherie)]

Comment on attachment 379695 [details] [diff] [review]
(Av1) Mochitest: remove '*** ' console output
[Checkin: Comment 2 & 3]


http://hg.mozilla.org/mozilla-central/rev/cf17e49a665a

Comment 3

[Serge Gautherie (:sgautherie)]

Comment on attachment 379695 [details] [diff] [review]
(Av1) Mochitest: remove '*** ' console output
[Checkin: Comment 2 & 3]


http://hg.mozilla.org/releases/mozilla-1.9.1/rev/a973204ab988

Comment 4

[Serge Gautherie (:sgautherie)]

Attachment: (Bv1) BrowserChrome: always use 'Passed' and 'Failed' idents [Checkin: See comment 7+14 & 15 & 16]

Bug 387455 added this to text output but it differs from html output,
and more importantly from other Mochitest output(s).

This has always needed a specific case in buildbot: see bug 473259 patch Ev1a.
(That buildbot file will need to be updated in sync' too...)

Comment 5

[Serge Gautherie (:sgautherie)]

rcampbell, ping for review ?

Comment 6

[Serge Gautherie (:sgautherie)]

rcampbell: ping for review?

Comment 7

[Serge Gautherie (:sgautherie)]

Comment on attachment 379786 [details] [diff] [review]
(Bv1) BrowserChrome: always use 'Passed' and 'Failed' idents
[Checkin: See comment 7+14 & 15 & 16]


http://hg.mozilla.org/mozilla-central/rev/b0c89755c903
Bv1, unbitrotted.

Comment 8

[Serge Gautherie (:sgautherie)]

Comment on attachment 379786 [details] [diff] [review]
(Bv1) BrowserChrome: always use 'Passed' and 'Failed' idents
[Checkin: See comment 7+14 & 15 & 16]


http://hg.mozilla.org/releases/mozilla-1.9.2/rev/eaea03ee6784

Comment 9

[Serge Gautherie (:sgautherie)]

I backed out patch Bv1 because I missed an additional change in buildbotcustom.
Will reland this patch when bbc is ready.

Comment 10

[Serge Gautherie (:sgautherie)]

Comment on attachment 379786 [details] [diff] [review]
(Bv1) BrowserChrome: always use 'Passed' and 'Failed' idents
[Checkin: See comment 7+14 & 15 & 16]


http://hg.mozilla.org/mozilla-central/rev/1d95bcb9b00f

Comment 11

[Mike Beltzner [:beltzner, not reading bugmail]]

Can we back this out? Since it landed there have been a ton of orange boxes.

Comment 12

[:Gijs (he/him)]

(In reply to comment #11)
> Can we back this out? Since it landed there have been a ton of orange boxes.

I backed this out per https://bugzilla.mozilla.org/show_bug.cgi?id=473259#c55, where Serge says it needs to be backed out because of an earlier backout.

http://hg.mozilla.org/mozilla-central/rev/2bf3249fe4bb

Comment 13

[Serge Gautherie (:sgautherie)]

(In reply to comment #11)
> Since it landed there have been a ton of orange boxes.

No, there was only one reported,
the other reports were bug 473259 comment 55 "mess".

(In reply to comment #12)
> I backed this out per https://bugzilla.mozilla.org/show_bug.cgi?id=473259#c55,

Bug 473259 patch was re-landed after that.
Yet, I'll let the situation settle for a while before re-landing this patch...

Comment 14

[Serge Gautherie (:sgautherie)]

Comment on attachment 379786 [details] [diff] [review]
(Bv1) BrowserChrome: always use 'Passed' and 'Failed' idents
[Checkin: See comment 7+14 & 15 & 16]


http://hg.mozilla.org/mozilla-central/rev/7f6e2f4abdfd

Comment 15

[Serge Gautherie (:sgautherie)]

Comment on attachment 379786 [details] [diff] [review]
(Bv1) BrowserChrome: always use 'Passed' and 'Failed' idents
[Checkin: See comment 7+14 & 15 & 16]


http://hg.mozilla.org/releases/mozilla-1.9.2/rev/58087da8caa6

Comment 16

[Serge Gautherie (:sgautherie)]

Comment on attachment 379786 [details] [diff] [review]
(Bv1) BrowserChrome: always use 'Passed' and 'Failed' idents
[Checkin: See comment 7+14 & 15 & 16]


http://hg.mozilla.org/releases/mozilla-1.9.1/rev/5e85a1d1c673