unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
* [PATCH] test: add test for syntax of emacs test library
@ 2014-01-12  0:10 David Bremner
  2014-01-12  9:56 ` Tomi Ollila
  2014-01-13 18:37 ` David Bremner
  0 siblings, 2 replies; 3+ messages in thread
From: David Bremner @ 2014-01-12  0:10 UTC (permalink / raw)
  To: notmuch

If there is a syntax error in the emacs test library, it causes other
tests to hang or crash without a useful error message.

This test could be eliminated if the error reporting for emacs tests
was somehow improved.
---
 test/emacs | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/test/emacs b/test/emacs
index 863219d..7dc9a8e 100755
--- a/test/emacs
+++ b/test/emacs
@@ -7,6 +7,10 @@ EXPECTED=$TEST_DIRECTORY/emacs.expected-output
 
 add_email_corpus
 
+# syntax errors in test-lib.el cause mysterious failures
+test_expect_success 'Syntax of emacs test library' \
+    "${TEST_EMACS} -Q --batch --load $TEST_DIRECTORY/test-lib.el"
+
 test_begin_subtest "Basic notmuch-hello view in emacs"
 test_emacs '(notmuch-hello)
 	    (test-output)'
-- 
1.8.5.2

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] test: add test for syntax of emacs test library
  2014-01-12  0:10 [PATCH] test: add test for syntax of emacs test library David Bremner
@ 2014-01-12  9:56 ` Tomi Ollila
  2014-01-13 18:37 ` David Bremner
  1 sibling, 0 replies; 3+ messages in thread
From: Tomi Ollila @ 2014-01-12  9:56 UTC (permalink / raw)
  To: David Bremner, notmuch

On Sun, Jan 12 2014, David Bremner <david@tethera.net> wrote:

> If there is a syntax error in the emacs test library, it causes other
> tests to hang or crash without a useful error message.
>
> This test could be eliminated if the error reporting for emacs tests
> was somehow improved.
> ---

This helps developers to deal with some mysteries in the test suite so LGTM.

Tomi


>  test/emacs | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/test/emacs b/test/emacs
> index 863219d..7dc9a8e 100755
> --- a/test/emacs
> +++ b/test/emacs
> @@ -7,6 +7,10 @@ EXPECTED=$TEST_DIRECTORY/emacs.expected-output
>  
>  add_email_corpus
>  
> +# syntax errors in test-lib.el cause mysterious failures
> +test_expect_success 'Syntax of emacs test library' \
> +    "${TEST_EMACS} -Q --batch --load $TEST_DIRECTORY/test-lib.el"
> +
>  test_begin_subtest "Basic notmuch-hello view in emacs"
>  test_emacs '(notmuch-hello)
>  	    (test-output)'
> -- 
> 1.8.5.2

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] test: add test for syntax of emacs test library
  2014-01-12  0:10 [PATCH] test: add test for syntax of emacs test library David Bremner
  2014-01-12  9:56 ` Tomi Ollila
@ 2014-01-13 18:37 ` David Bremner
  1 sibling, 0 replies; 3+ messages in thread
From: David Bremner @ 2014-01-13 18:37 UTC (permalink / raw)
  To: notmuch

David Bremner <david@tethera.net> writes:

> If there is a syntax error in the emacs test library, it causes other
> tests to hang or crash without a useful error message.
>
> This test could be eliminated if the error reporting for emacs tests
> was somehow improved.

pushed, and git even figured out what file to patch despite Tomi's best
efforts ;).

d

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2014-01-13 18:37 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-01-12  0:10 [PATCH] test: add test for syntax of emacs test library David Bremner
2014-01-12  9:56 ` Tomi Ollila
2014-01-13 18:37 ` David Bremner

Code repositories for project(s) associated with this public inbox

	https://yhetil.org/notmuch.git/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).