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

From: Michael Albinus <[email-masked]>
Date: Sun, 28 Aug 2016 11:21:30 +0200

Simen Heggestøyl <[email-masked]> 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.