[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 <simenheg@gmail.com> 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.