Kamesh Jayachandran <kamesh_at_collab.net> writes:
> On 02/17/2011 08:16 PM, Noorul Islam K M wrote:
>
>> Noorul Islam K M<noorul_at_collab.net> writes:
>>
>>> Log
>>> [[[
>>>
>>> Fix failing expected error regex.
>>>
>>> * subversion/tests/cmdline/blame_tests.py
>>> (blame_non_existent_url_target): Relax regex to allow errors from
>>> http: and svn: protocols.
>>>
>>> Patch by: Noorul Islam K M<noorul{_AT_}collab.net>
>>> ]]]
>>>
>>> Index: subversion/tests/cmdline/blame_tests.py
>>> ===================================================================
>>> --- subversion/tests/cmdline/blame_tests.py (revision 1071613)
>>> +++ subversion/tests/cmdline/blame_tests.py (working copy)
>>> @@ -759,8 +759,7 @@
>>> " 2 jrandom New contents for iota\n",
>>> ]
>>>
>>> - expected_err = "svn: warning: W160017: '/non-existent' " + \
>>> - "is not a file in revision 2\n" + \
>>> + expected_err = "svn: warning: W160017: .*\n" + \
>>> ".*\nsvn: E200009: Could not perform blame on all targets " + \
>>> "because some targets don't exist\n"
>
> Do we need full error message in the expected error?
>
I will be improving these test cases when I combine them to form one
later.
Thanks and Regards
Noorul
> With regards
> Kamesh Jayachandran
>>> expected_err_re = re.compile(expected_err)
>> Stefen,
>>
>> Please find attached modified patch.
>>
>> Log
>> [[[
>>
>> Fix failing expected error regex. Also capture ra_neon error.
>>
>> * subversion/svn/blame-cmd.c
>> (svn_cl__blame): Catch SVN_ERR_FS_NOT_FOUND and display warning.
>>
>> * subversion/tests/cmdline/blame_tests.py
>> (blame_non_existent_url_target): Relax regex to allow errors from
>> http: and svn: protocols.
>>
>> Patch by: Noorul Islam K M<noorul{_AT_}collab.net>
>> ]]]
>>
Received on 2011-02-17 16:21:49 CET