Opened 10 years ago

Closed 10 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

Description

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 10 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 10 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 10 years ago by Jean-Paul Calderone

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

Changed 10 years ago by Jean-Paul Calderone

Attachment: trial.txt added

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

Keywords: review added
Owner: Jonathan Lange deleted

comment:4 Changed 10 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 10 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 10 years ago by <automation>

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