1084322 - Allow reftest analyzer to accept pasting of non-raw log output from treeherder

Status: RESOLVED FIXED

(Testing :: Reftest, defect)

Description

[Jonathan Watt [:jwatt]]

Treeherder collapses the spaces between "REFTEST" and "IMAGE" in the lines:

  INFO -  REFTEST   IMAGE 1 (TEST): ...

Comment 1

[Jonathan Watt [:jwatt]]

Attachment: patch

Since I'm making the spaces optional, I've made the rest of the regexp a little more specific.

Note that one space is removed by the:

  var match = line.match(/^.*?REFTEST (.*)$/);

line, hence why we're dealing with two rather than three in this later regexp.

Comment 2

[David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)]

Comment on attachment 8506815 [details] [diff] [review]
patch

Now I learned about non-capturing parentheses.

r=dbaron

Comment 3

[Jonathan Watt [:jwatt]]

Comment on attachment 8506815 [details] [diff] [review]
patch

https://hg.mozilla.org/integration/mozilla-inbound/rev/c567e39f19a0

Comment 4

[Phil Ringnalda (:philor)]

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

Comment 5

[David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)]

This broke the display of failures of != tests, which just have "IMAGE:" with nothing between "IMAGE" and ":".  I'll land the obvious patch as a followup.

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/83e66875404e

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/83e66875404e