Re: [PATCH 0/4 v43] emacs test helpers

Subject: Re: [PATCH 0/4 v43] emacs test helpers

Date: Tue, 24 Jan 2012 22:13:14 +0200

To: David Edmondson, notmuch@notmuchmail.org

Cc:

On Tue, 24 Jan 2012 16:14:03 +0000, David Edmondson <dme@dme.org> wrote:
> Small changes as per Dmitry's last comments.
> 
> David Edmondson (4):
>   test: Don't return the result of checking for running emacs to the
>     tester.
>   test: Add `test_emacs_expect_t'.
>   test: Add more helpers for emacs tests.
>   test: Add address cleaning tests.

LGTM -- all 4.


Tomi

Previous message (by thread): Re: [PATCH 0/4 v43] emacs test helpers

Thread:

David Edmondson—emacs based tests, version 3 [inbox, unread]
- David Edmondson—[PATCH 1/4] test: Add `test_emacs_expect_t'. [inbox, notmuch::obsolete, notmuch::patch, unread]
  - Dmitry Kurochkin—Re: [PATCH 1/4] test: Add `test_emacs_expect_t'. [inbox, unread]
    - David Edmondson—Re: [PATCH 1/4] test: Add `test_emacs_expect_t'. [inbox, signed, unread]
    - David Edmondson—[PATCH 1/3] test: Don't return the result of checking for running emacs to the tester. [inbox, notmuch::obsolete, notmuch::patch, unread]
      - David Edmondson—[PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, notmuch::obsolete, notmuch::patch, unread]
        Dmitry Kurochkin—Re: [PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, notmuch::review, unread]
        David Edmondson—Re: [PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, signed, unread]
        Dmitry Kurochkin—Re: [PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, unread]
        David Edmondson—Re: [PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, signed, unread]
        Tomi Ollila—Re: [PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, unread]
        Tomi Ollila—Re: [PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, unread]
        David Edmondson—Re: [PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, signed, unread]
        Tomi Ollila—Re: [PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, unread]
        David Edmondson—Re: [PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, signed, unread]
        Tomi Ollila—Re: [PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, unread]
        David Edmondson—[PATCH 1/3] test: Don't return the result of checking for running emacs to the tester. [inbox, unread]
        David Edmondson—[PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, unread]
        David Edmondson—Re: [PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, signed, unread]
        Dmitry Kurochkin—Re: [PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, notmuch::review, unread]
        David Edmondson—[PATCH 3/3] test: Add address cleaning tests. [inbox, unread]
        Dmitry Kurochkin—Re: [PATCH 3/3] test: Add address cleaning tests. [inbox, notmuch::review, unread]
        Dmitry Kurochkin—Re: [PATCH 1/3] test: Don't return the result of checking for running emacs to the tester. [inbox, notmuch::review, unread]
      - David Edmondson—[PATCH 3/3] test: Add address cleaning tests. [inbox, notmuch::obsolete, notmuch::patch, unread]
      - Dmitry Kurochkin—Re: [PATCH 1/3] test: Don't return the result of checking for running emacs to the tester. [inbox, unread]
        David Edmondson—Re: [PATCH 1/3] test: Don't return the result of checking for running emacs to the tester. [inbox, signed, unread]
        Dmitry Kurochkin—Re: [PATCH 1/3] test: Don't return the result of checking for running emacs to the tester. [inbox, unread]
  - David Edmondson—[PATCH 1/4 v42] test: Don't return the result of checking for running emacs to the tester. [inbox, notmuch::obsolete, notmuch::patch, notmuch::trivial, unread]
    - David Edmondson—[PATCH 2/4 v42] test: Add `test_emacs_expect_t'. [inbox, notmuch::obsolete, notmuch::patch, unread]
      - Dmitry Kurochkin—Re: [PATCH 2/4 v42] test: Add `test_emacs_expect_t'. [inbox, unread]
    - David Edmondson—[PATCH 3/4 v42] test: Add more helpers for emacs tests. [inbox, notmuch::obsolete, notmuch::patch, unread]
      - Dmitry Kurochkin—Re: [PATCH 3/4 v42] test: Add more helpers for emacs tests. [inbox, unread]
        David Edmondson—Re: [PATCH 3/4 v42] test: Add more helpers for emacs tests. [inbox, signed, unread]
    - David Edmondson—[PATCH 4/4 v42] test: Add address cleaning tests. [inbox, notmuch::obsolete, notmuch::patch, unread]
      - Dmitry Kurochkin—Re: [PATCH 4/4 v42] test: Add address cleaning tests. [inbox, unread]
    - Dmitry Kurochkin—Re: [PATCH 1/4 v42] test: Don't return the result of checking for running emacs to the tester. [inbox, unread]
  - David Edmondson—[PATCH 0/4 v43] emacs test helpers [inbox, unread]
    - David Edmondson—[PATCH 1/4 v43] test: Don't return the result of checking for running emacs to the tester. [inbox, notmuch::patch, notmuch::pushed, unread]
    - David Edmondson—[PATCH 2/4 v43] test: Add `test_emacs_expect_t'. [inbox, notmuch::patch, notmuch::pushed, unread]
    - David Edmondson—[PATCH 3/4 v43] test: Add more helpers for emacs tests. [inbox, notmuch::patch, notmuch::pushed, unread]
    - David Edmondson—[PATCH 4/4 v43] test: Add address cleaning tests. [inbox, notmuch::patch, notmuch::pushed, unread]
    - Dmitry Kurochkin—Re: [PATCH 0/4 v43] emacs test helpers [inbox, unread]
    - Tomi Ollila—Re: [PATCH 0/4 v43] emacs test helpers [inbox, unread]
    - David Bremner—Re: [PATCH 0/4 v43] emacs test helpers [inbox, unread]
- David Edmondson—[PATCH 2/4] test: Add address cleaning tests. [inbox, notmuch::obsolete, notmuch::patch, unread]
  - Dmitry Kurochkin—Re: [PATCH 2/4] test: Add address cleaning tests. [inbox, unread]
    - David Edmondson—Re: [PATCH 2/4] test: Add address cleaning tests. [inbox, signed, unread]
- David Edmondson—[PATCH 3/4] emacs: Avoid `mail-header-parse-address' in `notmuch-show-clean-address'. [inbox, notmuch::obsolete, notmuch::patch, unread]
- David Edmondson—[PATCH 4/4] emacs: Another special case for `notmuch-show-clean-address'. [inbox, notmuch::obsolete, notmuch::patch, unread]