search

mahmoudimus / nose-timer

A timer plugin for nosetests (how much time does every test take?)
MIT License
126 stars 33 forks source link

Have timings show up on the same line as the test. #19

Closed jsalvatier closed 10 years ago

jsalvatier commented 10 years ago

It would be great if timings would show up on the same line as the test when run like nosetests -vv --with-timer --timer-verbose. I don't know if this is possible, but would definitely be preferable.

e0ne commented 10 years ago

Hi, @jsalvatier, --timer-verbose param should helps: You can output test times after each test with --timer-verbose flag. Does it work?

jsalvatier commented 10 years ago

I'm already using that flag :). However, it still outputs on the next line rather than on the same line.

skudriashev commented 10 years ago

@jsalvatier, if I understood correctly you want - original report with test time in the same line and nose-timer plugin report below sorted out from the quickest to the slowest test (like it is now). Am I right?

skudriashev commented 10 years ago

@jsalvatier, so what do you think?

jsalvatier commented 10 years ago

Yes, that sounds pretty good.

skudriashev commented 10 years ago

@jsalvatier, please, take a look!

e0ne commented 10 years ago

Looks like this issue is fixed. @jsalvatier, could you confirm it, please?

skudriashev commented 10 years ago

@jsalvatier, what do you think?

jsalvatier commented 10 years ago

I'm trying this out and it looks fine locally, but if I try to use it in Travis, I get an error "nosetests: error: no such option: --timer-verbose". Any ideas?

https://travis-ci.org/pymc-devs/pymc/jobs/22084430

On Mon, Mar 31, 2014 at 5:38 PM, Stanislav notifications@github.com wrote:

@jsalvatier https://github.com/jsalvatier, what do you think?

Reply to this email directly or view it on GitHubhttps://github.com/mahmoudimus/nose-timer/issues/19#issuecomment-39159254 .

skudriashev commented 10 years ago

@jsalvatier, you don't need the --timer-verbose option any more (it is removed by this patch). Timings are printed on the same line by default (previously this option was needed because timings were printed by plugin logger that was turned on by the --timer-verbose option).

So, removing it from your config will help.

skudriashev commented 10 years ago

@jsalvatier, it is good now?

jsalvatier commented 10 years ago

Yup, looks good.

On Sun, Apr 6, 2014 at 10:45 AM, Stanislav notifications@github.com wrote:

@jsalvatier https://github.com/jsalvatier, it is good now?

Reply to this email directly or view it on GitHubhttps://github.com/mahmoudimus/nose-timer/issues/19#issuecomment-39674690 .