I think the patch looks good, but it seems the preferred way to make an embedded-only suite has changed to
public static Test suite()
Minor nits (not important to fix, only suggestions on how to reduce the amount of code in case you plan to convert more tests to JUnit):
- I think fail(...) is clearer than assertTrue(..., false)
- it is not necessary to retrieve the connection object so often since BaseJDBCTestCase has helper methods like createStatement(), prepareStatement() and commit().