[ditrack commit] r2343 - src/trunk/tests/cmd-new/auto-commit.tc

Vlad Skvortsov vss at 73rus.com
Fri Jan 18 17:11:30 PST 2008


Ivan Glushkov wrote:
> On Tue, 27 Nov 2007 10:43:36 +0300, Vlad Skvortsov <vss at 73rus.com> wrote:
>
>   
>> gli at ditrack.org wrote:
>>     
>>> Author: gli
>>> Date: 2007-11-12 08:31:01 -0800 (Mon, 12 Nov 2007)
>>> New Revision: 2343
>>>
>>> Removed:
>>>    src/trunk/tests/cmd-new/auto-commit.tc/commit.expected
>>>    src/trunk/tests/cmd-new/auto-commit.tc/log.expected
>>>    src/trunk/tests/cmd-new/auto-commit.tc/new.expected.wc
>>>
>>>       
>> Why don't we check the output of the 'new' command in a test case
>> related to that?
>>     
>
> We do.
> 1. We check that output of the new for the case when local = True
>   

Ok.

> 2. For the case "local=False" we check the last string of output to
> find out the number of new assigned id.
>   

I think it's insufficient. It would be okay in other testcases, but 
since here we specifically test the 'new' command, I believe that we 
need to be very thorough about that.

-- 
Vlad Skvortsov, vss at 73rus.com, http://vss.73rus.com



More information about the Dev mailing list