#22449: Colorize output of test results
-------------------------------------+-------------------------------------
     Reporter:  Jake Rothenbuhler    |                    Owner:  Yuri
                                     |  Shikanov
         Type:  New feature          |                   Status:  assigned
    Component:  Core (Management     |                  Version:  master
  commands)                          |
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Ready for
                                     |  checkin
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Chris Jerdonek):

 FWIW, I agree with Tim. Also, the proposed implementation seems to tightly
 couple much of Django's test harness with the idea of coloring. For
 example, `DebugSQLTextTestResult` now inherits from
 `ColoredTextTestResult` when conceptually debugging SQL should be
 orthogonal to coloring. I would rather see a looser coupling (e.g. making
 it easier to "plug" coloring in through appropriate hooks or composition,
 etc.) rather than making the class hierarchies even heavier weight.

-- 
Ticket URL: <https://code.djangoproject.com/ticket/22449#comment:14>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/071.51dd1b96f6ee6c3624df4529cfa8fe1c%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to