lambdaisland / kaocha

Full featured next gen Clojure test runner
https://cljdoc.org/d/lambdaisland/kaocha/1.0.861/doc/1-introduction
Eclipse Public License 1.0
792 stars 81 forks source link

Run bb smoke test on CI #400

Closed borkdude closed 1 year ago

borkdude commented 1 year ago

@alysbrooks @plexus This PR adds a smoke test for bb on CI.

Notes:

borkdude commented 1 year ago

I see the tests are failing because of unexpected keys in the tests.edn file:

:kaocha.testable/desc "bb (clojure.test)"
alysbrooks commented 1 year ago
  • Does kaocha support providing a config on the command line without creating a file?

You can specify a particular tests.edn using --config-file. There isn't one for test-paths, at least not in our regular runner. I think you can use the exec-fn to do that, technically: https://cljdoc.org/d/lambdaisland/kaocha/1.80.1274/doc/readme#clojure-cli-toolsdeps-exec-fn-alternative.

borkdude commented 1 year ago

No option to pass config as edn without writing a file? If not, then I'll add another bb-tests.edn file.

alysbrooks commented 1 year ago

Might be good to add, but yeah, there's not a way to do it right now. You can include another EDN file since we have Aero support.

borkdude commented 1 year ago

@alysbrooks OK, this should settle it then.

codecov[bot] commented 1 year ago

Codecov Report

Patch and project coverage have no change.

Comparison is base (9242fd9) 75.34% compared to head (c6f365f) 75.34%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #400 +/- ## ======================================= Coverage 75.34% 75.34% ======================================= Files 52 52 Lines 2786 2786 Branches 267 267 ======================================= Hits 2099 2099 Misses 521 521 Partials 166 166 ``` | Flag | Coverage Δ | | |---|---|---| | integration | `56.56% <ø> (ø)` | | | unit | `69.63% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#carryforward-flags-in-the-pull-request-comment) to find out more. Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

alysbrooks commented 1 year ago

Thanks, @borkdude!

Banner reading THANK YOU in blue an moving up and down

borkdude commented 1 year ago

Woohoo!

plexus commented 1 year ago

Released in v1.81.1290

[lambdaisland/kaocha "1.81.1290"]                 ;; deps.edn
{lambdaisland/kaocha {:mvn/version "1.81.1290"}}  ;; project.clj
borkdude commented 1 year ago

Dank!