From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from localhost (localhost [127.0.0.1]) by arlo.cworth.org (Postfix) with ESMTP id 5BC316DE01EE for ; Wed, 21 Sep 2016 08:59:20 -0700 (PDT) X-Virus-Scanned: Debian amavisd-new at cworth.org X-Spam-Flag: NO X-Spam-Score: -0.019 X-Spam-Level: X-Spam-Status: No, score=-0.019 tagged_above=-999 required=5 tests=[AWL=-0.149, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_ENVFROM_END_DIGIT=0.25, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_PASS=-0.001] autolearn=disabled Received: from arlo.cworth.org ([127.0.0.1]) by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id tmo7N6xNWeMF for ; Wed, 21 Sep 2016 08:59:19 -0700 (PDT) Received: from mail-wm0-f65.google.com (mail-wm0-f65.google.com [74.125.82.65]) by arlo.cworth.org (Postfix) with ESMTPS id 9FA1B6DE0130 for ; Wed, 21 Sep 2016 08:59:18 -0700 (PDT) Received: by mail-wm0-f65.google.com with SMTP id 133so9392557wmq.2 for ; Wed, 21 Sep 2016 08:59:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:to:subject:in-reply-to:references:user-agent:date:message-id :mime-version; bh=2SPksRlui5ftXZvR6gzZ+9r4MI4fP0npufm/PP8ff1M=; b=ZkYTlzG4k5ORi+ilVxGEsLKAaisYIdnGM76bE1/Y8lX/0LLhdy+0W25HsBi/QMKNHk BhkLUcXm2a2gOqjXdBr8EeDx2jhTLgfg+syrcFc7JtorDiOCYshs5L846KaFHhXgveG1 5wWElKyoPsWaYznc//ZhqUSGKHIO3d9mQGmkTMEdq8/nXE+mMyq6DqOt1ubYJNn//bhp 2kh+JIECtZN5wo/QRCbgOUg6fu+QRpInahCiRsA2++pbQwL825N573KvLvzWJXiAw9j2 OIKQCGZKS+Vtky59i7HkmbUdMtZ+zzqQP3BbzI+EoB7D+/+cTyZvTmuHccEEDGWxzVe+ uP5g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:subject:in-reply-to:references :user-agent:date:message-id:mime-version; bh=2SPksRlui5ftXZvR6gzZ+9r4MI4fP0npufm/PP8ff1M=; b=D05pkfKSB3zcOwi/puymPfgksqP79PfHgYF80RPpH2AfAQJ1GrGkX3ff8Ykz+hMhee Iq/bN90vPPTURnpjsK4WEihgO+Wzb4bF24Ji3mrdwhNI6ER2HepF0pYXYZVjD8tzx3LI kuGAGTdhvffE0tiuynXh2WIcs3INH0kqndpfH7WR9JlPsJuaohwwpJxaQiTP7jBONqLT 2podVBUO664q2hKiVb7EJCBBXdqbae8JgYj4pnYYriGNDC2zB+x7T8MQZUugodcgNzIF zcqjWQ5I+QFfcfFUakSBgOKZ7PDEgnoGmuwM0BhkxsR5ynWHlrMVpGsMQA6U6LkIgLR+ T2dw== X-Gm-Message-State: AE9vXwPPKpHlZm4zsTpeVKEan7Hc+ZHEIkQrHOe9hUvts2vVNTV2tlbmmNin5fEh0qiC8A== X-Received: by 10.28.168.9 with SMTP id r9mr3720734wme.93.1474473557295; Wed, 21 Sep 2016 08:59:17 -0700 (PDT) Received: from localhost (5751dfa2.skybroadband.com. [87.81.223.162]) by smtp.gmail.com with ESMTPSA id xy4sm34217375wjc.2.2016.09.21.08.59.13 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 21 Sep 2016 08:59:13 -0700 (PDT) From: Mark Walters To: David Bremner , notmuch@notmuchmail.org Subject: Re: [PATCH] test/emacs: add broken test for message replying to msg to self In-Reply-To: <20160913022443.7728-1-david@tethera.net> References: <20160913022443.7728-1-david@tethera.net> User-Agent: Notmuch/0.18.1+485~gca076ce (https://notmuchmail.org) Emacs/24.4.1 (x86_64-pc-linux-gnu) Date: Wed, 21 Sep 2016 16:59:11 +0100 Message-ID: <87fuot1bxs.fsf@qmul.ac.uk> MIME-Version: 1.0 Content-Type: text/plain X-BeenThere: notmuch@notmuchmail.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: "Use and development of the notmuch mail system." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 21 Sep 2016 15:59:20 -0000 On Tue, 13 Sep 2016, David Bremner wrote: > This is a strange corner case where the removing of the user's address > from the To: header does the wrong thing. If we think it is > worth (eventually) fixing, this test can serve as a reminder. Hi I haven't tested the patch, but I think it worth having as a reminder. Presumably you ran into the bug yourself, and I think I have at some point, so people probably are hitting it. Best wishes Mark > --- > test/T310-emacs.sh | 24 ++++++++++++++++++++++++ > 1 file changed, 24 insertions(+) > > diff --git a/test/T310-emacs.sh b/test/T310-emacs.sh > index 21675b6..01385ae 100755 > --- a/test/T310-emacs.sh > +++ b/test/T310-emacs.sh > @@ -521,6 +521,30 @@ Notmuch Test Suite writes: > EOF > test_expect_equal_file OUTPUT EXPECTED > > +test_begin_subtest "Reply within emacs to message from self" > +test_subtest_known_broken > +add_message '[from]="test_suite@notmuchmail.org"' \ > + '[to]="test_suite@notmuchmail.org"' > +test_emacs "(let ((message-hidden-headers '())) > + (notmuch-show \"id:${gen_msg_id}\") > + (notmuch-show-reply) > + (test-output))" > +sed -i -e 's/^In-Reply-To: <.*>$/In-Reply-To: /' OUTPUT > +sed -i -e 's/^References: <.*>$/References: /' OUTPUT > +cat <EXPECTED > +From: Notmuch Test Suite > +To: test_suite@notmuchmail.org > +Subject: Re: Reply within emacs to message from self > +In-Reply-To: > +Fcc: ${MAIL_DIR}/sent > +References: > +--text follows this line-- > +test_suite@notmuchmail.org writes: > + > +> This is just a test message (#7) > +EOF > +test_expect_equal_file EXPECTED OUTPUT > + > test_begin_subtest "Quote MML tags in reply" > message_id='test-emacs-mml-quoting@message.id' > add_message [id]="$message_id" \ > -- > 2.9.3 > > _______________________________________________ > notmuch mailing list > notmuch@notmuchmail.org > https://notmuchmail.org/mailman/listinfo/notmuch