search

day8 / re-frame-test

Cross platform (cljs and clj) utilities for testing re-frame applications
MIT License
109 stars 12 forks source link

change *test-context* from dynvar to atom #16

Closed mccraigmccraig closed 5 years ago

mccraigmccraig commented 5 years ago

because *test-context* was a dynvar it was causing problems with an async test with promise-based pre-conditions, meaning that wait-for* was being called from a random thread, and was unable to access the correct *test-context* value

this PR replaces the *test-context* dynvar of atoms with a test-context* atom of pure data

superstructor commented 5 years ago

Thanks for the contribution @mccraigmccraig