Opened 9 years ago

Closed 9 years ago

#2856 enhancement closed fixed (fixed)

trial verbose color reporter should colorize final "PASSED" and "FAILED" message

Reported by: Jean-Paul Calderone Owned by:
Priority: highest Milestone:
Component: trial Keywords:
Cc: therve, Jonathan Lange Branch:
Author:

Description

It'd be much easier to read the summary output if "PASSED" were green and "FAILED" were red, or maybe if the entire lines were colored according to that status information.

Change History (6)

comment:1 Changed 9 years ago by therve

Owner: changed from Jonathan Lange to therve

comment:2 Changed 9 years ago by therve

(In [21407]) Color the ouput and add tests for it.

Refs #2856

comment:3 Changed 9 years ago by therve

Cc: therve Jonathan Lange added
Keywords: review added
Owner: therve deleted
Priority: lowhighest

That's a nice idea, implemented and ready to review in trial-colored-summary-2856. I choose to color only the main message, not the whole line.

comment:4 Changed 9 years ago by Jean-Paul Calderone

Keywords: review removed
Owner: set to therve

It's too bad Reporter still has that stream attribute. :(

printSummary doesn't seem to be part of any interface. I guess this is the reporter object. The runner calls result.printSummary(), anyway, and I guess result is supposed to provide IReporter. But that's an existing problem, not a new one.

Doing if not wasSuccessful/FAILED instead of if wasSuccessful/PASSED is slightly round-about, but I see that's just how the parent implementation being overridden works. Plus it's not a very big deal in the first place...

I'm just nit picking basically, the branch looks great. Thanks!

comment:5 Changed 9 years ago by therve

Resolution: fixed
Status: newclosed

(In [21410]) Merge trial-colored-summary-2856

Author: therve Reviewer: exarkun Fixes #2856

Make the trial colored verbose reporter colorize final PASSED and FAILED messages.

comment:6 Changed 6 years ago by <automation>

Owner: therve deleted
Note: See TracTickets for help on using tickets.