Re: svn commit: rev 5534 - trunk/subversion/tests/clients/cmdline

> brane@xbc.nu writes:
> kraai@tigris.org wrote:
>
> >Author: kraai
> >Date: Wed Apr 2 17:14:54 2003
> >New Revision: 5534
> >
> >Modified:
> > trunk/subversion/tests/clients/cmdline/diff_tests.py
> >Log:
> >* subversion/tests/clients/cmdline/diff_tests.py
> > (diff_nonextant_urls): Check all output for error message.
> >
> >
> >Modified: trunk/subversion/tests/clients/cmdline/diff_tests.py
> >============================================================================
> ==
> >--- trunk/subversion/tests/clients/cmdline/diff_tests.py (original)
> >+++ trunk/subversion/tests/clients/cmdline/diff_tests.py Wed Apr 2 17:1
> 4:54 2003
> >@@ -818,12 +818,18 @@
> >
> > diff_output, err_output = svntest.main.run_svn(None, 'diff', non_extant_u
> rl,
> > extant_url)
> >- if not re.match('svn: Filesystem has no item$', err_output[0]):
> >+ for line in err_output:
> >+ if re.match('svn: Filesystem has no item$', line):
> >+ break
> >+ else:
> > raise svntest.Failure
> >
> > diff_output, err_output = svntest.main.run_svn(None, 'diff', extant_url,
> > non_extant_url)
> >- if not re.match('svn: Filesystem has no item$', err_output[0]):
> >+ for line in err_output:
> >+ if re.match('svn: Filesystem has no item$', line):
> >+ break
> >+ else:
> > raise svntest.Failure
> >
> >
>
> We now have at least 4 such checks in the tests that I know of. Wouldn't
> it be time for abstracting them into a function in svntest.actions?

Or maybe run_and_verify_svn should accept a string and a regex for
expected_stdin/expected_stdout in addition to a list and SVNAnyOutput?

-Fitz

--
Brian W. Fitzpatrick    <fitz_at_red-bean.com>   http://www.red-bean.com/fitz/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org