Re: Input validation observations
On Fri, 2010-12-03, Julian Foad wrote:
> On Fri, 2010-12-03 at 19:15 +0530, Noorul Islam K M wrote:
> > Julian Foad <julian.foad_at_wandisco.com> writes:
> > > Noorul Islam K M wrote:
> > >> Julian Foad <julian.foad_at_wandisco.com> writes:
> > >> > * "svn mkdir ^/ a" -> "Illegal repository URL 'a'"; should say "can't
> > >> > mix URL and local targets"?
> > > [...]
> > > Does this validation make the check at the beginning of mkdir_urls()
> > > redundant? If so, we should get rid of it.
> > Attached is the modified patch.
> Committed in r1041900.
EXPECTED STDERR (regexp):
.*Illegal repository URL 'subdir'
svn: Try 'svn help' for more info
svn: Cannot mix repository and working copy targets
FAIL: basic_tests.py 49: mkdir mix url and local path should error
I'll fix it.
Received on 2010-12-03 17:54:02 CET
This is an archived mail posted to the Subversion Dev