Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

clojure.test is assertion highlighting does not work in remote nREPL #1824

Open
thheller opened this issue Aug 2, 2017 · 2 comments
Open

Comments

@thheller
Copy link

thheller commented Aug 2, 2017

When I connect to a remote nREPL (eg. lein repl) and Run test under caret in REPL I get warnings for each is assertion in that test. The warning is printed as red text into the REPL output window.

(deftest something
  (is (= 1 1)))

causes

Error handling response - class java.lang.IllegalArgumentException: Argument for @NotNull parameter 'path' of com/intellij/openapi/vfs/impl/local/LocalFileSystemBase.findFileByPath must not be null

and repeats that warning for each additional is, it does not matter if the is succeeds or not.

The highlighting Cursive usually shows for is does not work.

@TheNewBob
Copy link

I'm having the same issue, except also in a local REPL. The issue suddenly appeared today.

I am pretty sure that this is not a problem of namespace mismatch as reported here: After a while of trying to locate the error I started to check out earlier versions that tested without issue back in the day, and they're suddenly having the same problem. I tried with taged versions of the project as far back as November, and it's still there.

Other projects, however, do not appear to have it, so there does seem to be something project specific, but I can't get a fix on it.

I don't see anything particularly helpful in the logs, but here is an entry anyways (they are all identical):

2017-09-20 15:51:42,927 [1041042]  ERROR -             cursive.repl.nrepl - Error handling nREPL response
java.lang.IllegalArgumentException: Argument for @NotNull parameter 'path' of com/intellij/openapi/vfs/impl/local/LocalFileSystemBase.findFileByPath must not be null
 at com.intellij.openapi.vfs.impl.local.LocalFileSystemBase.findFileByPath (LocalFileSystemBase.java:-1)
    cursive.testing.actions$find_file_and_line.invoke (actions.clj:198)
    cursive.testing.actions/fn (actions.clj:206)
    clojure.lang.MultiFn.invoke (MultiFn.java:229)
    cursive.testing.actions$test_handler$reify__11407.out (actions.clj:280)
    cursive.repl.nrepl$handler.invoke (nrepl.clj:342)
    cursive.repl.nrepl$nrepl_start$fn__3769$fn__3773$fn__3780.invoke (nrepl.clj:403)
    cursive.repl.nrepl$nrepl_start$fn__3769$fn__3773.invoke (nrepl.clj:402)
    clojure.lang.AFn.call (AFn.java:18)
    java.util.concurrent.FutureTask.run (FutureTask.java:266)
    java.util.concurrent.ThreadPoolExecutor.runWorker (ThreadPoolExecutor.java:1142)
    java.util.concurrent.ThreadPoolExecutor$Worker.run (ThreadPoolExecutor.java:617)
    java.lang.Thread.run (Thread.java:745)

I would be glad for any help resolving this issue.

@Procrat
Copy link

Procrat commented Dec 5, 2017

Like I just mentioned in #1258 (comment), this is no longer a problem for me by having upgraded to IntelliJ 2017.3 and Cursive 1.6.2-2017.3. Perhaps this also solves it for you and this issue can potentially be closed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants