[PATCH 04/13] test: add 'Emacs' prereq to dependent 'emacs' tests

Pieter Praet pieter at praet.org
Tue Nov 1 12:57:59 PDT 2011


On Mon,  3 Oct 2011 18:47:18 +0200, Thomas Jost <schnouki at schnouki.net> wrote:
> [...]

Nice catches @

  - "Sending a message via (fake) SMTP"
  - "Hiding message following HTML part"

(though I think the latter test wasn't there yet when I submitted my
patch series)


New "issues":

  - The SCREEN prereq would preferrably be added in a separate commit.

  - @ "Verify that sent messages are saved/searchable (via FCC)":

    This doesn't need an EMACS prereq.

  - @ "Reply within emacs":

    `sed' will want to treat "EMACS" as an input file, so I think you
    meant to do this:

      test_have_prereq EMACS \
        && sed -i -e 's/^In-Reply-To: <.*>$/In-Reply-To: <XXX>/' OUTPUT


Peace

-- 
Pieter


More information about the notmuch mailing list