feat!: use jdtls compiler

rcasia / neotest-java

Neotest adapter for Java.

MIT License

45 stars 25 forks source link

feat!: use jdtls compiler #153

Closed rcasia closed 1 month ago

rcasia commented 1 month ago

Relates to #75

This is maybe what should have been done in the first place.

Reasons to use jdtls compiler are:

it is faster
it is more reliable
allows neotest-java to have less responsabilities

TODO:

[x] Remove old code
[x] Add dependency to documentation and luarock
[x] Handle when compilation fails (already handled by nvim-jdtls)
[x] Also support debugger
[x] Check dependencies are present
[x] Use java_home from jdtls settings
[x] ability to run tests from the summary window

rcasia commented 1 month ago

Noticed that when running tests that contain reflection (as this one here) throw this errors with this message:

Request processing failed: java.lang.IllegalArgumentException: Name for argument
 of type [int] not specified, and parameter name information not available via reflection. 
Ensure that the compiler uses the '-parameters' flag.

It is solved when adding org.eclipse.jdt.core.compiler.codegen.methodParameters=generate to ./your-project/.settings/org.eclipse.jdt.core.prefs