diff options
| author | David Bremner <david@tethera.net> | 2018-08-30 08:29:11 -0300 |
|---|---|---|
| committer | David Bremner <david@tethera.net> | 2018-09-06 08:07:13 -0300 |
| commit | ea08032ae40119b24ef22c1c30aaf631cda02dd0 (patch) | |
| tree | c7966451c7dc0ffbccd2f772acfb60a51de30419 | |
| parent | 46dce33abc82ea6ebd3be2e0887506af4185c739 (diff) | |
test: add known broken test for good In-Reply-To / bad References
The current scheme of choosing the replyto (i.e. the default parent
for threading purposes) does not work well for mailers that put
the oldest Reference last.
| -rwxr-xr-x | test/T510-thread-replies.sh | 14 | ||||
| -rw-r--r-- | test/corpora/threading/parent-priority/cur/child | 11 | ||||
| -rw-r--r-- | test/corpora/threading/parent-priority/cur/grand-child | 10 | ||||
| -rw-r--r-- | test/corpora/threading/parent-priority/cur/root | 7 |
4 files changed, 42 insertions, 0 deletions
diff --git a/test/T510-thread-replies.sh b/test/T510-thread-replies.sh index 915e68ef..8ff2ade9 100755 --- a/test/T510-thread-replies.sh +++ b/test/T510-thread-replies.sh @@ -209,4 +209,18 @@ End of search results. EOF test_expect_equal_file EXPECTED OUTPUT +test_begin_subtest "trusting reply-to (tree view)" +test_subtest_known_broken +test_emacs '(notmuch-tree "id:B00-root@example.org") + (notmuch-test-wait) + (test-output) + (delete-other-windows)' +cat <<EOF > EXPECTED + 2016-06-17 Alice ┬►root message (inbox unread) + 2016-06-18 Alice ╰┬►child message (inbox unread) + 2016-06-18 Alice ╰─►grand-child message (inbox unread) +End of search results. +EOF +test_expect_equal_file EXPECTED OUTPUT + test_done diff --git a/test/corpora/threading/parent-priority/cur/child b/test/corpora/threading/parent-priority/cur/child new file mode 100644 index 00000000..23ee6495 --- /dev/null +++ b/test/corpora/threading/parent-priority/cur/child @@ -0,0 +1,11 @@ +From: Alice <alice@example.org> +To: Daniel <daniel@example.org> +Subject: child message +Message-ID: <B01-child@example.org> +In-Reply-To: <B00-root@example.org> +References: <B00--root@example.org> +Date: Fri, 17 Jun 2016 22:14:41 -0400 + +This is a normal-ish reply, and has both a references header and an +in-reply-to header. + diff --git a/test/corpora/threading/parent-priority/cur/grand-child b/test/corpora/threading/parent-priority/cur/grand-child new file mode 100644 index 00000000..028371d4 --- /dev/null +++ b/test/corpora/threading/parent-priority/cur/grand-child @@ -0,0 +1,10 @@ +From: Alice <alice@example.org> +To: Daniel <daniel@example.org> +Subject: grand-child message +Message-ID: <B01-grand-child@example.org> +In-Reply-To: <B01-child@example.org> +References: <B01-child@example.org> <B00-root@example.org> +Date: Fri, 17 Jun 2016 22:24:41 -0400 + +This has the references headers in the wrong order, with oldest first. +Debbugs does this. diff --git a/test/corpora/threading/parent-priority/cur/root b/test/corpora/threading/parent-priority/cur/root new file mode 100644 index 00000000..3990843d --- /dev/null +++ b/test/corpora/threading/parent-priority/cur/root @@ -0,0 +1,7 @@ +From: Alice <alice@example.org> +To: Daniel <daniel@example.org> +Subject: root message +Message-ID: <B00-root@example.org> +Date: Thu, 16 Jun 2016 22:14:41 -0400 + +This message has no reply-to |