Fork me on GitHub
#calva
<
2019-01-29
>
pez08:01:26

If anyone saw the questions about the repl window posted here just a minute ago, please report to me and I can zap your memory. You didn't see that, just yet. 😃

😀 5
😂 5
5
claynon15:01:31

I'm migrating from midje to clojure.test and I'm experiencing a problem on running my tests on vscode when I Run all tests it only loads the tests in namespaces that I loaded. Is there a way to load all files in order to be able to run all tests? What worked so far was running refresh-all (from clojure.tools.namespace.repl) and then Run all tests

claynon15:01:10

Is there an easier/smarter way?

claynon15:01:23

on a similar note (but I guess you already know that): Run all tests can't find tests in files that don't match .*test but Run tests from current namespace works

pez15:01:55

I think refresh-all is the best you can do for now. But please report it so that we have a handle to it when we fix it so that Calva does it for you when you run all tests.

metal 5
mseddon16:01:25

@claynon yeah that only testing ns's ending in -test has been addressed in git I think, silly bug introduced by me 🙂

mseddon16:01:07

I think it's possible that cider invokes the nREPL refresh-all op before tests, I will examine it's behaviour and do whatever it does to ensure the tests are loaded first 🙂

mseddon16:01:41

cider has recently changed a lot of how it's test runner works, so I need to look at what's changed

claynon16:01:46

Couldn't this test runner be part of nrepl (or one of the many orchard projects)? Because running tests is a pretty standard feature in an editor and the output of it seems to be standardized and extensible

pez17:01:46

I think @claynon was using the fixed version, @mseddon . At least that's how i interpret the report.

claynon17:01:11

I'm using this last vsix you posted

claynon17:01:18

(I believe)

pez17:01:44

We are using nrepl for running the tests, btw. Not sure if that was the suggestion, but anyway 😀

pez17:01:41

The buggy version didn't run tests at all.

pez17:01:09

Btw, I have forgotten to tell about it here, maybe... The latest Calva on the marketplace is now using the new repl client.

parrot 5
claynon17:01:23

does the marketplace one have all the changes that this latest vsix you posted have?

pez18:01:14

Yes, they are identical.

pez18:01:10

So please start using that and soon there will be coming new vsix’s here with more exciting stuff than “should work exactly as the published one”. 😃

pez18:01:37

It actually works better, but anyway.