diff options
| author | Jani Nikula <jani@nikula.org> | 2016-09-13 20:14:19 +0300 |
|---|---|---|
| committer | David Bremner <david@tethera.net> | 2016-09-17 09:11:07 -0300 |
| commit | 536b1f9df989915161024d2334e206f538d59f9b (patch) | |
| tree | 424fa9180030d3311d9250a83f7ca7a0b51b52bd /test/T220-reply.sh | |
| parent | ca82d481a1040ed3a47ad9c5e7cf7ec16e5e2ced (diff) | |
cli/reply: return internet address list from get header funcs
Pass in GMimeMessage to simplify To/Cc/Bcc headers. We'll eventually
remove the notmuch message passing altogether, but keep both for now
to not make too big changes at once.
Getting the headers from GMimeMessage using GMime functions fixes the
error on duplicate Cc headers reported by Daniel Kahn Gillmor
<dkg@fifthhorseman.net> in id:87d1ngv95p.fsf@alice.fifthhorseman.net.
Get rid of an intermediate function.
The small annoyance is the ownership differences in the address lists.
Diffstat (limited to 'test/T220-reply.sh')
| -rwxr-xr-x | test/T220-reply.sh | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/test/T220-reply.sh b/test/T220-reply.sh index a72068dd..818a8654 100755 --- a/test/T220-reply.sh +++ b/test/T220-reply.sh @@ -254,7 +254,6 @@ test_expect_equal_json "$output" ' }' test_begin_subtest "Reply to a message with multiple Cc headers" -test_subtest_known_broken add_email_corpus broken output=$(notmuch reply id:multiple-cc@example.org) test_expect_equal "$output" "From: Notmuch Test Suite <test_suite@notmuchmail.org> |