RFR: JDK-8236997 tools/javac tests fail with --illegal-access=deny

[Adam Sotona]

Thank you for the review, all three comments are fixed in the attached patch.

Thanks,
Adam

-------------- next part --------------
A non-text attachment was scrubbed...
Name: JDK-8236997-2.patch
Type: application/octet-stream
Size: 23253 bytes
Desc: not available
URL: <https://mail.openjdk.java.net/pipermail/compiler-dev/attachments/20200124/a38346f6/JDK-8236997-2-0001.patch>
-------------- next part --------------


> On 24 Jan 2020, at 15:04, Jan Lahoda <jan.lahoda at oracle.com> wrote:
> 
> Overall, looks fine to me. A few comments:
> -in test/langtools/tools/javac/classwriter/ExtraAttributes.java, calling .options several times will clear the values set by previous invocations. The plugin here will start as it is an autostart plugin, so I guess I'd suggest to remove the first call to .options, as the first call is not really needed
> 
> -in test/langtools/tools/javac/failover/CheckAttributedTree.java, I would suggest to return back the "java.desktop" after "@modules". It is not needed to compile or run the test itself, but the test is trying to compile all other tests as well, and "java.desktop" will prevent attempts to run the test on limited JDKs without java.desktop (which is expected)
> 
> -I am afraid there is another test using the combo test framework in the jdk (as opposed to langtools) tests:
> test/jdk/java/lang/reflect/Proxy/ProxyGeneratorCombo.java
> 
> That one should receive the same handling as the other combo tests (this is the only test outside of langtools I was able to find).
> 
> Jan
> 
> On 23. 01. 20 17:13, Adam Sotona wrote:
>> Hi,
>> I would like to ask for review of tools/javac tests patch (attached) so the tests work correctly with --illegal-access=deny switch.
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8236997
>> Please let me know if this is OK or if there are any issues with the proposed patch.
>> Thank you,
>> Adam