[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#24313: [PATCH] Add tests for dom.el

From: Michael Albinus
Subject: bug#24313: [PATCH] Add tests for dom.el
Date: Sun, 28 Aug 2016 11:21:30 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1.50 (gnu/linux)

Simen Heggestøyl <address@hidden> writes:

Hi Simen,

>> It would also be nice if you could add a docstring per test.
> Does it any value to such trivial test cases? For most of these I don't
> think there's more to add than "Tests <name-of-function>.", which is
> already conveyed by the name of the test.

It depends. Likely you are right, but sometimes it might be worth to
note also what is NOT covered by a test, or to mention possible side
effects which are tested implicitely. But it is your decision, of course.

> Thank you all for the feedback.
> -- Simen

Best regards, Michael.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]