Autophone - newlogparser - parse Robocop summary markers

RESOLVED FIXED

Status

RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: bc, Assigned: bc)

Tracking

Trunk
x86
All
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
'newlogparser' does not handle the Robocop summary properly as it thinks it is another test run rather than a summary of all of the test runs. The result is a reported malformed test log. bug 826308 added new markers to bound the summary so that newlogparser can distinguish the overall summary from the individual summaries.

Test summary: start.
0 INFO SimpleTest START
99 INFO TEST-START | Shutdown
99 INFO Passed: %s
99 INFO Failed: %s
99 INFO Todo: %s
99 INFO SimpleTest FINISHED
Test summary: end.
(Assignee)

Comment 1

6 years ago
Created attachment 698785 [details] [diff] [review]
patch

Distinguish between test run summaries and overall Robocop summaries.
Attachment #698785 - Flags: review?(mcote)
Comment on attachment 698785 [details] [diff] [review]
patch

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

Seems fine.
Attachment #698785 - Flags: review?(mcote) → review+
(Assignee)

Comment 3

6 years ago
https://github.com/mozilla/autophone/commit/875b63e9107f8bc7245c58fd4cb9b8025b80bd5d
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.