[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] [PATCH 2/2] test-ob-emacs-lisp: Test :lexical src block header a
Re: [O] [PATCH 2/2] test-ob-emacs-lisp: Test :lexical src block header argument
Thu, 14 Mar 2019 15:58:15 +0100
Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux)
Sebastian Miele <address@hidden> writes:
> * testing/lisp/test-ob-emacs-lisp.el
> ob-emacs-lisp/dynamic-lexical-edit): Add tests that check the
> correct handling of the :lexical header argument when executing
> source blocks and when creating editing buffers for source blocks.
However, your tests are very convoluted. It is better than no test, but
if, unfortunately, one of them fail in some distant future, it may take
more time understanding what happens in the test than actually fixing
Would you mind rewriting them with simple macros like, e.g.,
`org-test-with-temp-text', and use as little helper functions as
possible? IMO, code repetition in tests is not a problem.