subversion-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Daniel Shahaf <...@daniel.shahaf.name>
Subject Re: svn commit: r1068411 - /subversion/trunk/subversion/tests/cmdline/authz_tests.py
Date Fri, 11 Feb 2011 18:10:17 GMT
Hyrum K Wright wrote on Fri, Feb 11, 2011 at 18:09:14 +0000:
> On Fri, Feb 11, 2011 at 5:59 PM, Kamesh Jayachandran <kamesh@collab.net> wrote:
> >>kameshj@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.

Thanks.  But why remove the E170001?  Its very purpose is to provide
a machine-readable identification of the error... (exactly what tests
need)

Mime
View raw message