-
Notifications
You must be signed in to change notification settings - Fork 158
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
[major] [bug] Gradle projects add test compile path in the launcher configuration #507
Comments
Related bug originally reported in eclipse-jdtls/eclipse.jdt.ls#2819 |
I have checked how the Eclipse UI generate the run command. It seems when Exclude test code is used, it uses the output path, bin/default and when Exclude test code is not used, both bin/main and bin/test are added. And in the extension scenario we end using all 3 paths always. When debugging it was found that bin/test comes as part of bin/test/generated-sources/annotations 's output location. Either we can report this to buildship to add support in this scenario to provide correct paths as in Eclise or we can filter the output paths looking and the project raw classpath's output locations. |
Hey @gayanper We recently just released a new tool to support Gradle project - Build Server for Gradle, maybe that can help solve your problem. If you are interested, please follow the steps below:
|
Yes that seems to fix this issue, So I think we can close this if this build-server is the future for gradle projects, But if we are keeping the possibility to use buildship I think we might need to find away to resolve this. |
The long-term plan is to leverage bs approach to support the Gradle project. But since currently we are still in the exploring phase for the bs approach, maybe keep this issue open until we make the decision to use BS to replace Buildship in VS Code. |
Run the attached project, and when checking the arg file, we can see that the
bin/test
is added into class path which cause issues when there are different resource files for testing purpose in applications.micrometer-demo.zip
The text was updated successfully, but these errors were encountered: