Re: [PATCH 07/13] test: add external prereqs to many emacs tests

Subject: Re: [PATCH 07/13] test: add external prereqs to many emacs tests

Date: Sat, 01 May 2021 23:19:25 +0300

To: Felipe Contreras, notmuch@notmuchmail.org

Cc: Daniel Kahn Gillmor

From: Tomi Ollila


On Sat, May 01 2021, Felipe Contreras wrote:

> The tests fail otherwise.

LGTM.

Tomi


>
> Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
> ---
>  test/T310-emacs.sh            | 1 +
>  test/T350-crypto.sh           | 1 +
>  test/T355-smime.sh            | 1 +
>  test/T357-index-decryption.sh | 1 +
>  test/T450-emacs-show.sh       | 1 +
>  test/T460-emacs-tree.sh       | 1 +
>  test/T730-emacs-forwarding.sh | 2 ++
>  test/test-lib.sh              | 6 ++++++
>  8 files changed, 14 insertions(+)
>
> diff --git a/test/T310-emacs.sh b/test/T310-emacs.sh
> index 78ac19a8..e64627c6 100755
> --- a/test/T310-emacs.sh
> +++ b/test/T310-emacs.sh
> @@ -5,6 +5,7 @@ test_description="emacs interface"
>  
>  EXPECTED=$NOTMUCH_SRCDIR/test/emacs.expected-output
>  
> +test_require_emacs
>  add_email_corpus
>  
>  # syntax errors in test-lib.el cause mysterious failures
> diff --git a/test/T350-crypto.sh b/test/T350-crypto.sh
> index 0aada4df..ae1d6a98 100755
> --- a/test/T350-crypto.sh
> +++ b/test/T350-crypto.sh
> @@ -9,6 +9,7 @@ test_description='PGP/MIME signature verification and decryption'
>  
>  ##################################################
>  
> +test_require_emacs
>  add_gnupg_home
>  
>  test_begin_subtest "emacs delivery of signed message"
> diff --git a/test/T355-smime.sh b/test/T355-smime.sh
> index 8b2b52be..12ac2525 100755
> --- a/test/T355-smime.sh
> +++ b/test/T355-smime.sh
> @@ -3,6 +3,7 @@
>  test_description='S/MIME signature verification and decryption'
>  . $(dirname "$0")/test-lib.sh || exit 1
>  
> +test_require_emacs
>  test_require_external_prereq openssl
>  test_require_external_prereq gpgsm
>  
> diff --git a/test/T357-index-decryption.sh b/test/T357-index-decryption.sh
> index 1ed5f28c..b81bdfe1 100755
> --- a/test/T357-index-decryption.sh
> +++ b/test/T357-index-decryption.sh
> @@ -7,6 +7,7 @@ test_description='indexing decrypted mail'
>  
>  ##################################################
>  
> +test_require_emacs
>  add_gnupg_home
>  
>  # create a test encrypted message
> diff --git a/test/T450-emacs-show.sh b/test/T450-emacs-show.sh
> index cca56ca3..bd76d378 100755
> --- a/test/T450-emacs-show.sh
> +++ b/test/T450-emacs-show.sh
> @@ -5,6 +5,7 @@ test_description="emacs notmuch-show view"
>  
>  EXPECTED=$NOTMUCH_SRCDIR/test/emacs-show.expected-output
>  
> +test_require_emacs
>  add_email_corpus
>  
>  test_begin_subtest "Hiding Original Message region at beginning of a message"
> diff --git a/test/T460-emacs-tree.sh b/test/T460-emacs-tree.sh
> index cb2c90b8..195485c1 100755
> --- a/test/T460-emacs-tree.sh
> +++ b/test/T460-emacs-tree.sh
> @@ -5,6 +5,7 @@ test_description="emacs tree view interface"
>  
>  EXPECTED=$NOTMUCH_SRCDIR/test/emacs-tree.expected-output
>  
> +test_require_emacs
>  add_email_corpus
>  
>  test_begin_subtest "Basic notmuch-tree view in emacs"
> diff --git a/test/T730-emacs-forwarding.sh b/test/T730-emacs-forwarding.sh
> index 45e61568..5d6ac9f0 100755
> --- a/test/T730-emacs-forwarding.sh
> +++ b/test/T730-emacs-forwarding.sh
> @@ -3,6 +3,8 @@
>  test_description="emacs forwarding"
>  . $(dirname "$0")/test-lib.sh || exit 1
>  
> +test_require_emacs
> +
>  test_begin_subtest "Forward setting the correct references header"
>  # Check that, when forwarding a message, the new message has
>  # a References-header pointing to the original (forwarded) message.
> diff --git a/test/test-lib.sh b/test/test-lib.sh
> index 72ac2e89..88e2a82f 100644
> --- a/test/test-lib.sh
> +++ b/test/test-lib.sh
> @@ -114,6 +114,12 @@ unset ALTERNATE_EDITOR
>  unset EMAIL
>  unset NAME
>  
> +test_require_emacs () {
> +    test_require_external_prereq emacs
> +    test_require_external_prereq ${TEST_EMACSCLIENT}
> +    test_require_external_prereq dtach
> +}
> +
>  add_gnupg_home ()
>  {
>      [ -e "${GNUPGHOME}/gpg.conf" ] && return
> -- 
> 2.31.0
> _______________________________________________
> notmuch mailing list -- notmuch@notmuchmail.org
> To unsubscribe send an email to notmuch-leave@notmuchmail.org
_______________________________________________
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-leave@notmuchmail.org

Thread: