search

digego / extempore

A cyber-physical programming environment
1.41k stars 127 forks source link

Get xtmtest-result working in a `let` environment #384

Closed pdcawley closed 4 years ago

pdcawley commented 4 years ago

Evaluating the test form in (interaction-environment) is okay if you're running your tests exclusively at the top level, but if, like me, you've set up a fixture in a (let ...) form, none of your carefully introduced symbols are in scope and Bad Things happen.

I think this, or something like it, might be why attempting to remove the double evaluation from xtmtest itself caused all the tests to blow up.

benswift commented 4 years ago

Nice one @pdcawley . As an aside, I'm loving having all the tests triggered on PR---so convenient.

pdcawley commented 4 years ago

I'll see if that fixes the issues where removing the double eval in xtmtest broke the test suite as well. Because if so: Yay!