Re: svn commit: r25357 - in trunk/subversion/tests/cmdline: . svntest

On Wed, 13 Jun 2007, David Glasser wrote:

> On 6/13/07, Daniel Rall <dlr@collab.net> wrote:
> >On Tue, 12 Jun 2007, David Glasser wrote:
> >
> >> On 6/12/07, Kamesh Jayachandran <kamesh@collab.net> wrote:
> >> >I originally thought on the same lines. But I could not pass a negating
> >> >expression as a lambda function.
> >>
> >> SkipUnless?
> >
> >Ah, good suggestion. How about this patch (attached), which adds and
> >uses the new predicate?
>
> Looks good to me (though maybe the predicates in prop_tests and
> commit_tests want to move into svntest.main too?).

I committed the patch in r25406.

I thought of moving the other predicates, but wonder at the value if
they're only used in that one source file (ATM). Do you think it's
worth moving them into the test framework itself doing so to avoid the
chance of redundant implementations springing up at a later date?