On Fri, Oct 24, 2014 at 10:00 AM, Ivan Zhakov <i...@visualsvn.com> wrote:
> On 24 October 2014 02:11, <stef...@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.
