[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] input_validation_tests.py: Reduce overhead of multiple checkouts.

From: Noorul Islam K M <noorul_at_collab.net>
Date: Mon, 21 Feb 2011 16:02:39 +0530

Noorul Islam K M <noorul_at_collab.net> writes:

> Stefan Sperling <stsp_at_elego.ed> writes:
>
>> On Mon, Feb 21, 2011 at 01:44:35PM +0530, Noorul Islam K M wrote:
>>
>>>
>>> This patch reduces checkout by around 23 times.
>>
>> On my system the difference is 43 seconds vs. 30 seconds.
>>
>> We lose the ability to easily spot which of the subtest is failing
>> if we do this. I.e. instead of:
>>
>> ...
>> PASS: input_validation_tests.py 19: non-working copy paths for 'status'
>> FAIL: input_validation_tests.py 20: non-working copy paths for 'patch'
>> PASS: input_validation_tests.py 21: non-working copy paths for 'switch'
>> ...
>>
>> all we'd get is:
>>
>> FAIL: input_validation_tests.py 1: inavlid wc and url targets
>>
>> Is there a way of keeping these as individual tests but also
>> avoiding the overhead of creating a repository and a working copy?
>> If there isn't I would prefer to just leave this as it is now because
>> I prefer the current output.
>
> I think it will be possible by keeping sandbox global. I will modify and
> send an updated patch.
>

I looked into it. I don't think it is straight forward. I will leave it
as such. As you said 13 seconds gain is no big deal.

Thanks and Regards
Noorul
Received on 2011-02-21 11:34:45 CET

This is an archived mail posted to the Subversion Dev mailing list.