Add unit testing support to the Figwheel template. #13
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I thought it'd be nice to include unit testing support by default now that CLJS has a core library for it.
So I did it :-) ....
Includes the necessary configuration in project.clj, plus supporting files, to allow the user to execute
lein cljsbuild once test
and have unit tests run in PhantomJS if it's installed.I had to make one small compromise; running the unit tests naturally loads the core.cljs, which prior to my modifications caused Reagent / Om (if used) to trigger, because they were being called directly from the core NS.
I've changed core.cljs and index.html to explicitly call a main function when we're running normally - and not to call it when we're testing (unless the user expliclty does so) which prevents Om / Reagent erroring because they can't find the app root when we're running tests.
(The alternative would have been to add a