[PATCH v2] test: Add test for messages with missing headers
Tomi Ollila
tomi.ollila at iki.fi
Wed Aug 8 01:08:30 PDT 2012
On Wed, Aug 08 2012, Austin Clements <amdragon at MIT.EDU> wrote:
> Currently the JSON tests for search and show are broken because
> notmuch attempts to dereference a NULL pointer.
> ---
> This version fixes the "Show: text" test so that it sanitize its
> output and doesn't hard-code my test paths.
+1
Tomi
w/ python json(.tool) the original order cannot be preserved as the parser
stores content into dictionary -- without sorting those came in some
internal python order... the following code could be used to use less
indentation, though:
python -c 'import sys,json; j = json.load(sys.stdin);
json.dump(j, sys.stdout, sort_keys=True, indent=2)' < input_file
The other "problem" with json.tool is that it converts non-ascii chars
to \uNNNN values :/.
What we could do is to dig a simple c json formatter -- someday in the
future, maybe -- but for *now* this is the best we can have :D
>
> test/missing-headers | 162 ++++++++++++++++++++++++++++++++++++++++++++++++++
> test/notmuch-test | 1 +
> 2 files changed, 163 insertions(+)
> create mode 100755 test/missing-headers
>
> diff --git a/test/missing-headers b/test/missing-headers
> new file mode 100755
> index 0000000..e79f922
> --- /dev/null
> +++ b/test/missing-headers
> @@ -0,0 +1,162 @@
> +#!/usr/bin/env bash
> +test_description='messages with missing headers'
> +. ./test-lib.sh
> +
> +# Notmuch requires at least one of from, subject, or to or it will
> +# ignore the file. Generate two messages so that together they cover
> +# all possible missing headers. We also give one of the messages a
> +# date to ensure stable result ordering.
> +
> +cat <<EOF > "${MAIL_DIR}/msg-2"
> +To: Notmuch Test Suite <test_suite at notmuchmail.org>
> +Date: Fri, 05 Jan 2001 15:43:57 +0000
> +
> +Body
> +EOF
> +
> +cat <<EOF > "${MAIL_DIR}/msg-1"
> +From: Notmuch Test Suite <test_suite at notmuchmail.org>
> +
> +Body
> +EOF
> +
> +NOTMUCH_NEW
> +
> +test_begin_subtest "Search: text"
> +output=$(notmuch search '*' | notmuch_search_sanitize)
> +test_expect_equal "$output" "\
> +thread:XXX 2001-01-05 [1/1] (null); (inbox unread)
> +thread:XXX 1970-01-01 [1/1] Notmuch Test Suite; (inbox unread)"
> +
> +test_begin_subtest "Search: json"
> +test_subtest_known_broken
> +output=$(notmuch search --format=json '*' | notmuch_search_sanitize)
> +test_expect_equal_json "$output" '
> +[
> + {
> + "authors": "",
> + "date_relative": "2001-01-05",
> + "matched": 1,
> + "subject": "",
> + "tags": [
> + "inbox",
> + "unread"
> + ],
> + "thread": "XXX",
> + "timestamp": 978709437,
> + "total": 1
> + },
> + {
> + "authors": "Notmuch Test Suite",
> + "date_relative": "1970-01-01",
> + "matched": 1,
> + "subject": "",
> + "tags": [
> + "inbox",
> + "unread"
> + ],
> + "thread": "XXX",
> + "timestamp": 0,
> + "total": 1
> + }
> +]'
> +
> +test_begin_subtest "Show: text"
> +output=$(notmuch show '*' | notmuch_show_sanitize)
> +test_expect_equal "$output" "\
> +message{ id:notmuch-sha1-7a6e4eac383ef958fcd3ebf2143db71b8ff01161 depth:0 match:1 excluded:0 filename:/XXX/mail/msg-2
> +header{
> + (2001-01-05) (inbox unread)
> +Subject: (null)
> +From: (null)
> +To: Notmuch Test Suite <test_suite at notmuchmail.org>
> +Date: Fri, 05 Jan 2001 15:43:57 +0000
> +header}
> +body{
> +part{ ID: 1, Content-type: text/plain
> +Body
> +part}
> +body}
> +message}
> +message{ id:notmuch-sha1-ca55943aff7a72baf2ab21fa74fab3d632401334 depth:0 match:1 excluded:0 filename:/XXX/mail/msg-1
> +header{
> +Notmuch Test Suite <test_suite at notmuchmail.org> (1970-01-01) (inbox unread)
> +Subject: (null)
> +From: Notmuch Test Suite <test_suite at notmuchmail.org>
> +Date: Thu, 01 Jan 1970 00:00:00 +0000
> +header}
> +body{
> +part{ ID: 1, Content-type: text/plain
> +Body
> +part}
> +body}
> +message}"
> +
> +test_begin_subtest "Show: json"
> +test_subtest_known_broken
> +output=$(notmuch show --format=json '*' | notmuch_json_show_sanitize)
> +test_expect_equal_json "$output" '
> +[
> + [
> + [
> + {
> + "body": [
> + {
> + "content": "Body\n",
> + "content-type": "text/plain",
> + "id": 1
> + }
> + ],
> + "date_relative": "2001-01-05",
> + "excluded": false,
> + "filename": "YYYYY",
> + "headers": {
> + "Date": "Fri, 05 Jan 2001 15:43:57 +0000",
> + "From": "",
> + "Subject": "",
> + "To": "Notmuch Test Suite <test_suite at notmuchmail.org>"
> + },
> + "id": "XXXXX",
> + "match": true,
> + "tags": [
> + "inbox",
> + "unread"
> + ],
> + "timestamp": 978709437
> + },
> + []
> + ]
> + ],
> + [
> + [
> + {
> + "body": [
> + {
> + "content": "Body\n",
> + "content-type": "text/plain",
> + "id": 1
> + }
> + ],
> + "date_relative": "1970-01-01",
> + "excluded": false,
> + "filename": "YYYYY",
> + "headers": {
> + "Date": "Thu, 01 Jan 1970 00:00:00 +0000",
> + "From": "Notmuch Test Suite <test_suite at notmuchmail.org>",
> + "Subject": ""
> + },
> + "id": "XXXXX",
> + "match": true,
> + "tags": [
> + "inbox",
> + "unread"
> + ],
> + "timestamp": 0
> + },
> + []
> + ]
> + ]
> +]'
> +
> +
> +test_done
> diff --git a/test/notmuch-test b/test/notmuch-test
> index ea39dfc..cc732c3 100755
> --- a/test/notmuch-test
> +++ b/test/notmuch-test
> @@ -59,6 +59,7 @@ TESTS="
> emacs-address-cleaning
> emacs-hello
> emacs-show
> + missing-headers
> "
> TESTS=${NOTMUCH_TESTS:=$TESTS}
>
> --
> 1.7.10
>
> _______________________________________________
> notmuch mailing list
> notmuch at notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch
More information about the notmuch
mailing list