RFR: JDK-8183372 : Refactor java/lang/Class shell tests to java

Brent Christian bchristi at openjdk.java.net
Fri Jan 22 20:27:40 UTC 2021


On Fri, 22 Jan 2021 16:57:41 GMT, Mahendra Chhipa <github.com+34924738+mahendrachhipa at openjdk.org> wrote:

>> Can this be done all in `EnclosingClassTest.java`, without a new `RunEnclosingClassTest.java`?
>> 
>> Adding the `@BeforeClass` and `@AfterClass` methods to what's there, you may just need to 
>> change the `test()` calls to use reflection - something along the lines of: 
>> 
>>> test(`Class.forName("EnclosingClass").getDeclaredConstructor().newInstance());`
>
> Review comments implemented.

Can the new code be added to the existing `NonJavaNames.java` instead of adding a new `NonJavaNameTest.java` file?

-------------

PR: https://git.openjdk.java.net/jdk/pull/2170


More information about the core-libs-dev mailing list