unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
From: Pieter Praet <pieter@praet.org>
To: David Edmondson <dme@dme.org>
Cc: Notmuch Mail <notmuch@notmuchmail.org>
Subject: [PATCH 2/3] test: emacs: toggle eliding of non-matching messages in `notmuch-show'
Date: Sun, 19 Feb 2012 21:56:29 +0100	[thread overview]
Message-ID: <1329684990-12504-3-git-send-email-pieter@praet.org> (raw)
In-Reply-To: <1329684990-12504-1-git-send-email-pieter@praet.org>

See commits 44a544ed, 866ce8b1, 668b66ec.
---
 test/emacs                                         |   38 ++++++++++
 .../notmuch-show-elide-non-matching-messages-off   |   79 ++++++++++++++++++++
 .../notmuch-show-elide-non-matching-messages-on    |   75 +++++++++++++++++++
 3 files changed, 192 insertions(+), 0 deletions(-)
 create mode 100644 test/emacs.expected-output/notmuch-show-elide-non-matching-messages-off
 create mode 100644 test/emacs.expected-output/notmuch-show-elide-non-matching-messages-on

diff --git a/test/emacs b/test/emacs
index b207d20..320057a 100755
--- a/test/emacs
+++ b/test/emacs
@@ -553,5 +553,43 @@ test_emacs '(let ((notmuch-crypto-process-mime nil))
 	(test-visible-output))'
 test_expect_equal_file OUTPUT $EXPECTED/notmuch-show-process-crypto-mime-parts-on
 
+test_begin_subtest "notmuch-show: don't elide non-matching messages"
+test_emacs '(let ((notmuch-show-only-matching-messages nil))
+	(notmuch-search "from:lars@seas.harvard.edu and subject:\"Maildir storage\"")
+	(notmuch-test-wait)
+	(notmuch-search-show-thread)
+	(notmuch-test-wait)
+	(test-visible-output))'
+test_expect_equal_file OUTPUT $EXPECTED/notmuch-show-elide-non-matching-messages-off
+
+test_begin_subtest "notmuch-show: elide non-matching messages"
+test_emacs '(let ((notmuch-show-only-matching-messages t))
+	(notmuch-search "from:lars@seas.harvard.edu and subject:\"Maildir storage\"")
+	(notmuch-test-wait)
+	(notmuch-search-show-thread)
+	(notmuch-test-wait)
+	(test-visible-output))'
+test_expect_equal_file OUTPUT $EXPECTED/notmuch-show-elide-non-matching-messages-on
+
+test_begin_subtest "notmuch-show: elide non-matching messages (w/ notmuch-show-toggle-elide-non-matching)"
+test_emacs '(let ((notmuch-show-only-matching-messages nil))
+	(notmuch-search "from:lars@seas.harvard.edu and subject:\"Maildir storage\"")
+	(notmuch-test-wait)
+	(notmuch-search-show-thread)
+	(notmuch-test-wait)
+	(notmuch-show-toggle-elide-non-matching)
+	(test-visible-output))'
+test_expect_equal_file OUTPUT $EXPECTED/notmuch-show-elide-non-matching-messages-on
+
+test_begin_subtest "notmuch-show: elide non-matching messages (w/ prefix arg to notmuch-show)"
+test_emacs '(let ((notmuch-show-only-matching-messages nil))
+	(notmuch-search "from:lars@seas.harvard.edu and subject:\"Maildir storage\"")
+	(notmuch-test-wait)
+	(let ((current-prefix-arg t))
+	  (notmuch-search-show-thread))
+	(notmuch-test-wait)
+	(test-visible-output))'
+test_expect_equal_file OUTPUT $EXPECTED/notmuch-show-elide-non-matching-messages-on
+
 
 test_done
diff --git a/test/emacs.expected-output/notmuch-show-elide-non-matching-messages-off b/test/emacs.expected-output/notmuch-show-elide-non-matching-messages-off
new file mode 100644
index 0000000..b31fe62
--- /dev/null
+++ b/test/emacs.expected-output/notmuch-show-elide-non-matching-messages-off
@@ -0,0 +1,79 @@
+Lars Kellogg-Stedman <lars@seas.harvard.edu> (2009-11-17) (inbox signed)
+Subject: [notmuch] Working with Maildir storage?
+To: notmuch@notmuchmail.org
+Date: Tue, 17 Nov 2009 14:00:54 -0500
+
+[ multipart/mixed ]
+[ multipart/signed ]
+[ text/plain ]
+I saw the LWN article and decided to take a look at notmuch.  I'm
+currently using mutt and mairix to index and read a collection of
+Maildir mail folders (around 40,000 messages total).
+
+notmuch indexed the messages without complaint, but my attempt at
+searching bombed out. Running, for example:
+
+  notmuch search storage
+
+Resulted in 4604 lines of errors along the lines of:
+
+  Error opening
+  /home/lars/Mail/read-messages.2008/cur/1246413773.24928_27334.hostname,U=3026:2,S:
+  Too many open files
+
+I'm curious if this is expected behavior (i.e., notmuch does not work
+with Maildir) or if something else is going on.
+
+Cheers,
+
+[ 4-line signature. Click/Enter to show. ]
+[ application/pgp-signature ]
+[ text/plain ]
+[ 4-line signature. Click/Enter to show. ]
+ Mikhail Gusarov <dottedmag@dottedmag.net> (2009-11-17) (inbox signed unread)
+  Lars Kellogg-Stedman <lars@seas.harvard.edu> (2009-11-17) (inbox signed)
+  Subject: Re: [notmuch] Working with Maildir storage?
+  To: Mikhail Gusarov <dottedmag@dottedmag.net>
+  Cc: notmuch@notmuchmail.org
+  Date: Tue, 17 Nov 2009 15:33:01 -0500
+
+  [ multipart/mixed ]
+  [ multipart/signed ]
+  [ text/plain ]
+  > See the patch just posted here.
+
+  Is the list archived anywhere?  The obvious archives
+  (http://notmuchmail.org/pipermail/notmuch/) aren't available, and I
+  think I subscribed too late to get the patch (I only just saw the
+  discussion about it).
+
+  It doesn't look like the patch is in git yet.
+
+  -- Lars
+
+  [ 4-line signature. Click/Enter to show. ]
+  [ application/pgp-signature ]
+  [ text/plain ]
+  [ 4-line signature. Click/Enter to show. ]
+   Mikhail Gusarov <dottedmag@dottedmag.net> (2009-11-17) (inbox unread)
+   Keith Packard <keithp@keithp.com> (2009-11-17) (inbox unread)
+    Lars Kellogg-Stedman <lars@seas.harvard.edu> (2009-11-18) (inbox signed unread)
+    Subject: Re: [notmuch] Working with Maildir storage?
+    To: Keith Packard <keithp@keithp.com>
+    Cc: notmuch@notmuchmail.org
+    Date: Tue, 17 Nov 2009 19:50:40 -0500
+
+    [ multipart/mixed ]
+    [ multipart/signed ]
+    [ text/plain ]
+    > I've also pushed a slightly more complicated (and complete) fix to my
+    > private notmuch repository
+
+    The version of lib/messages.cc in your repo doesn't build because it's
+    missing "#include <stdint.h>" (for the uint32_t on line 466).
+
+    [ 4-line signature. Click/Enter to show. ]
+    [ application/pgp-signature ]
+    [ text/plain ]
+    [ 4-line signature. Click/Enter to show. ]
+ Carl Worth <cworth@cworth.org> (2009-11-18) (inbox unread)
diff --git a/test/emacs.expected-output/notmuch-show-elide-non-matching-messages-on b/test/emacs.expected-output/notmuch-show-elide-non-matching-messages-on
new file mode 100644
index 0000000..bafb479
--- /dev/null
+++ b/test/emacs.expected-output/notmuch-show-elide-non-matching-messages-on
@@ -0,0 +1,75 @@
+Lars Kellogg-Stedman <lars@seas.harvard.edu> (2009-11-17) (inbox signed)
+Subject: [notmuch] Working with Maildir storage?
+To: notmuch@notmuchmail.org
+Date: Tue, 17 Nov 2009 14:00:54 -0500
+
+[ multipart/mixed ]
+[ multipart/signed ]
+[ text/plain ]
+I saw the LWN article and decided to take a look at notmuch.  I'm
+currently using mutt and mairix to index and read a collection of
+Maildir mail folders (around 40,000 messages total).
+
+notmuch indexed the messages without complaint, but my attempt at
+searching bombed out. Running, for example:
+
+  notmuch search storage
+
+Resulted in 4604 lines of errors along the lines of:
+
+  Error opening
+  /home/lars/Mail/read-messages.2008/cur/1246413773.24928_27334.hostname,U=3026:2,S:
+  Too many open files
+
+I'm curious if this is expected behavior (i.e., notmuch does not work
+with Maildir) or if something else is going on.
+
+Cheers,
+
+[ 4-line signature. Click/Enter to show. ]
+[ application/pgp-signature ]
+[ text/plain ]
+[ 4-line signature. Click/Enter to show. ]
+  Lars Kellogg-Stedman <lars@seas.harvard.edu> (2009-11-17) (inbox signed)
+  Subject: Re: [notmuch] Working with Maildir storage?
+  To: Mikhail Gusarov <dottedmag@dottedmag.net>
+  Cc: notmuch@notmuchmail.org
+  Date: Tue, 17 Nov 2009 15:33:01 -0500
+
+  [ multipart/mixed ]
+  [ multipart/signed ]
+  [ text/plain ]
+  > See the patch just posted here.
+
+  Is the list archived anywhere?  The obvious archives
+  (http://notmuchmail.org/pipermail/notmuch/) aren't available, and I
+  think I subscribed too late to get the patch (I only just saw the
+  discussion about it).
+
+  It doesn't look like the patch is in git yet.
+
+  -- Lars
+
+  [ 4-line signature. Click/Enter to show. ]
+  [ application/pgp-signature ]
+  [ text/plain ]
+  [ 4-line signature. Click/Enter to show. ]
+    Lars Kellogg-Stedman <lars@seas.harvard.edu> (2009-11-18) (inbox signed unread)
+    Subject: Re: [notmuch] Working with Maildir storage?
+    To: Keith Packard <keithp@keithp.com>
+    Cc: notmuch@notmuchmail.org
+    Date: Tue, 17 Nov 2009 19:50:40 -0500
+
+    [ multipart/mixed ]
+    [ multipart/signed ]
+    [ text/plain ]
+    > I've also pushed a slightly more complicated (and complete) fix to my
+    > private notmuch repository
+
+    The version of lib/messages.cc in your repo doesn't build because it's
+    missing "#include <stdint.h>" (for the uint32_t on line 466).
+
+    [ 4-line signature. Click/Enter to show. ]
+    [ application/pgp-signature ]
+    [ text/plain ]
+    [ 4-line signature. Click/Enter to show. ]
-- 
1.7.8.1

  parent reply	other threads:[~2012-02-19 20:59 UTC|newest]

Thread overview: 87+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-25 10:18 [PATCH 0/3] reworked crypto toggle, plus a couple of other toggles David Edmondson
2012-01-25 10:18 ` [PATCH 1/3] emacs: Rework crypto switch toggle David Edmondson
2012-01-25 10:18 ` [PATCH 2/3] emacs: Allow `notmuch-show-mode' to display only matching messages David Edmondson
2012-01-25 10:18 ` [PATCH 3/3] emacs: Allow the indentation of content to be toggled David Edmondson
2012-01-29 22:30   ` Mark Walters
2012-01-30  9:29     ` David Edmondson
2012-01-30 16:32     ` David Edmondson
2012-01-29 20:06 ` [PATCH 0/3] reworked crypto toggle, plus a couple of other toggles Jameson Graef Rollins
2012-01-30  9:37   ` David Edmondson
2012-01-30 11:26     ` David Edmondson
2012-01-30 17:44       ` Jameson Graef Rollins
2012-01-30 11:31 ` [PATCH 0/3 v2] " David Edmondson
2012-01-30 11:31   ` [PATCH 1/3 v2] emacs: Rework crypto switch toggle David Edmondson
2012-01-30 11:31   ` [PATCH 2/3 v2] emacs: Allow `notmuch-show-mode' to display only matching messages David Edmondson
2012-01-30 11:31   ` [PATCH 3/3 v2] emacs: Allow the indentation of content to be toggled David Edmondson
2012-01-30 16:30 ` [PATCH 0/5 v3] reworked crypto toggle, plus a few other toggles David Edmondson
2012-01-30 16:31   ` [PATCH 1/5] emacs: Rework crypto switch toggle David Edmondson
2012-01-30 16:31   ` [PATCH 2/5] emacs: Allow `notmuch-show-mode' to display only matching messages David Edmondson
2012-01-30 16:31   ` [PATCH 3/5] emacs: Allow the indentation of content to be toggled David Edmondson
2012-01-30 16:31   ` [PATCH 4/5] emacs: Add a binding (>) to toggle the truncation of long lines David Edmondson
2012-01-30 17:33     ` Jani Nikula
2012-01-30 17:45       ` David Edmondson
2012-01-30 16:31   ` [PATCH 5/5] emacs: Retain the state of the buffer during `notmuch-show-refresh-view' David Edmondson
2012-01-30 20:19     ` Mark Walters
2012-01-31  6:17       ` David Edmondson
2012-01-30 17:03   ` [PATCH 0/5 v3] reworked crypto toggle, plus a few other toggles Tomi Ollila
2012-01-30 17:47   ` Jameson Graef Rollins
2012-01-31  8:09     ` David Edmondson
2012-01-31 16:31       ` Jameson Graef Rollins
2012-01-31 17:01         ` David Edmondson
2012-02-01 14:39 ` [PATCH v5 0/6] " David Edmondson
2012-02-01 14:39   ` [PATCH v5 1/6] emacs: Rework crypto switch toggle David Edmondson
2012-02-01 14:39   ` [PATCH v5 2/6] emacs: Allow `notmuch-show-mode' to display only matching messages David Edmondson
2012-02-01 14:39   ` [PATCH v5 3/6] emacs: Allow the indentation of content to be toggled David Edmondson
2012-02-01 14:39   ` [PATCH v5 4/6] emacs: Add a binding (t) to toggle the truncation of long lines David Edmondson
2012-02-01 14:39   ` [PATCH v5 5/6] emacs: Optionally retain the state of the buffer during `notmuch-show-refresh-view' David Edmondson
2012-02-01 14:39   ` [PATCH v5 6/6] emacs: Check that the parent buffer is alive before using it David Edmondson
2012-02-02 11:23 ` [PATCH v6 0/8] reworked crypto toggle, plus a few other toggles David Edmondson
2012-02-02 11:23   ` [PATCH v6 1/8] emacs: Rework crypto switch toggle David Edmondson
2012-02-03 10:47     ` David Edmondson
2012-02-02 11:23   ` [PATCH v6 2/8] emacs: Allow `notmuch-show-mode' to display only matching messages David Edmondson
2012-02-02 11:23   ` [PATCH v6 3/8] emacs: Allow the indentation of content to be toggled David Edmondson
2012-02-02 11:23   ` [PATCH v6 4/8] emacs: Add a binding (t) to toggle the truncation of long lines David Edmondson
2012-02-02 11:23   ` [PATCH v6 5/8] emacs: Optionally retain the state of the buffer during `notmuch-show-refresh-view' David Edmondson
2012-02-02 11:23   ` [PATCH v6 6/8] emacs: Check that the parent buffer is alive before using it David Edmondson
2012-02-02 11:23   ` [PATCH v6 7/8] emacs: Add `notmuch-show-only-matching-messages' David Edmondson
2012-02-02 11:23   ` [PATCH v6 8/8] emacs: A prefix argument to `notmuch-show' should invert the matching message behaviour David Edmondson
2012-02-06  9:21 ` [PATCH v7 0/8] reworked crypto toggle, plus a few other toggles David Edmondson
2012-02-06  9:21   ` [PATCH v7 1/8] emacs: Rework crypto switch toggle David Edmondson
2012-02-08  5:10     ` Austin Clements
2012-02-08  6:21       ` David Edmondson
2012-02-06  9:21   ` [PATCH v7 2/8] emacs: Allow `notmuch-show-mode' to display only matching messages David Edmondson
2012-02-06  9:21   ` [PATCH v7 3/8] emacs: Allow the indentation of content to be toggled David Edmondson
2012-02-06  9:21   ` [PATCH v7 4/8] emacs: Add a binding (t) to toggle the truncation of long lines David Edmondson
2012-02-06  9:21   ` [PATCH v7 5/8] emacs: Optionally retain the state of the buffer during `notmuch-show-refresh-view' David Edmondson
2012-02-06  9:21   ` [PATCH v7 6/8] emacs: Check that the parent buffer is alive before using it David Edmondson
2012-02-06  9:21   ` [PATCH v7 7/8] emacs: Add `notmuch-show-only-matching-messages' David Edmondson
2012-02-06  9:21   ` [PATCH v7 8/8] emacs: A prefix argument to `notmuch-show' should invert the matching message behaviour David Edmondson
2012-02-06 11:32   ` [PATCH v7 0/8] reworked crypto toggle, plus a few other toggles Tomi Ollila
2012-02-08  8:02 ` [PATCH v8 " David Edmondson
2012-02-08  8:02   ` [PATCH v8 1/8] emacs: Rework crypto switch toggle David Edmondson
2012-02-08 14:38     ` Austin Clements
2012-02-08  8:02   ` [PATCH v3 1/2] emacs: Use text properties rather than overlays in `notmuch-show-mode' David Edmondson
2012-02-12 15:47     ` David Bremner
2012-02-08  8:02   ` [PATCH v8 2/8] emacs: Allow `notmuch-show-mode' to display only matching messages David Edmondson
2012-02-08  8:02   ` [PATCH v3 2/2] test: Update test to match previous patch David Edmondson
2012-10-12 20:51     ` Ethan Glasser-Camp
2012-10-14 23:17       ` Jameson Graef Rollins
2012-02-08  8:02   ` [PATCH v8 3/8] emacs: Allow the indentation of content to be toggled David Edmondson
2012-02-08  8:02   ` [PATCH v8 4/8] emacs: Add a binding (t) to toggle the truncation of long lines David Edmondson
2012-02-08  8:02   ` [PATCH v8 5/8] emacs: Optionally retain the state of the buffer during `notmuch-show-refresh-view' David Edmondson
2012-02-08  8:02   ` [PATCH v8 6/8] emacs: Check that the parent buffer is alive before using it David Edmondson
2012-02-08  8:02   ` [PATCH v8 7/8] emacs: Add `notmuch-show-only-matching-messages' David Edmondson
2012-02-08  8:02   ` [PATCH v8 8/8] emacs: A prefix argument to `notmuch-show' should invert the matching message behaviour David Edmondson
2012-02-08 19:15   ` [PATCH v8 0/8] reworked crypto toggle, plus a few other toggles David Edmondson
2012-02-12 17:36   ` David Bremner
2012-02-19 20:56   ` Pieter Praet
2012-02-19 20:56     ` [PATCH 1/3] test: emacs: toggle processing of cryptographic MIME parts in `notmuch-show' Pieter Praet
2012-10-17 23:59       ` [PATCH 1/3] test: emacs: toggle eliding of non-matching messages " Ethan Glasser-Camp
2012-10-18  2:16         ` [PATCH] test: Move tests from emacs to emacs-show Ethan Glasser-Camp
2012-10-18 10:19           ` Tomi Ollila
2012-10-18 12:02           ` David Bremner
2012-02-19 20:56     ` Pieter Praet [this message]
2012-04-17 17:45       ` [PATCH 2/3] test: emacs: toggle eliding of non-matching messages in `notmuch-show' Mark Walters
2012-02-19 20:56     ` [PATCH 3/3] test: emacs: toggle thread content indentation " Pieter Praet
2012-10-15 22:17     ` [PATCH v8 0/8] reworked crypto toggle, plus a few other toggles Ethan Glasser-Camp
2012-10-18  0:41       ` David Bremner

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://notmuchmail.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1329684990-12504-3-git-send-email-pieter@praet.org \
    --to=pieter@praet.org \
    --cc=dme@dme.org \
    --cc=notmuch@notmuchmail.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://yhetil.org/notmuch.git/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).