Mailing-List: contact dev-help@subversion.apache.org; run by ezmlm
Precedence: bulk
Received-SPF: pass (nike.apache.org: domain of ivan@visualsvn.com designates
 209.85.213.47 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <20141023221117.EEBFB23889D5@eris.apache.org>
References: <20141023221117.EEBFB23889D5@eris.apache.org>
From: Ivan Zhakov <ivan@visualsvn.com>
Date: Fri, 24 Oct 2014 12:00:02 +0400
Message-ID: 
 <CABw-3Yc-cXeZyq4aSS6voFi89stg_Qp2g_XFsViN3uwxLLJKvg@mail.gmail.com>
Subject: Re: svn commit: r1633967 -
 /subversion/trunk/subversion/tests/libsvn_fs/fs-test.c
To: Subversion Development <dev@subversion.apache.org>,
 Stefan Fuhrman <stefan2@apache.org>
Content-Type: text/plain; charset=UTF-8

On 24 October 2014 02:11,  <stefan2@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