[PATCH 1/4] test: Add `test_emacs_expect_t'.
David Edmondson
dme at dme.org
Tue Jan 17 05:24:55 PST 2012
(And one for the list...)
On Tue, 17 Jan 2012 17:09:35 +0400, Dmitry Kurochkin <dmitry.kurochkin at gmail.com> wrote:
> -1
>
> This is not what I suggested. I do not like the approach when a single
> function is used to both declare a subtest and test for result (as
> opposed to test_begin_subtest). The fact that it is possible to write
> tests in two different ways makes it hard to maintain and improve the
> test framework (one example would be known broken test support). I
> consider the proper way to write tests to be using the
> test_begin_subtest function. Other functions are not currently
> deprecated, but I am against adding new code that make the situation
> worse.
Sigh. Okay.
> Also, please consider documenting new functions in README.
Missed that, sorry.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120117/5f6808fd/attachment.pgp>
More information about the notmuch
mailing list