Re: [PATCH v2 2/4] test: emacs: new test "notmuch-search: replying to a thread (sending)"
authorPieter Praet <pieter@praet.org>
Wed, 22 Feb 2012 18:51:12 +0000 (19:51 +0100)
committerW. Trevor King <wking@tremily.us>
Fri, 7 Nov 2014 17:44:50 +0000 (09:44 -0800)
96/1d5fbb31698b5ede93bca7559e85538521878a [new file with mode: 0644]

diff --git a/96/1d5fbb31698b5ede93bca7559e85538521878a b/96/1d5fbb31698b5ede93bca7559e85538521878a
new file mode 100644 (file)
index 0000000..8c3cc03
--- /dev/null
@@ -0,0 +1,164 @@
+Return-Path: <pieter@praet.org>\r
+X-Original-To: notmuch@notmuchmail.org\r
+Delivered-To: notmuch@notmuchmail.org\r
+Received: from localhost (localhost [127.0.0.1])\r
+       by olra.theworths.org (Postfix) with ESMTP id C082D431FBD\r
+       for <notmuch@notmuchmail.org>; Wed, 22 Feb 2012 10:53:31 -0800 (PST)\r
+X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: -0.7\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
+       tests=[RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
+Received: from olra.theworths.org ([127.0.0.1])\r
+       by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
+       with ESMTP id G4xAsPOVd0N9 for <notmuch@notmuchmail.org>;\r
+       Wed, 22 Feb 2012 10:53:31 -0800 (PST)\r
+Received: from mail-we0-f181.google.com (mail-we0-f181.google.com\r
+       [74.125.82.181]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
+       (No client certificate requested)\r
+       by olra.theworths.org (Postfix) with ESMTPS id E8E94431FAE\r
+       for <notmuch@notmuchmail.org>; Wed, 22 Feb 2012 10:53:30 -0800 (PST)\r
+Received: by werp13 with SMTP id p13so272659wer.26\r
+       for <notmuch@notmuchmail.org>; Wed, 22 Feb 2012 10:53:29 -0800 (PST)\r
+Received-SPF: pass (google.com: domain of pieter@praet.org designates\r
+       10.180.101.200 as permitted sender) client-ip=10.180.101.200; \r
+Authentication-Results: mr.google.com;\r
+       spf=pass (google.com: domain of pieter@praet.org\r
+       designates 10.180.101.200 as permitted sender)\r
+       smtp.mail=pieter@praet.org\r
+Received: from mr.google.com ([10.180.101.200])\r
+       by 10.180.101.200 with SMTP id fi8mr37804971wib.20.1329936809975\r
+       (num_hops = 1); Wed, 22 Feb 2012 10:53:29 -0800 (PST)\r
+Received: by 10.180.101.200 with SMTP id fi8mr31281009wib.20.1329936809838;\r
+       Wed, 22 Feb 2012 10:53:29 -0800 (PST)\r
+Received: from localhost ([109.131.181.26])\r
+       by mx.google.com with ESMTPS id df3sm18089553wib.1.2012.02.22.10.53.28\r
+       (version=TLSv1/SSLv3 cipher=OTHER);\r
+       Wed, 22 Feb 2012 10:53:29 -0800 (PST)\r
+From: Pieter Praet <pieter@praet.org>\r
+To: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>,\r
+       Notmuch Mail <notmuch@notmuchmail.org>\r
+Subject: Re: [PATCH v2 2/4] test: emacs: new test "notmuch-search: replying to\r
+       a thread (sending)"\r
+In-Reply-To: <87fwe5lln3.fsf@gmail.com>\r
+References: <1310313335-4159-1-git-send-email-pieter@praet.org>\r
+       <1329683908-5435-1-git-send-email-pieter@praet.org>\r
+       <1329683908-5435-3-git-send-email-pieter@praet.org>\r
+       <87fwe5lln3.fsf@gmail.com>\r
+User-Agent: Notmuch/0.11.1+210~g6afc43e (http://notmuchmail.org) Emacs/23.3.1\r
+       (x86_64-unknown-linux-gnu)\r
+Date: Wed, 22 Feb 2012 19:51:12 +0100\r
+Message-ID: <87k43e8zxb.fsf@praet.org>\r
+MIME-Version: 1.0\r
+Content-Type: text/plain; charset=us-ascii\r
+X-Gm-Message-State:\r
+ ALoCoQntJAuwx6h0Rj8qD9FW4ISbwsZgxQzxFdceMigz0UcxcbcHR92H3ns1pgom9u0YOWXOnoeK\r
+X-BeenThere: notmuch@notmuchmail.org\r
+X-Mailman-Version: 2.1.13\r
+Precedence: list\r
+List-Id: "Use and development of the notmuch mail system."\r
+       <notmuch.notmuchmail.org>\r
+List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
+List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
+List-Post: <mailto:notmuch@notmuchmail.org>\r
+List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
+List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
+X-List-Received-Date: Wed, 22 Feb 2012 18:53:31 -0000\r
+\r
+On Mon, 20 Feb 2012 16:44:32 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmail.com> wrote:\r
+> On Sun, 19 Feb 2012 21:38:26 +0100, Pieter Praet <pieter@praet.org> wrote:\r
+> > As pointed out in the previous commit, the test for replying from\r
+> > within Emacs didn't actually submit the reply. This one does.\r
+> > ---\r
+> >  test/emacs |   41 +++++++++++++++++++++++++++++++++++++++++\r
+> >  1 files changed, 41 insertions(+), 0 deletions(-)\r
+> > \r
+> > diff --git a/test/emacs b/test/emacs\r
+> > index 308d749..0f4f42b 100755\r
+> > --- a/test/emacs\r
+> > +++ b/test/emacs\r
+> > @@ -273,6 +273,47 @@ On 01 Jan 2000 12:00:00 -0000, Notmuch Test Suite <test_suite@notmuchmail.org> w\r
+> >  EOF\r
+> >  test_expect_equal_file OUTPUT EXPECTED\r
+> >  \r
+> > +test_begin_subtest "notmuch-search: replying to a thread (sending)"\r
+> > +$TEST_DIRECTORY/smtp-dummy sent_message &\r
+> > +smtp_dummy_pid=$!\r
+> > +test_emacs \\r
+> > +'(let ((message-send-mail-function '\''message-smtpmail-send-it)\r
+> > +       (smtpmail-smtp-server "localhost")\r
+> > +       (smtpmail-smtp-service "25025"))\r
+> > +  (notmuch-search "subject:\"testing message sent via SMTP\"")\r
+> > +  (notmuch-test-wait)\r
+> > +  (notmuch-search-reply-to-thread)\r
+> > +  (end-of-buffer)\r
+> > +  (newline)\r
+> > +  (insert "Reply to a message via Emacs with fake SMTP")\r
+> > +  (message-send-and-exit))' >/dev/null 2>&1\r
+> > +wait ${smtp_dummy_pid}\r
+> > +notmuch new >/dev/null\r
+> > +sed \\r
+> > +    -e s',^User-Agent: Notmuch/.* Emacs/.*,User-Agent: Notmuch/XXX Emacs/XXX,' \\r
+> > +    -e s',^Message-ID: <.*>$,Message-ID: <XXX>,' \\r
+> > +    -e s',^In-Reply-To: <.*>$,In-Reply-To: <XXX>,' \\r
+> > +    -e s',^References: <.*>$,References: <XXX>,' \\r
+> > +    -e s',^Date: .*$,Date: Fri\, 29 Mar 1974 10:05:00 -0000,' < sent_message >OUTPUT\r
+> \r
+> This sed command is a (partial) copy past from "Sending a message via\r
+> (fake) SMTP" test.  [...]\r
+\r
+Aye, that's the one.\r
+\r
+\r
+> [...] I suggest adding notmuch_sent_message_sanitize\r
+> function which would replace common headers with XXX.\r
+> \r
+> Replace Date with XXX for Date for consistency with other headers,\r
+> AFAICT a valid date value is not needed here.\r
+>\r
+\r
+Agreed.\r
+\r
+Will have another look at it when Tomi's refactoring is done [1,2].\r
+\r
+\r
+> Regards,\r
+>   Dmitry\r
+> \r
+> > +cat <<EOF >EXPECTED\r
+> > +From: Notmuch Test Suite <test_suite@notmuchmail.org>\r
+> > +To: user@example.com\r
+> > +Subject: Re: Testing message sent via SMTP\r
+> > +In-Reply-To: <XXX>\r
+> > +References: <XXX>\r
+> > +User-Agent: Notmuch/XXX Emacs/XXX\r
+> > +Date: Fri, 29 Mar 1974 10:05:00 -0000\r
+> > +Message-ID: <XXX>\r
+> > +MIME-Version: 1.0\r
+> > +Content-Type: text/plain; charset=us-ascii\r
+> > +\r
+> > +On 01 Jan 2000 12:00:00 -0000, Notmuch Test Suite <test_suite@notmuchmail.org> wrote:\r
+> > +> This is a test that messages are sent via SMTP\r
+> > +\r
+> > +Reply to a message via Emacs with fake SMTP\r
+> > +EOF\r
+> > +test_expect_equal_file OUTPUT EXPECTED\r
+> > +\r
+> >  test_begin_subtest "Quote MML tags in reply"\r
+> >  message_id='test-emacs-mml-quoting@message.id'\r
+> >  add_message [id]="$message_id" \\r
+> > -- \r
+> > 1.7.8.1\r
+> > \r
+\r
+\r
+Peace\r
+\r
+-- \r
+Pieter\r
+\r
+[1] id:"m239a6wn4p.fsf@guru.guru-group.fi"\r
+[2] id:"87mx8a900t.fsf@praet.org"\r