RFR: 8376038: Refactor java/sql tests to use JUnit

Johannes Döbler duke at openjdk.org
Thu Jan 22 20:55:23 UTC 2026


On Thu, 22 Jan 2026 14:57:16 GMT, Lance Andersen <lancea at openjdk.org> wrote:

>> Please review this PR which converts the JDBC TestNG tests to use JUnit.
>> 
>> This is mainly done using the automated tool in https://github.com/openjdk/jdk/commit/0cec3097aec02e72ccb6ebbf0b2b046220578d1b, with some manual follow up commits. The testng folder is migrated to junit with the TEST.properties updated as well. Most changes are annotation updates and switching from testNG imports to JUnit. I decided to simplify cases of `BaseTest.trueFalse()` to use booleans in a `ValueSource` directly in https://github.com/openjdk/jdk/commit/757e7966666d39748db2912b32ccf8b1df18bd62.
>> 
>> Framework test stats before:
>> 680 = 680 TestNG + 0 JUnit
>> 
>> Framework test stats after:
>> 680 = 0 TestNG + 680 JUnit
>
> test/jdk/java/sql/junit/test/sql/CallableStatementTests.java line 135:
> 
>> 133:     @Test
>> 134:     public void test08() throws SQLException {
>> 135:         Assertions.assertThrows(NullPointerException.class, () -> cstmt.enquoteNCharLiteral(null));
> 
> Maybe use an import so we can shorten to just assertThrow?

Justin mentioned that the bulk refactoring was done by an automated tool, so maybe this could be moved to that tool too?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29354#discussion_r2718593641


More information about the core-libs-dev mailing list