argumentcomputer / lurk-lisp

Lisp implementation of Lurk.
MIT License
67 stars 6 forks source link

Add test and fix discarding env bug. #15

Closed porcuquine closed 3 years ago

porcuquine commented 3 years ago

This adds a test case and fix for the bug described in a discussion ending here: https://github.com/lurk-lang/lurk/issues/14#issuecomment-955119558

NOTE: This fix does cause the meta-tests.lurk to fail when run in :IMPL mode. For now, we do not expect those tests to pass, so we should accept the regression there given that we know this fix addresses a real issue.

The other small changes here are to repl testing ergonomics.

namin commented 3 years ago

Can you confirm that you want the other changes: DIRNAME and minor such refactorings that are mixed in with this change?

Ah, my bad. I see that you want them!