cider-test-run-tests fails to run tests unless tests are in a -test.clj file

[hugoduncan]

When trying to run clojure.test tests in a namespace that doesn't end in -test, cider-test-run-tests tries to locate the namespace named by the current buffer with -test appended.

This makes it unusable in such buffers. leiningen's tests do not follow this expected naming scheme for example.

[jeffvalk]

The current approach is admittedly imperfect; it just accommodates the most common naming convention. Perhaps non-existence of a -test suffixed namespace should simply cause cider-test to run tests for the current namespace. @hugoduncan Thoughts?

[hugoduncan]

That would work for me, as it just disables the convenience of running tests from an implementation namespace in this case.

[scgilardi]

I think it would be useful to have a config option to specify a way to map to and vice-versa. I can imagine this being done using some kind of pattern string, perhaps a regex. Ideally it would have presets available for common conventions such as the current default and the leiningen default and a way to specify a custom pattern.

[bbatsov]

I think it would be useful to have a config option to specify a way to map to and vice-versa. I can imagine this being done using some kind of pattern string, perhaps a regex. Ideally it would have presets available for common conventions such as the current default and the leiningen default and a way to specify a custom pattern.

Perhaps. We tried to do something similar in clojure-mode at some point, but we decided it was not worth it, as there are some pretty crazy edge cases - you might decide to have several test namespaces for the same namespace or you might use inconsistent convention across your project.

Anyways, we'll think of something.

[jeremyheiler]

+1 for being able to run the tests when in the test namespace.

[bbatsov]

Seems to me the best (and simplest) idea would be simply to prompt the user for the matching namespace and remember it for future test runs.

[bbatsov]

@hugoduncan Turned out we already had a way to customize this - cider-test-infer-test-ns.

[bbatsov]

I've also added prefix arg support for C-c , - with a prefix the command won't do any ns inference and will simply run the tests in the current ns. It's not the most refined solution, but it was an easy fix and solves the problem at hand.

Generally, most people will probably just customize cider-test-infer-test-ns.

[nicolasGuillouet]

ok, fine but how do you customize it in the init.el ? I would like the same behavior as before : running the current test file with C-c , ...

[nicolasGuillouet]

(setq cider-test-infer-test-ns (lambda (ns) ns)) is the best way ?

[bbatsov]

(setq cider-test-infer-test-ns (lambda (ns) ns)) is the best way ?

Looks good to me, although by doing this you won't be able to run the tests from ns they are related to.