2022-08-19 babashka-sci-dev | Clojure Slack Archive

babashka-sci-dev

teodorlu 2022-08-19T11:08:51.780449Z

We can pick this up after any vacations 🙂 ----- Hi! After hitting some friction working on tests for neil dep upgrade, I figured that all I really wanted was to be able to run a few tests of my choice, not the whole test suite. So I gave it a shot. Example invocation:

bb test-only tests/dep-search-help-test tests/dep-search-test

I've got an implementation that looks like it's working well.

;; bb.edn task:

         test-only {:depends [gen-script]
                    :requires ([babashka.neil.test-runner])
                    :task (exec 'babashka.neil.test-runner/run-only)}

;; new function in babashka.neil.test-runner:

(defn run-only
  {:org.babashka/cli {:coerce {:only [:symbol]}
                      :args->opts (repeat :only)}}
  [{:keys [only]}]
  (doseq [test-sym only]
    (let [t (requiring-resolve test-sym)]
      (assert t)
      (t))))

WIP branch: https://github.com/teodorlu/neil/tree/test-only

teodorlu 2022-08-19T11:15:20.036599Z

unrelated babashka/cli comment: exec seems a bit arbitrarily mapped to something that relates to :org.babashka/cli. I would perhaps expect something like this in tasks.edn:

:tasks {:requires ([babashka.fs :as fs]
                    [clojure.string :as str]
                    [selmer.parser :as p]
                    [selmer.util :refer [without-escaping]]
                    [babashka/cli :as cli])

then

test-only {:depends [gen-script]
                    :requires ([babashka.neil.test-runner])
                    :task (cli/exec 'babashka.neil.test-runner/run-only)}

. But now there's a public API, so keeping it as-is might be better. Not sure, others can be the judge of that.

teodorlu 2022-08-19T11:24:05.165049Z

Just creating a new test namespace would also kind of solve the problem. Depends on how we want to encourage people working with tests, I guess.

teodorlu 2022-08-19T11:09:20.616329Z

If this is an addition we want, I can add it to the neil dep upgrade PR, or create a separate PR.

borkdude 2022-08-19T14:13:35.281669Z

@teodorlu I think that's ok, but we could also just use the cognitect test runner for this maybe

borkdude 2022-08-19T14:14:02.521869Z

See here: https://github.com/babashka/tools.namespace

teodorlu 2022-08-19T15:36:58.127789Z

Makes sense - accidentally inventing a test runner cli is probably not something we want to do if we can use a tool made for the job.

Clojurians Log v2

babashka-sci-dev