Test support is sweet! Thank you!
Is support for reporting on test var available yet?
I don't understand code yet. I do see https://github.com/squint-cljs/squint/blob/v0.11.187/src/squint/test.cljs#L73-L74
Can I override the :begin-test-var defaults?
@lee try the newest version, locally I have the test multimethod working (v0.11.188)
Oh cool! Will give it a go!
Sweet!:
Testing cljdoc.client.single-docset-search.logic-test
=== tokenize-test
=== sub-tokenize-test
=== tokenizer-test
=== search-test
Ran 4 tests containing 31 assertions.
0 failures, 0 errors.Thanks for making this a wee simple PR: https://github.com/cljdoc/cljdoc/pull/1185/changes
That's one bit I didn't implement yet. I'm surprised you're hitting this gap this soon laughcry
I think we first need multimethods for this
Ha! I just like the extra report var feedback when running tests. I can certainly live without it!
It's a natural follow up issue, thanks for the inspiration. Upvote welcome. https://github.com/squint-cljs/squint/issues/806
And thanks for being so quick with testing it
(I'll get back to your fs PRs soon, working through a pile of other stuff too)
(no prob, no hurry, do your thing)
Here's my switch to using squints' new test support for cljdoc: https://github.com/cljdoc/cljdoc/pull/1183/changes/9c3cb3cf8db4763647fa37cce394a3abac653513 (Before the new test support, I had based my test support on what you did for nextjournal/clojure-mode, so diff should look familiar!).
does it work? that's so rewarding to see
Yeah, works awesomely!
Testing cljdoc.client.single-docset-search.logic-test
Ran 4 tests containing 31 assertions.
0 failures, 0 errors.(I did sanity-test that assertions fail when they should too)