Fork me on GitHub
#cider
<
2020-12-08
>
Kevin F00:12:25

When running cider-test-run-ns-tests I get the message No assertions (or no tests) were run.Did you forget to use 'is' in your tests? unless I load the file with my tests in it. Is this expected behavior? Is there anyway to have cider-test-run-ns-tests load those tests automatically?

Daniel Östling07:12:32

So this is probably already answered, but is there a way to have Cider pick up new dependencies in an ns-require statement and in a leinigen project.clj? Or do I have to restart the Cider process and re-jack-in ?

iarenaza09:12:39

As far as I know, you need to restart the Cider process and re-jack-in (unless you are using pomegranate). There is some work in progress in CIDER to let this work, but as far as I know, there are still some missing pieces.

Daniel Östling10:12:13

Alright, thanks.

kirill.salykin18:12:32

goodmorning/day/evening With the latest changes in c.t.d there are some changes on how the -m is handled and current cider jack-in fails in certain situations for instance i have this test alias

:test {:extra-paths ["test"]
         :extra-deps {lambdaisland/kaocha           {:mvn/version "1.0.690"}
                      lambdaisland/kaocha-cloverage {:mvn/version "1.0.63"}
                      lambdaisland/kaocha-junit-xml {:mvn/version "0.0.76"}
                      ring/ring-mock                {:mvn/version "0.3.2"}
                      mockery                       {:mvn/version "0.1.4"}
                      http-kit.fake/http-kit.fake    {:mvn/version "0.2.1"}}
         :main-opts  ["-m" "kaocha.runner"]}
doing cider-jack-in fails because of multiple -m one from test/main-opts and another from the cider The recommendation from #tools-deps was to define alias for cider and specify -main-opts in the alias:
clojure -Sdeps '{:deps ... :aliases {:cider/jack-in {:main-opts ["-m" ...]}}}' -M:dev:test:cider/jack-in --middleware ...
what do you think about this?

dpsutton18:12:31

https://github.com/clojure-emacs/cider/issues/2941 created for you @kirill.salykin. add any context you feel i missed

👍 3
dpsutton18:12:36

if you could put a minimal repro that would be helpful. obviously those deps but perhaps a single ns with a test? be nice to confirm that your exact situation would work

kirill.salykin18:12:15

I’ll try to do it tomorrow link to github repo is enough? @dpsutton

dpsutton18:12:59

actually i think i can do it

kirill.salykin18:12:34

I am about to go to bed, please let me know how i can help tomorrow

andrea.crotti20:12:31

did the auto-test-mode stopped working some time ago

andrea.crotti20:12:34

or is it just me?

practicalli-johnny23:12:37

Sorry, I obviously didnt understand the issue... seems some of the context is not covered here. I'll look over the issue in more detail.

dpsutton23:12:43

the point of this is to dictate which main function is run. if an alias contains a main function it will be run rather than the cider.nrepl one. the point of this issue is to ensure that the cider.nrepl main always wins the way multiple mains are managed

👍 3
dpsutton23:12:24

its not an external test runner when its just on the classpath.