Squeaky - Clean - Fix test runner alias - update deps.edn

JoshuaKyleLambert · April 11, 2025, 9:27pm

Hello, I’m new to Clojure and I’ve been enjoying exercism. When attempting the Squeaky Clean exercise I noticed that the tests would not run. I use the exercism cli and running exercism test nothing would happen. The result was the same when using clj -X:test. I decided to look into it and see if I could fix it. I replaced the deps.edn with the contents from the example on cognitect test runner repo. That fixed the problem. The lein test didnt seem to have a problem. I found this repo and made a PR but it was closed automatically and so now I’m here.

Hello everyone.

It’s a small contribution but I hope it helps someone.

Thank you,

Josh

Tests fail to run with current deps.edn when using the test alias or exercism test.

tasx · April 11, 2025, 9:40pm

Hi @JoshuaKyleLambert, and welcome to the forum

Thanks for spotting this issue. Upon further examination, it appears to affect multiple exercises. Ideally, we should update all deps.edn files at the same time, which I plan to do soon.

However, I’ll go ahead and accept your PR that fixes just this exercise. To keep the deps.edn files consistent, please replace your changes with the deps.edn from the log-levels exercise (assuming it resolves the issue), which you can find here:

github.com

exercism/clojure/blob/main/exercises/concept/log-levels/deps.edn

{:aliases {:test {:extra-paths ["test"]
                  :extra-deps {io.github.cognitect-labs/test-runner
                               {:git/url "https://github.com/cognitect-labs/test-runner.git"
                                :sha "705ad25bbf0228b1c38d0244a36001c2987d7337"}}
                  :main-opts ["-m" "cognitect.test-runner"]
                  :exec-fn cognitect.test-runner.api/test}}}

Link to your PR: Fix test runner alias update deps.edn by JoshuaKyleLambert · Pull Request #845 · exercism/clojure · GitHub

JoshuaKyleLambert · April 11, 2025, 10:22pm

tasx:

{:aliases {:test {:extra-paths ["test"]
                  :extra-deps {io.github.cognitect-labs/test-runner
                               {:git/url "https://github.com/cognitect-labs/test-runner.git"
                                :sha "705ad25bbf0228b1c38d0244a36001c2987d7337"}}
                  :main-opts ["-m" "cognitect.test-runner"]
                  :exec-fn cognitect.test-runner.api/test}}}

HI, I’ve updated my PR with the new contents.

Thank you for the prompt reply.

tasx · April 11, 2025, 10:24pm

@JoshuaKyleLambert I’ve merged the PR. Thanks!