Closed
Bug 768222
Opened 13 years ago
Closed 13 years ago
Make pprint-color the default logging format for Mozmill
Categories
(Testing Graveyard :: Mozmill, defect)
Testing Graveyard
Mozmill
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: whimboo, Assigned: whimboo)
Details
(Whiteboard: [mozmill-2.0])
Attachments
(2 files)
Pure JSON output for the Mozmill CLI tool is not the right solution. It's obvious that this output is hard to read by any tester who is running the tool manually. For automation tools we should enforce that those have to set the --format option to JSON if they really need JSON output.
Assignee | ||
Comment 1•13 years ago
|
||
Pointer to Github pull-request
Assignee | ||
Updated•13 years ago
|
Attachment #636495 -
Attachment description: Pointer to Github pull request: https://github.com/mozautomation/mozmill/pull/58 → Patch
Attachment #636495 -
Flags: review?(jhammel)
Assignee | ||
Updated•13 years ago
|
Attachment #636495 -
Flags: review?(jhammel) → review?(ctalbert)
Attachment #636495 -
Flags: review?(ctalbert) → review+
Assignee | ||
Comment 2•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [mozmill-2.0]
Assignee | ||
Comment 3•13 years ago
|
||
Pointer to Github pull-request
Assignee | ||
Comment 4•13 years ago
|
||
Comment on attachment 637552 [details]
Follow-up for class instances
Sorry that I have missed that but while working on the automation scripts I have seen that we didn't use the pprint_color mode when the LoggerListener gets instantiated.
Attachment #637552 -
Attachment description: Pointer to Github pull request: https://github.com/mozautomation/mozmill/pull/60/files → Follow-up for class instances
Attachment #637552 -
Flags: review?(ctalbert)
Assignee | ||
Updated•13 years ago
|
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Attachment #637552 -
Flags: review?(ctalbert) → review+
Assignee | ||
Comment 5•13 years ago
|
||
Status: REOPENED → RESOLVED
Closed: 13 years ago → 13 years ago
Resolution: --- → FIXED
Updated•9 years ago
|
Product: Testing → Testing Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•