On 5/24/06, Madan U Sreenivasan <madan@collab.net> wrote:
> On Wed, 24 May 2006 14:46:42 +0530, Peter N. Lundblad
> <peter@famlundblad.se> wrote:
>
> > Madan U Sreenivasan writes:
> [snip]
> > > + lock_info = output[-6:-1]
> >
> > While here, we could as well make this not break if we add more fields
> > to the info output in the future. I think we should search for the
> > specific
> > fields, which is much easier with this refactorization.
>
> Aye! Aye! captain!, pl. find the patch attached. :)
>
> [[[
> Factorize out code for locking a path and validating
> the contents of the lock.
>
> Suggested by: djames
> Review by: lundblad
>
> * subversion/tests/cmdline/svntest/actions.py
> (run_and_validate_lock): New function to lock a path, and
> validate the contents of the lock.
>
> * subversion/tests/cmdline/lock_tests.py
> (examine_lock, examine_lock_via_url, examine_lock_encoded_recurse):
> Modified to use svntest.actions.run_and_validate_lock().
> ]]]
Committed in r20086. Thanks,
-garrett
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Jun 13 21:40:59 2006