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

2011-12-21 Thread 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

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

2011-12-21 Thread Thomas Jost
Test that it's called once when `notmuch-hello' is called, and twice when calling `notmuch-hello-update' after that. --- Here it is. No broken subtest first since the first patch was already pushed though. Regards, Thomas test/emacs | 23 +++ 1 files changed, 23

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

2011-12-21 Thread 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