On Fri, Feb 11, 2011 at 5:59 PM, Kamesh Jayachandran <kam...@collab.net> wrote:
>>kame...@apache.org wrote on Tue, Feb 08, 2011 at 14:25:06 -0000:
>>> +  expected_error_for_cat = "svn: E175013: Unable to connect to a
>>> repository"+ \
>>> +                            " at URL '" + mu_url + "'\n" + \
>>> +                            "svn: E175013: Access to '/" + mu_repo_path
>>> + \
>>> +                            "' forbidden\n" + \
>>> +                            "svn: E175009: XML parsing failed: (403
>>> Forbidden)"
>
>>Bad idea to hard code so much of the specific message we're generating
>>today; it's bound to change later.
>
>>(eg, the XML parsing thing would ideally go away at some point)
>
>>Just 'Access.*${mu_url}.*forbidden' is enough IMO.
>
>
> Thanks We fixed this in r1068802 Still we have "E170001:" for svnserve would
> remove it as soon as I reach before my working copy.

Note that instead of having to parse the error yourself, you can now
just give the error code for the expected error, and it should Do The
Right Thing.  (Error codes can be found in svntest.err)

-Hyrum

Reply via email to