[PATCH] test: fix test database backup/restore location
Daniel Kahn Gillmor
dkg at fifthhorseman.net
Thu Oct 26 14:17:42 PDT 2017
On Tue 2017-10-24 21:36:11 +0300, Jani Nikula wrote:
> backup_database() and restore_database() used to store the backups in
> the test specific temporary directory, through the current working
> directory being there. Commit 8e7fb88237ae ("test: use source and
> build paths in test-lib-common.sh") started using a test specific
> backup directories under the build tree test directory. This was in
> error. Switch back to the old location, but using paths to the
> location instead of relying on current working directory.
>
> Reported by Daniel Kahn Gillmor <dkg at fifthhorseman.net>.
> ---
> test/test-lib-common.sh | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/test/test-lib-common.sh b/test/test-lib-common.sh
> index 4300eb65418f..6c3571d4c560 100644
> --- a/test/test-lib-common.sh
> +++ b/test/test-lib-common.sh
> @@ -31,14 +31,14 @@ fi
>
> backup_database () {
> test_name=$(basename $0 .sh)
> - rm -rf $NOTMUCH_BUILDDIR/test/notmuch-dir-backup."$test_name"
> - cp -pR ${MAIL_DIR}/.notmuch $NOTMUCH_BUILDDIR/test/notmuch-dir-backup."${test_name}"
> + rm -rf $TMP_DIRECTORY/notmuch-dir-backup."$test_name"
> + cp -pR ${MAIL_DIR}/.notmuch $TMP_DIRECTORY/notmuch-dir-backup."${test_name}"
> }
>
> restore_database () {
> test_name=$(basename $0 .sh)
> rm -rf ${MAIL_DIR}/.notmuch
> - cp -pR $NOTMUCH_BUILDDIR/test/notmuch-dir-backup."${test_name}" ${MAIL_DIR}/.notmuch
> + cp -pR $TMP_DIRECTORY/notmuch-dir-backup."${test_name}" ${MAIL_DIR}/.notmuch
> }
>
> # Test the binaries we have just built. The tests are kept in
Thanks for this, Jani.
i can confirm that with this patch applied, i no longer see the extra
directories lingering after a run of the test suite. This solved my
problem. I recommend merging :)
--dkg
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 832 bytes
Desc: not available
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20171026/1ce1dd78/attachment.sig>
More information about the notmuch
mailing list