[PATCH] test: add simple tests for post-insert hook

diff --git a/34/818c4a32005a686a60ec457c1b4e202941e595 b/34/818c4a32005a686a60ec457c1b4e202941e595
new file mode 100644 (file)
index 0000000..4f0b033
--- /dev/null
+++ b/34/818c4a32005a686a60ec457c1b4e202941e595
@@ -0,0 +1,101 @@
+Return-Path: <bremner@tesseract.cs.unb.ca>\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 A479B431FBC\r
+       for <notmuch@notmuchmail.org>; Sat, 18 Oct 2014 00:22:54 -0700 (PDT)\r
+X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: 0\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]\r
+       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 v-9I-8xYx1Is for <notmuch@notmuchmail.org>;\r
+       Sat, 18 Oct 2014 00:22:49 -0700 (PDT)\r
+Received: from yantan.tethera.net (yantan.tethera.net [199.188.72.155])\r
+       (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits))\r
+       (No client certificate requested)\r
+       by olra.theworths.org (Postfix) with ESMTPS id 27EAE431FB6\r
+       for <notmuch@notmuchmail.org>; Sat, 18 Oct 2014 00:22:49 -0700 (PDT)\r
+Received: from remotemail by yantan.tethera.net with local (Exim 4.80)\r
+       (envelope-from <bremner@tesseract.cs.unb.ca>)\r
+       id 1XfOLg-0001X3-Om; Sat, 18 Oct 2014 04:22:48 -0300\r
+Received: (nullmailer pid 27657 invoked by uid 1000); Sat, 18 Oct 2014\r
+       07:22:43 -0000\r
+From: David Bremner <david@tethera.net>\r
+To: notmuch@notmuchmail.org\r
+Subject: [PATCH] test: add simple tests for post-insert hook\r
+Date: Sat, 18 Oct 2014 09:22:35 +0200\r
+Message-Id: <1413616955-27612-1-git-send-email-david@tethera.net>\r
+X-Mailer: git-send-email 2.1.1\r
+In-Reply-To:\r
+ <27c7ada91f96deda6e51732e0397bffd2ef2ab42.1411914914.git.jani@nikula.org>\r
+References:\r
+ <27c7ada91f96deda6e51732e0397bffd2ef2ab42.1411914914.git.jani@nikula.org>\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: Sat, 18 Oct 2014 07:22:54 -0000\r
+\r
+Most of the existing tests for pre/post-new hook don't seem to apply.\r
+---\r
+ test/T400-hooks.sh | 18 ++++++++++++++++++\r
+ 1 file changed, 18 insertions(+)\r
+\r
+diff --git a/test/T400-hooks.sh b/test/T400-hooks.sh\r
+index 77e8569..e741211 100755\r
+--- a/test/T400-hooks.sh\r
++++ b/test/T400-hooks.sh\r
+@@ -30,6 +30,8 @@ rm_hooks () {\r
+ \r
+ # add a message to generate mail dir and database\r
+ add_message\r
++# create maildir structure for notmuch-insert\r
++mkdir -p "$MAIL_DIR"/{cur,new,tmp}\r
+ \r
+ test_begin_subtest "pre-new is run"\r
+ rm_hooks\r
+@@ -45,6 +47,16 @@ create_echo_hook "post-new" expected output\r
+ notmuch new > /dev/null\r
+ test_expect_equal_file expected output\r
+ \r
++test_begin_subtest "post-insert hook is run"\r
++rm_hooks\r
++generate_message\r
++create_echo_hook "post-insert" expected output\r
++echo $gen_msg_filename\r
++cat output\r
++notmuch insert < "$gen_msg_filename"\r
++echo $?\r
++test_expect_equal_file expected output\r
++\r
+ test_begin_subtest "pre-new is run before post-new"\r
+ rm_hooks\r
+ generate_message\r
+@@ -82,6 +94,12 @@ test_expect_equal_file expected output\r
+ # depends on the previous subtest leaving broken hook behind\r
+ test_expect_code 1 "post-new non-zero exit status (notmuch status)" "notmuch new"\r
+ \r
++rm_hooks\r
++generate_message\r
++create_failing_hook "post-insert"\r
++test_expect_success "post-insert hook does not affect insert status" \\r
++    "notmuch insert < \"$gen_msg_filename\" > /dev/null"\r
++\r
+ # test_begin_subtest "hook without executable permissions"\r
+ rm_hooks\r
+ mkdir -p ${HOOK_DIR}\r
+-- \r
+2.1.1\r
+\r