garydgregory commented on code in PR #324:
URL: https://github.com/apache/commons-dbutils/pull/324#discussion_r1850246561
##########
src/test/java/org/apache/commons/dbutils/BaseTestCase.java:
##########
@@ -94,25 +98,25 @@ public void setResultSet(final ResultSet resultSet) {
/**
* This is called before each test method so ResultSet will be fresh each
time.
*
- * @see junit.framework.TestCase#setUp()
+ * @see
Review Comment:
This Javadoc tag is now missing its target.
##########
src/test/java/org/apache/commons/dbutils/handlers/ArrayHandlerTest.java:
##########
@@ -16,24 +16,30 @@
*/
package org.apache.commons.dbutils.handlers;
+ import static org.junit.jupiter.api.Assertions.assertEquals;
Review Comment:
Indentation is messed up.
##########
src/test/java/org/apache/commons/dbutils/BasicRowProcessorTest.java:
##########
@@ -16,6 +16,13 @@
*/
package org.apache.commons.dbutils;
+ import static org.junit.jupiter.api.Assertions.assertEquals;
Review Comment:
The formatting looks messed up in this import section.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]