snuyanzin commented on code in PR #22632:
URL: https://github.com/apache/flink/pull/22632#discussion_r1204256905
##########
flink-runtime/src/test/java/org/apache/flink/runtime/rest/FileUploadHandlerITCase.java:
##########
@@ -56,26 +56,26 @@
import java.util.concurrent.TimeUnit;
import java.util.stream.Collectors;
-import static org.junit.Assert.assertArrayEquals;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertTrue;
-import static org.junit.Assert.fail;
+import static org.assertj.core.api.Assertions.assertThat;
+import static org.assertj.core.api.Assertions.assertThatCode;
/**
* Tests for the {@link FileUploadHandler}. Ensures that multipart http
messages containing files
* and/or json are properly handled.
*/
-public class FileUploadHandlerITCase extends TestLogger {
+class FileUploadHandlerITCase {
Review Comment:
There are several tests in this class with `public` modifier which could be
hardened
--
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]
