Fork me on GitHub
#kaocha
<
2020-08-17
>
plexus13:08:25

@reefersleep 1.0.658 is out! first release in over a month, it includes several hooks improvements. This should help to get your hook to run.

metal 1
clj 1
reefersleep13:08:48

Should my tests.edn be like this? I still can’t get :post-summary output.

#kaocha/v1
    {:tests                     [{:id          :unit
                                  :test-paths  ["test"]
                                  :ns-patterns [".*"]}]
     :plugins                   [:kaocha.plugin/hooks]
     :kaocha.hooks/post-summary [util.hooks/blame-report]}

plexus13:08:28

that looks correct, let me try that

reefersleep13:08:41

I tried just :post-summary without the ns, that didn’t work either

plexus13:08:05

you definitely need the ns

plexus14:08:55

bugfix release incoming...

plexus14:08:45

please try 1.0.663, sorry for the hassle

reefersleep15:08:24

Are you kidding, you’ve been a tremendous help 🙏☺️

reefersleep15:08:55

So, :kaocha.hooks/post-summary works when running lein test now, but not in the REPL.

reefersleep15:08:18

That’s great!

reefersleep15:08:24

I’m in doubt about whether you wanted to enable it in the REPL, though?

reefersleep15:08:56

When calling run, that is.

plexus07:08:22

Hmmm yeah that's supposed to work too. I'll have a look to see what's missing.

reefersleep20:08:59

In any case, the blame functionality is working as expected. Thanks a bunch, @U07FP7QJ0!

plexus15:08:33

Cool!!! Would you be able to share the code?

reefersleep16:08:00

Sure 🙂 I could make a repo that does only this, with code annotations and a README that explains how to work with it. Maybe others would find that handy.

reefersleep16:08:13

In that case, did you make is so that :post-summary is available from the REPL? That would remove what would otherwise be a small blemish in the ease of understanding how to work with it.

plexus04:08:05

I thought so. If you're gonna put it in a separate repo then it makes sense to make it a plug-in. I can help with that.

reefersleep13:08:47

Thanks! I’ll try it right away 😄