On Fri, 18 Apr 2025 01:41:36 GMT, Ioi Lam <ik...@openjdk.org> wrote: > > AddopensOption.java only checks if the --add-opens is accepted. It doesn't > check if the option has any effect. I think it's better to run `TEST_MODULE1` > to make sure that `setAccessible` can be called without throwing an > exception. That will verify that the option has achieved the intended effect.
I've modified the AppOpens.java test to include `--add-opens` during dump time. Also needed to change ModuleBootStrap.java slightly for the test to run correctly. The change is to add the call to `addExtraExportsAndOpens(bootLayer)` in the case where the bootLayer is obtained from CDS archive. ------------- PR Comment: https://git.openjdk.org/jdk/pull/24695#issuecomment-2817735297
