[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: Branko Čibej <brane_at_e-reka.si>
Date: Mon, 21 Feb 2011 12:53:31 +0100

On 21.02.2011 11:32, Noorul Islam K M wrote:
> 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.

Given that the one commandment for writing test cases is to make them
independent of each other, I can hardly see how you could keep a global
sandbox. The most important "optimization" for tests is to make them
more reliable, testcase performance comes third on the list at best
(after comprehensive).

-- Brane
Received on 2011-02-21 12:54:14 CET

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.