Re: [PATCH] Time taken to execute the test suite

Julian Foad wrote:
> Bhuvaneswaran A wrote:
>
>> Thank you once again for the review comments, Branko.
>>
>> Please find attached the revised patch. The log message remains the
>> same.
>>
>
> I'm not sure Branko is still on line now,

'Course I'm online. It's not even midnight yet. :)

> but it looks good to me. +1.
>
> [[[
> PASS: autoprop_tests.py 15: fail to add a file with mixed EOL style
> CLEANUP: svn-test-work/working_copies/autoprop_tests-15
> CLEANUP: svn-test-work/repositories/autoprop_tests-15
> END: autoprop_tests.py
> ELAPSED: 00:00:14
>
> START: basic_tests.py
> [...]
> ]]]
>
> (You can always tweak it later if, for example, you decide it would be
> better to include the name of the test in the ELAPSED line.)
>

Yes, I was going to say ... to make parsing the log file easier, the
name of the test should be in the ELAPSED: line -- as per my suggestion
in an earlier mail.

But +1, please go ahead and commit this. It looks like a nice change to
back-port to 1.6, too.

-- Brane

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2405271
Received on 2009-10-08 20:33:02 CEST