gradlew/gradlew.bat should be respected

andy-bell101 / neotest-java

Java plugin for Neotest

MIT License

11 stars 4 forks source link

gradlew/gradlew.bat should be respected #8

Closed steffsommer closed 1 year ago

steffsommer commented 1 year ago

Currently the adapter relies on the gradle installation available in PATH. Since Gradle has quite a few breaking changes between major releases, their recommendation is to run the Gradle Wrapper using the (gradlew/gradlew.bat) scripts.

the neotest-java adapter should look for those scripts in the root directory and execute the one appropriate for the OS.

andy-bell101 commented 1 year ago

Hopefully the latest commit should include this functionality. I don't have a windows machine to test on though so your help would be appreciated

andy-bell101 commented 1 year ago

Looks like I spoke too soon. Thought I should add test cases for completeness and I can't get them to pass

andy-bell101 commented 1 year ago

Ok test cases are in now and they pass

steffsommer commented 1 year ago

It works. gradlew.bat is now used. Thanks!