Re: [PATCH] test: unbreak performance tests
David Bremnerwrites: > In 8e7fb88237aedea22 Jani replaced the use of $(pwd -P) to find the > TEST_DIRECTORY in order to better support out of tree > builds. Unfortunately the performance-tests need a different value for > the variable and were thus broken. > > This commit splits out the setting of this variable for the two sets > of tests. Performance tests still don't work out of tree, because > the handling of the downloaded corpus needs to be updated. pushed, d ___ notmuch mailing list notmuch@notmuchmail.org https://notmuchmail.org/mailman/listinfo/notmuch
Re: [PATCH] test: unbreak performance tests
On Sat, Dec 02 2017, David Bremner wrote: > In 8e7fb88237aedea22 Jani replaced the use of $(pwd -P) to find the > TEST_DIRECTORY in order to better support out of tree > builds. Unfortunately the performance-tests need a different value for > the variable and were thus broken. > > This commit splits out the setting of this variable for the two sets > of tests. Performance tests still don't work out of tree, because > the handling of the downloaded corpus needs to be updated. Change fine by me. Tomi > --- > performance-test/perf-test-lib.sh | 3 +++ > test/test-lib-common.sh | 4 > test/test-lib.sh | 3 +++ > 3 files changed, 6 insertions(+), 4 deletions(-) > > diff --git a/performance-test/perf-test-lib.sh > b/performance-test/perf-test-lib.sh > index 56538abd..b70288cc 100644 > --- a/performance-test/perf-test-lib.sh > +++ b/performance-test/perf-test-lib.sh > @@ -29,6 +29,9 @@ done > # Ensure NOTMUCH_SRCDIR and NOTMUCH_BUILDDIR are set. > . $(dirname "$0")/../test/export-dirs.sh || exit 1 > > +# Where to run the tests > +TEST_DIRECTORY=$NOTMUCH_BUILDDIR/performance-test > + > . "$NOTMUCH_SRCDIR/test/test-lib-common.sh" || exit 1 > > set -e > diff --git a/test/test-lib-common.sh b/test/test-lib-common.sh > index 6c3571d4..2f7950ac 100644 > --- a/test/test-lib-common.sh > +++ b/test/test-lib-common.sh > @@ -41,10 +41,6 @@ restore_database () { > cp -pR $TMP_DIRECTORY/notmuch-dir-backup."${test_name}" > ${MAIL_DIR}/.notmuch > } > > -# Test the binaries we have just built. The tests are kept in > -# test/ subdirectory and are run in 'trash directory' subdirectory. > -TEST_DIRECTORY=$NOTMUCH_BUILDDIR/test > - > # Prepend $TEST_DIRECTORY/../lib to LD_LIBRARY_PATH, to make tests work > # on systems where ../notmuch depends on LD_LIBRARY_PATH. > LD_LIBRARY_PATH=${TEST_DIRECTORY%/*}/lib${LD_LIBRARY_PATH:+:$LD_LIBRARY_PATH} > diff --git a/test/test-lib.sh b/test/test-lib.sh > index 42a45f15..4c8607a1 100644 > --- a/test/test-lib.sh > +++ b/test/test-lib.sh > @@ -1066,6 +1066,9 @@ test_init_ () { > } > > > +# Where to run the tests > +TEST_DIRECTORY=$NOTMUCH_BUILDDIR/test > + > . "$NOTMUCH_SRCDIR/test/test-lib-common.sh" || exit 1 > > if [ "${NOTMUCH_GMIME_MAJOR}" = 3 ]; then > -- > 2.15.0 ___ notmuch mailing list notmuch@notmuchmail.org https://notmuchmail.org/mailman/listinfo/notmuch
[PATCH] test: unbreak performance tests
In 8e7fb88237aedea22 Jani replaced the use of $(pwd -P) to find the TEST_DIRECTORY in order to better support out of tree builds. Unfortunately the performance-tests need a different value for the variable and were thus broken. This commit splits out the setting of this variable for the two sets of tests. Performance tests still don't work out of tree, because the handling of the downloaded corpus needs to be updated. --- performance-test/perf-test-lib.sh | 3 +++ test/test-lib-common.sh | 4 test/test-lib.sh | 3 +++ 3 files changed, 6 insertions(+), 4 deletions(-) diff --git a/performance-test/perf-test-lib.sh b/performance-test/perf-test-lib.sh index 56538abd..b70288cc 100644 --- a/performance-test/perf-test-lib.sh +++ b/performance-test/perf-test-lib.sh @@ -29,6 +29,9 @@ done # Ensure NOTMUCH_SRCDIR and NOTMUCH_BUILDDIR are set. . $(dirname "$0")/../test/export-dirs.sh || exit 1 +# Where to run the tests +TEST_DIRECTORY=$NOTMUCH_BUILDDIR/performance-test + . "$NOTMUCH_SRCDIR/test/test-lib-common.sh" || exit 1 set -e diff --git a/test/test-lib-common.sh b/test/test-lib-common.sh index 6c3571d4..2f7950ac 100644 --- a/test/test-lib-common.sh +++ b/test/test-lib-common.sh @@ -41,10 +41,6 @@ restore_database () { cp -pR $TMP_DIRECTORY/notmuch-dir-backup."${test_name}" ${MAIL_DIR}/.notmuch } -# Test the binaries we have just built. The tests are kept in -# test/ subdirectory and are run in 'trash directory' subdirectory. -TEST_DIRECTORY=$NOTMUCH_BUILDDIR/test - # Prepend $TEST_DIRECTORY/../lib to LD_LIBRARY_PATH, to make tests work # on systems where ../notmuch depends on LD_LIBRARY_PATH. LD_LIBRARY_PATH=${TEST_DIRECTORY%/*}/lib${LD_LIBRARY_PATH:+:$LD_LIBRARY_PATH} diff --git a/test/test-lib.sh b/test/test-lib.sh index 42a45f15..4c8607a1 100644 --- a/test/test-lib.sh +++ b/test/test-lib.sh @@ -1066,6 +1066,9 @@ test_init_ () { } +# Where to run the tests +TEST_DIRECTORY=$NOTMUCH_BUILDDIR/test + . "$NOTMUCH_SRCDIR/test/test-lib-common.sh" || exit 1 if [ "${NOTMUCH_GMIME_MAJOR}" = 3 ]; then -- 2.15.0 ___ notmuch mailing list notmuch@notmuchmail.org https://notmuchmail.org/mailman/listinfo/notmuch