Fork me on GitHub
#kaocha
<
2020-11-30
>
plexus07:11:46

I would push it clojars, I don't think that's overkill. For deps.edn user's a git repo or even a gist would be enough, but then you're excluding lein/boot users.

plexus07:11:16

Just make sure the namespace/plugin name starts with something else than kaocha., so it's clear that it's third party

👌 1
✔️ 1
andrea.crotti15:11:14

ah interesting it looks like if I don't specify a test-id kaocha will just pick the last one defined in tests.edn

andrea.crotti15:11:22

if I just run the command with --focusfor example

andrea.crotti15:11:29

I guess I always have to specify the test id

plexus15:11:39

not sure what you're referring to @andrea.crotti... in tests.edn you can leave off the test id, then it will default to :unit (so you should only omit it at most once). If you don't put a test suite name on the command line it will run all suites

andrea.crotti16:11:50

ah right ok it's running them all then

andrea.crotti16:11:13

running all the suites when you specify something with --focus seems a bit unnecessary though

andrea.crotti16:11:23

but I guess it would not know what to run if I don 't specify the test id of course

plexus16:11:40

yes, exactly... there's always this kind of chicken and egg problem. This becomes especially problematic when there's significant setup/teardown for a suite, as with kaocha-cljs/kaocha-cljs2, since we can't filter until we know the test plan...

andrea.crotti16:11:15

yeah maybe it should just error if you try to --focuswithout passing a test-id?

andrea.crotti16:11:26

not sure that would ever what you want if you have multiple test ids?

plexus16:11:56

yeah I tihnk it would be reasonable for that to be an error

andrea.crotti16:11:34

I'll open an issue to keep track of it

andrea.crotti22:11:38

if that's a reasonable approach

andrea.crotti22:11:18

not sure if only --focus should throw an error or also --focus-meta skip/skip-meta