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.
-- Ivan Zhakov
