Skip to content

fix: add module opens for all selected test classes#1885

Merged
wenytang-ms merged 2 commits into
mainfrom
fix/1883-multi-package-add-opens-v2
Jul 13, 2026
Merged

fix: add module opens for all selected test classes#1885
wenytang-ms merged 2 commits into
mainfrom
fix/1883-multi-package-add-opens-v2

Conversation

@wenytang-ms

Copy link
Copy Markdown
Contributor

Summary

  • pass JDT handles for multi-class JUnit launches
  • let the upstream JDT delegate evaluate the exact selected types so it generates --add-opens for every package
  • preserve the fast single-main-type launch setup and add a modular multi-package regression test

Testing

  • npm test
  • mvn -f java-extension\pom.xml -pl com.microsoft.java.test.target,com.microsoft.java.test.plugin,com.microsoft.java.test.plugin.test -am verify

Fixes #1883

Pass selected class JDT handles to the launch delegate so Eclipse JDT can generate VM arguments for every test package without rediscovering the project.

Fixes #1883

Copilot-Session: c3cd113b-dad4-4d4d-90c2-60abfd204405
Copilot-Session: c3cd113b-dad4-4d4d-90c2-60abfd204405
@wenytang-ms

Copy link
Copy Markdown
Contributor Author

before
image

after
image

@wenytang-ms wenytang-ms merged commit 56feff7 into main Jul 13, 2026
14 checks passed
@wenytang-ms wenytang-ms deleted the fix/1883-multi-package-add-opens-v2 branch July 13, 2026 06:31
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

Successfully merging this pull request may close these issues.

--add-opens VM args incomplete for modular projects when running a multi-class test suite

2 participants