Re: [PATCH] test/emacs: revert invalid-from test to pre-86f89385 behaviour
David Bremner writes: > To the best of my understanding, this original behaviour was what > Carl's homebrew parser produced. With commit 86f89385 Austin switched > to using GMime (2.6). This produced arguably worse results, but since > the input was bad, we could live with it. Now with GMime 3.0 we are > getting the original results again, and there is no reason to consider > this test broken. pushed to master d ___ notmuch mailing list notmuch@notmuchmail.org https://notmuchmail.org/mailman/listinfo/notmuch
Re: [PATCH] test/emacs: revert invalid-from test to pre-86f89385 behaviour
On Sat 2019-05-11 20:45:59 -0600, David Bremner wrote: > To the best of my understanding, this original behaviour was what > Carl's homebrew parser produced. With commit 86f89385 Austin switched > to using GMime (2.6). This produced arguably worse results, but since > the input was bad, we could live with it. Now with GMime 3.0 we are > getting the original results again, and there is no reason to consider > this test broken. this works for me. I reviewed this earlier and tried to understand why one ugly representation of invalid data was any worse than the other ugly representation of invalid data. :P It'd be great if someone wants to propose a principled way to handle this invalid input, but just keeping the "broken" test around isn't a good way to track that problem. Please merge! --dkg signature.asc Description: PGP signature ___ notmuch mailing list notmuch@notmuchmail.org https://notmuchmail.org/mailman/listinfo/notmuch
Re: [PATCH] test/emacs: revert invalid-from test to pre-86f89385 behaviour
On Sat, May 11 2019, David Bremner wrote: > To the best of my understanding, this original behaviour was what > Carl's homebrew parser produced. With commit 86f89385 Austin switched > to using GMime (2.6). This produced arguably worse results, but since > the input was bad, we could live with it. Now with GMime 3.0 we are > getting the original results again, and there is no reason to consider > this test broken. if it works, push it ! > --- > test/T310-emacs.sh | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/test/T310-emacs.sh b/test/T310-emacs.sh > index c06a8133..5f74305d 100755 > --- a/test/T310-emacs.sh > +++ b/test/T310-emacs.sh > @@ -86,14 +86,13 @@ test_emacs "(let ((notmuch-show-indent-messages-width 4)) > test_expect_equal_file > $EXPECTED/notmuch-show-thread-maildir-storage-with-fourfold-indentation OUTPUT > > test_begin_subtest "notmuch-show for message with invalid From" > -test_subtest_known_broken > add_message "[subject]=\"message-with-invalid-from\"" \ > "[from]=\"\\\"Invalid \\\" From\\\" \"" > thread=$(notmuch search --output=threads subject:message-with-invalid-from) > test_emacs "(notmuch-show \"$thread\") > (test-output \"OUTPUT.raw\")" > cat-"Invalid " (2001-01-05) (inbox) > +Invalid " From (2001-01-05) (inbox) > Subject: message-with-invalid-from > To: Notmuch Test Suite > Date: GENERATED_DATE > -- > 2.11.0 > > ___ > notmuch mailing list > notmuch@notmuchmail.org > https://notmuchmail.org/mailman/listinfo/notmuch ___ notmuch mailing list notmuch@notmuchmail.org https://notmuchmail.org/mailman/listinfo/notmuch
[PATCH] test/emacs: revert invalid-from test to pre-86f89385 behaviour
To the best of my understanding, this original behaviour was what Carl's homebrew parser produced. With commit 86f89385 Austin switched to using GMime (2.6). This produced arguably worse results, but since the input was bad, we could live with it. Now with GMime 3.0 we are getting the original results again, and there is no reason to consider this test broken. --- test/T310-emacs.sh | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/test/T310-emacs.sh b/test/T310-emacs.sh index c06a8133..5f74305d 100755 --- a/test/T310-emacs.sh +++ b/test/T310-emacs.sh @@ -86,14 +86,13 @@ test_emacs "(let ((notmuch-show-indent-messages-width 4)) test_expect_equal_file $EXPECTED/notmuch-show-thread-maildir-storage-with-fourfold-indentation OUTPUT test_begin_subtest "notmuch-show for message with invalid From" -test_subtest_known_broken add_message "[subject]=\"message-with-invalid-from\"" \ "[from]=\"\\\"Invalid \\\" From\\\" \"" thread=$(notmuch search --output=threads subject:message-with-invalid-from) test_emacs "(notmuch-show \"$thread\") (test-output \"OUTPUT.raw\")" cat