Opened 9 years ago

Closed 9 years ago

#4487 enhancement closed fixed (fixed)

report multiple tests with the same results at the same time

Reported by: Jean-Paul Calderone Owned by:
Priority: normal Milestone:
Component: trial Keywords:
Cc: Branch: branches/trial-shorter-summary-4487
branch-diff, diff-cov, branch-cov, buildbot
Author: exarkun


If there are several tests which are skipped (or todo'd, or fail) for the same reason, trial reports them each separately, duplicating the reason each time.

It would be nicer if it grouped together these reports by reason, so that any particular reason would only be reported once, and all of the tests affected by it would be collected together nearby.

Attachments (1)

trial.txt (39.5 KB) - added by Jean-Paul Calderone 9 years ago.

Download all attachments as: .zip

Change History (7)

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

Author: exarkun
Branch: branches/trial-shorter-summary-4487

(In [29464]) Branching to 'trial-shorter-summary-4487'

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

Implemented something. Attached is an example of the new output when running Twisted's test suite.

Changed 9 years ago by Jean-Paul Calderone

Attachment: trial.txt added

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

Keywords: review added
Owner: Jonathan Lange deleted

comment:4 Changed 9 years ago by Screwtape

Keywords: review removed
Owner: set to Jean-Paul Calderone

I ran the tests with every reporter that works on my system, and it prints nicely summarized output every time (and no output at all for "summary", as it should). I couldn't test "subunit", but I don't know any reason why it would break when everything else works.

+1 for merging from me.

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

Resolution: fixed
Status: newclosed

(In [29761]) Merge trial-shorter-summary-4487

Author: exarkun Reviewer: Screwtape Fixes: #4487

Most trial reporters now reports multiple tests with the same results in a combined display in the summary section, shorting the overall output.

comment:6 Changed 8 years ago by <automation>

Owner: Jean-Paul Calderone deleted
Note: See TracTickets for help on using tickets.