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

Thank you, Branko and Julian.

On Thu, 2009-10-08 at 20:32 +0200, Branko Čibej wrote:
>
> 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.

Yeah, i've included the test name in the ELAPSED: line.

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

hmm, i'm unable to commit. Is commit restricted for partial committers
to commit outside their area? I see this error:

bhuvan_at_rainbow:~/projects/subversion/build$ svn ci run_tests.py
--username bhuvan
svn: Commit failed (details follow):
svn: Server sent unexpected return value (403 Forbidden) in response to
MKACTIVITY request for '/repos/svn/!
svn/act/f251f847-ce9d-46f8-8020-0c35f977b0fb'
svn: Your commit message was left in a temporary file:
svn: '/home/bhuvan/projects/subversion/build/svn-commit.tmp'

If yes, can someone commit the attached patch and propose to back-port
it to 1.6? If no, can someone who has access to svn.collab.net (Mike?)
fix this issue, please?

[[[
This fix adds the ability to print the time taken to execute each test.
The time taken is printed in the log file, if specified; otherwise in
the stdout interface.
  
* build/run_tests.py
  import time
  (TestHarness._run_test): Print the time taken by each test using
  following format:
  ELAPSED: test_name HH:MM:SS

Reviewed by: julianfoad
             brane
]]]

-- 
Bhuvaneswaran A    
CollabNet Software P Ltd.  |  www.collab.net
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2405407