basilisp-lang / basilisp / #435

99%

main: 99%

push

coveralls-python

Include the Function or Method context in the AST Node environment (#549)

This PR is a set of otherwise-independent changes I pulled out of #547. It includes the following changes:
 * The titular Function/Method context stored in the `NodeEnv` object of Lisp AST nodes
 * Changing the generated module-wide `__NS` variable to `_NS` because I learned that variables prefixed by double underscores are [always mangled](https://bugs.python.org/issue27793) in a class lexical context in Python regardless of the presence of a `global` statement.
 * Added a few tests for asserting the resolution of `def`'ed Basilisp Vars match the expected behavior from Clojure

Fixes #548

2434 of 2491 branches covered (97.71%)

Branch coverage included in aggregate %.

6573 of 6651 relevant lines covered (98.83%)

0.99 hits per line