basilisp-lang / basilisp / #588

99%

main: 99%

push

coveralls-python

Clean up the test suite (#596)

 * Moved `ns` and `lcompile` fixtures into `tests/basilisp/conftest.py` so they can be used across the entire test suite. Updated all of the tests using similar fixtures to use these common fixtures instead.
    * This fixes a potential issue that can happen when Basilisp tests are run by filenames (as they are on CircleCI). If certain improperly isolated tests run before `tests/basilisp/prompt_test.py`, the current Namespace when that test is run may potentially have new `Vars` which cause the completion tests to fail with those additional Var names.
 * Moved `core_ns` fixtures into `tests/basilisp/conftest.py` so they can be used by all tests.
 * Parameterized all of the `tests/basilisp/lrepr_test.py` tests.
 * Moved remaining `basilisp.string` tests from `tests/basilisp/string_test.py` into `tests/basilisp/test_string.py` so all the tests are in one location and written in Basilisp.
 * Collapse `tests/basilisp/test_munge.py` into `tests/basilisp/test_langutil.py` (where `demunge` tests already lived alone).
 * Collapse `tests/basilisp/maybe_test.py` into `tests/basilisp/util_test.py` (where other `basilisp.util` tests already lived).

2508 of 2563 branches covered (97.85%)

Branch coverage included in aggregate %.

7009 of 7090 relevant lines covered (98.86%)

0.99 hits per line