Re: [PATCH] test/emacs: mark one test as fixed in newer emacs.

Subject: Re: [PATCH] test/emacs: mark one test as fixed in newer emacs.

Date: Mon, 06 Dec 2021 19:38:15 +0200

To: David Bremner, notmuch@notmuchmail.org

Cc:

From: Tomi Ollila


On Sun, Dec 05 2021, David Bremner wrote:

> The remaining problem in this test is fixed upstream in Emacs
> 28. While most people are using earlier versions of emacs, the test
> still provides some documentation of a known bug.
> ---
>  test/T450-emacs-show.sh | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/test/T450-emacs-show.sh b/test/T450-emacs-show.sh
> index 4b5f5fde..2a192ed0 100755
> --- a/test/T450-emacs-show.sh
> +++ b/test/T450-emacs-show.sh
> @@ -220,7 +220,9 @@ test_emacs '(notmuch-show "id:basic-encrypted@crypto.notmuchmail.org")
>  test_expect_equal_file $EXPECTED/notmuch-show-decrypted-message OUTPUT
>  
>  test_begin_subtest "show encrypted rfc822 message"
> -test_subtest_known_broken
> +if ${TEST_EMACS} --quick --batch --eval '(if (version< emacs-version
>  "28") (kill-emacs 0) (kill-emacs 1)))'; then

looks good but perhaps "drier" version:

    (kill-emacs (if (version< emacs-version "28") 0 1))

Tomi

> +    test_subtest_known_broken
> +fi
>  test_emacs '(notmuch-show "id:encrypted-rfc822-attachment@crypto.notmuchmail.org")
>              (test-visible-output)'
>  test_expect_code 1 'fgrep "!!!" OUTPUT'
> -- 
> 2.33.0
_______________________________________________
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-leave@notmuchmail.org

Thread: