lfe / lfe

Lisp Flavoured Erlang (LFE)
Apache License 2.0
2.31k stars 138 forks source link

Pull in the rest of the LFE macros for eunit #413

Open oubiwann opened 3 years ago

oubiwann commented 3 years ago

This is a proposal, for @rvirding's approval:

Some of the macros that wrap eunit (from the ltest project) are already in LFE-proper (here), yet these are only here in order to run the unit tests on the clj.lfe library; they could be used by everyone/any project, if we structured things a little differently. The idea of providing an eunit wrapper in LFE-proper isn't all that bad, considering eunit comes with Erlang/OTP and isn't a third-party library. Note that the eunit wrapper DSL has only one dependency: LFE itself (it doesn't even require rebar3).

Required tasks:

Additional support:

oubiwann commented 3 years ago

I did a quick check on lines of code, and about 1/3rd of the eunit wrappers were brought into LFE.

Just 2/3rds to go ... 😉

oubiwann commented 3 years ago

Also, I should note that these wrappers have been used on numerous projects over the past 7 years (6 years in its current location). Only minor updates have been made in the macros over the last 4 years. All-told, this is stable code with a stable API.