#634 closed enhancement (fixed)

trial Reporter which shows per-test times

Reported by: warner Owned by: warner
Priority: minor Milestone: 1.4.1
Component: dev-infrastructure Version: 1.3.0
Keywords: Cc:
Launchpad Bug:

Description

I've been casually trying to speed up our unit test suite, and it would help me focus my attention if the Reporter class could tell me which tests were taking the most time (especially on the slower buildslaves, where I'm not watching the output in realtime).

So it's be nice if there were a Reporter class which would emit the elapsed time of each test case, right next to the [OK] line, something like:

allmydata.test.test_system
  SystemTest
    test_mutable ...                    [OK] 2.12s

We have a custom Reporter class for figleaf already, so it shouldn't be too hard to build this one.

I'd want to run this on all the buildbot builds. I'm not sure I'd want to make it the default for 'make test', but maybe it wouldn't be too distracting.

Change History (2)

comment:1 Changed at 2009-02-19T05:33:39Z by warner

Oh, hey, look at that. Trial has had this built-in since at least Twisted-2.3.0, using --reporter=timing. The output looks different, but it's close enough for me:

allmydata.test.test_crawler.Basic.test_cpu_usage ... [OK]
(4.220 secs)
allmydata.test.test_crawler.Basic.test_empty_subclass ... [OK]
(0.899 secs)

I think I'll switch the buildslaves to use this reporter.

comment:2 Changed at 2009-02-19T05:44:05Z by warner

  • Milestone changed from undecided to 1.3.1
  • Resolution set to fixed
  • Status changed from new to closed

that works great

Note: See TracTickets for help on using tickets.