On Fri, Oct 24, 2014 at 10:00 AM, Ivan Zhakov <ivan_at_visualsvn.com> wrote:
> On 24 October 2014 02:11, <stefan2_at_apache.org> wrote:
> > Author: stefan2
> > Date: Thu Oct 23 22:11:17 2014
> > New Revision: 1633967
> >
> > URL: http://svn.apache.org/r1633967
> > Log:
> > Extend the test "txn list" list test to still return the correct list
> > after some txn got aborted. Covers svn_fs_purge_txn as well now.
> >
> > * subversion/tests/libsvn_fs/fs-test.c
> > (verify_txn_list): Purge txns one-by-one and verify txn list after
> > each step.
> >
> >
> > Modified: subversion/trunk/subversion/tests/libsvn_fs/fs-test.c
> > URL:
> http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/libsvn_fs/fs-test.c?rev=1633967&r1=1633966&r2=1633967&view=diff
> >
> ==============================================================================
> > --- subversion/trunk/subversion/tests/libsvn_fs/fs-test.c (original)
> > +++ subversion/trunk/subversion/tests/libsvn_fs/fs-test.c Thu Oct 23
> 22:11:17 2014
> > @@ -319,6 +319,25 @@ verify_txn_list(const svn_test_opts_t *o
> > "Got a bogus txn list.");
> > all_good:
> >
> > + /* Get rid of the txns one at a time. */
> > + SVN_ERR(svn_fs_purge_txn(fs, name1, pool));
> > +
> > + /* There should be exactly one left. */
> > + SVN_ERR(svn_fs_list_transactions(&txn_list, fs, pool));
> > +
> > + /* Check the list. It should have *exactly* two entries. */
> It seems comment doesn't match the code.
>
Thanks for pointing out! Fixed in r1634097.
-- Stefan^2.
Received on 2014-10-24 19:11:48 CEST