Re: [PATCH] test: don't bail out of `run_emacs' too early when missing prereqs

Subject: Re: [PATCH] test: don't bail out of `run_emacs' too early when missing prereqs

Date: Sun, 15 Jan 2012 22:44:53 -0400

To: Pieter Praet, Dmitry Kurochkin

Cc: Notmuch Mail

From: David Bremner

On Sat, 14 Jan 2012 10:09:37 +0100, Pieter Praet <pieter@praet.org> wrote:
> When running the Emacs tests in verbose mode, only the first missing
> prereq is reported because the `run_emacs' function is short-circuited
> early:

pushed.

d

• Previous message (by thread): Re: [PATCH] test: don't bail out of `run_emacs' too early when missing prereqs

Thread:

• Dmitry Kurochkin—[PATCH v2 00/10] test: (hopefully) better test prerequisites [inbox, unread]
  • Dmitry Kurochkin—[PATCH v2 01/10] test: move subtest variables reset into a dedicated function [inbox, notmuch::patch, notmuch::pushed, unread]
  • Dmitry Kurochkin—[PATCH v2 02/10] test: set EMACS_SERVER variable only after dtach(1) was successfully started [inbox, notmuch::patch, notmuch::pushed, unread]
  • Dmitry Kurochkin—[PATCH v2 03/10] test: add test state reset to test_expect_* functions that did not have it [inbox, notmuch::patch, notmuch::pushed, unread]
    • David Bremner—Re: [PATCH v2 03/10] test: add test state reset to test_expect_* functions that did not have it [inbox, unread]
      • Dmitry Kurochkin—[PATCH v3 00/7] test: (hopefully) better test prerequisites [inbox, unread]
        • Dmitry Kurochkin—[PATCH v3 1/7] test: add support for external executable dependencies [inbox, notmuch::patch, notmuch::pushed, notmuch::test, unread]
        • Dmitry Kurochkin—[PATCH v3 2/7] test: fix "skipping test" verbose output [inbox, notmuch::patch, notmuch::pushed, notmuch::test, unread]
        • Dmitry Kurochkin—[PATCH v3 3/7] test: skip all subtests if external dependencies are missing during init [inbox, notmuch::patch, notmuch::pushed, notmuch::test, unread]
        • Dmitry Kurochkin—[PATCH v3 4/7] test: declare external dependencies for the tests [inbox, notmuch::patch, notmuch::pushed, notmuch::test, unread]
        • Dmitry Kurochkin—[PATCH v3 5/7] test: add function to explicitly check for external dependencies [inbox, notmuch::patch, notmuch::pushed, notmuch::test, unread]
        • Dmitry Kurochkin—[PATCH v3 6/7] test: check if emacs is available in the beginning of test_emacs [inbox, notmuch::patch, notmuch::pushed, notmuch::test, unread]
        • Dmitry Kurochkin—[PATCH v3 7/7] test: fix "Stashing in notmuch-search" test when emacs is not available [inbox, notmuch::patch, notmuch::pushed, notmuch::test, unread]
        • David Bremner—Re: [PATCH v3 00/7] test: (hopefully) better test prerequisites [inbox, unread]
        • Pieter Praet—Re: [PATCH v3 00/7] test: (hopefully) better test prerequisites [inbox, notmuch::review, unread]
          • Pieter Praet—[PATCH] test: don't bail out of `run_emacs' too early when missing prereqs [inbox, notmuch::obsolete, notmuch::patch, notmuch::test, unread]
            • Dmitry Kurochkin—Re: [PATCH] test: don't bail out of `run_emacs' too early when missing prereqs [inbox, notmuch::review, unread]
              • Pieter Praet—Re: [PATCH] test: don't bail out of `run_emacs' too early when missing prereqs [inbox, unread]
                • Pieter Praet—[PATCH] test: don't bail out of `run_emacs' too early when missing prereqs [inbox, notmuch::patch, notmuch::pushed, notmuch::test, unread]
                  • Dmitry Kurochkin—Re: [PATCH] test: don't bail out of `run_emacs' too early when missing prereqs [inbox, notmuch::review, unread]
                  • David Bremner—Re: [PATCH] test: don't bail out of `run_emacs' too early when missing prereqs [inbox, unread]
                • Dmitry Kurochkin—Re: [PATCH] test: don't bail out of `run_emacs' too early when missing prereqs [inbox, unread]
                  • Pieter Praet—Re: [PATCH] test: don't bail out of `run_emacs' too early when missing prereqs [inbox, unread]
  • Dmitry Kurochkin—[PATCH v2 04/10] test: add support for external executable dependencies [inbox, notmuch::patch, notmuch::stale, unread]
  • Dmitry Kurochkin—[PATCH v2 05/10] test: fix "skipping test" verbose output [inbox, notmuch::patch, notmuch::stale, unread]
  • Dmitry Kurochkin—[PATCH v2 06/10] test: skip all subtests if external dependencies are missing during init [inbox, notmuch::patch, notmuch::stale, unread]
  • Dmitry Kurochkin—[PATCH v2 07/10] test: declare external dependencies for the tests [inbox, notmuch::patch, notmuch::stale, unread]
  • Dmitry Kurochkin—[PATCH v2 08/10] test: add function to explicitly check for external dependencies [inbox, notmuch::patch, notmuch::stale, unread]
  • Dmitry Kurochkin—[PATCH v2 09/10] test: check if emacs is available in the beginning of test_emacs [inbox, notmuch::patch, notmuch::stale, unread]
  • Dmitry Kurochkin—[PATCH v2 10/10] test: fix "Stashing in notmuch-search" test when emacs is not available [inbox, notmuch::patch, notmuch::stale, unread]