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

Re: svn commit: r26803 - in trunk/subversion: libsvn_client tests/cmdline

From: Kamesh Jayachandran <kamesh_at_collab.net>
Date: 2007-09-28 09:43:13 CEST

>>
>>
> So our test suite lacks this test?
>
>> Anyhow, in r26813 I added an intialization and also tweaked
>> drive_merge_report_editor() to check for a possible NULL
>> children_with_mergeinfo (not really necessary right now, but you check
>> it elsewhere in that function, so in the name of safety and
>> consistency...
>>
>>
> Just wondering why you didn't add that test in r26813? If you don't have
> time, why not add an XFail-ing place holder explaining in comments the
> above scenario, then we can still add it later.
>
> Lieven
>
>
Lieven,
Yes I *love* tests that shows detects all the problems in my code well
in advance(This is what exactly I feel when I change something in
merge-tracking code). But writing a testcase for short lived bugs of
this nature would be an overkill.

With regards
Kamesh Jayachandran

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Sep 28 09:43:31 2007

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