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

Re: svn commit: r1068411 - /subversion/trunk/subversion/tests/cmdline/authz_tests.py

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Fri, 11 Feb 2011 20:10:17 +0200

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_at_collab.net> wrote:
> >>kameshj_at_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)
Received on 2011-02-11 19:15:21 CET

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.