No, did you check if your test file is part of the source-paths?

i used deps.edn: it’s in :extra-paths in my :test alias. should I move it up to :paths ?

No, clojure-lsp should resolve that, can you double check the clojure-lsp server log?

It should say: auto-resolved source paths xyz

Automatically resolved source-paths from deps.edn: #{"src" "resources"}

oh, my test alias is :test/cognitect. lemme try if it’s just :test

Yeah, you can set source-aliases to fix that

https://clojure-lsp.github.io/clojure-lsp/settings/#source-paths-discovery

is :source-aliases a map or a set?

Set or vec

cool

LMK if you see anything to improve that on server side

But I think there is no way we find :test/cognitect is your test alias automatically

i’m using coc.nvim, so the config is written in json. i’ve written the alias in an array: [":test/cognitect", ":test/kaocha"], and I can see that it’s sent in the initialize call from coc.nvim:

"initializationOptions": {
        "text-document-sync-kind": ":incremental",
        "use-metadata-for-privacy?": true,
        "ignore-classpath-directories": true,
        "source-aliases": [
            ":test/cognitect",
            ":test/kaocha"
        ]
    },

but then it logs: Automatically resolved source-paths from deps.edn: #{"src" "resources"}

I'd suggest not configure via initializationOptions, but via .lsp/config.edn

do they work different?

Not really, but since it's a json, I don't know it will behave the parsing on vim client

hm okay

maybe i can take a look at how clojure-lsp processes this

thanks, i’ll try that out

Alright, lmk if it works

Ah, try removing the : inside the string

which part inside the string? the : is already inside the alias string

Try: "source-aliases": ["test/cognitect"]

doesn’t work. putting the keywords into .lsp/config.edn works great tho

Good to know, it's the first time I see initializationOptions not behaving similar to .lsp/config.edn

https://github.com/clojure-lsp/clojure-lsp/pull/516

🚀

i’ll get some tests written!

i’ve updated the docs to make note of this as well