Re: [PATCH] test: add tests for `notmuch-hello-refresh-hook'

Subject: Re: [PATCH] test: add tests for `notmuch-hello-refresh-hook'

Date: Wed, 21 Dec 2011 22:26:40 +0400

To: Thomas Jost

Cc: notmuch@notmuchmail.org

From: Dmitry Kurochkin

Hi Thomas.

I think all hook count tests must be consistent.  We should either add
hook counters in test-lib.el (like it is already done for
`notmuch-hello-mode-hook') or use lambdas in individual test cases (like
you did in the patch).

I believe the former approach is better (that is why I used it in the
first place :)).  So I sent patches [1] that add general hook counter to
test-lib.el and `notmuch-hello-mode-hook' tests that use it.

Regards,
  Dmitry

[1] id:"1324491506-1134-1-git-send-email-dmitry.kurochkin@gmail.com"

Thread: