This is an automated email from the ASF dual-hosted git repository.
panyuepeng pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/flink.git
The following commit(s) were added to refs/heads/master by this push:
new 6c75724aa4d [FLINK-38982][test] Migrate the JUnit5 for
ClassLoaderUtilsTest of flink-core module (#27477)
6c75724aa4d is described below
commit 6c75724aa4dc3667e01d1d18e6bc8c6f19a4b4c0
Author: Yuepeng Pan <[email protected]>
AuthorDate: Thu Jan 29 09:47:45 2026 +0800
[FLINK-38982][test] Migrate the JUnit5 for ClassLoaderUtilsTest of
flink-core module (#27477)
---
.../flink/testutils/ClassLoaderUtilsTest.java | 26 ++++++++++------------
1 file changed, 12 insertions(+), 14 deletions(-)
diff --git
a/flink-core/src/test/java/org/apache/flink/testutils/ClassLoaderUtilsTest.java
b/flink-core/src/test/java/org/apache/flink/testutils/ClassLoaderUtilsTest.java
index 8e752a89645..a869f6cc764 100644
---
a/flink-core/src/test/java/org/apache/flink/testutils/ClassLoaderUtilsTest.java
+++
b/flink-core/src/test/java/org/apache/flink/testutils/ClassLoaderUtilsTest.java
@@ -18,37 +18,36 @@
package org.apache.flink.testutils;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
import java.util.function.Supplier;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNotEquals;
-import static org.junit.Assert.fail;
+import static org.assertj.core.api.Assertions.assertThat;
+import static org.assertj.core.api.Assertions.fail;
/** Tests for the {@link ClassLoaderUtils}. */
-public class ClassLoaderUtilsTest {
+class ClassLoaderUtilsTest {
@Test
- public void testObjectFromNewClassLoaderObject() throws Exception {
+ void testObjectFromNewClassLoaderObject() {
testObjectFromNewClassLoaderObject(
ClassLoaderUtils::createSerializableObjectFromNewClassLoader);
}
@Test
- public void testObjectFromNewClassLoaderClassLoaders() throws Exception {
+ void testObjectFromNewClassLoaderClassLoaders() {
testObjectFromNewClassLoaderClassLoaders(
ClassLoaderUtils::createSerializableObjectFromNewClassLoader);
}
@Test
- public void testExceptionObjectFromNewClassLoaderObject() throws Exception
{
+ void testExceptionObjectFromNewClassLoaderObject() {
testObjectFromNewClassLoaderObject(
ClassLoaderUtils::createExceptionObjectFromNewClassLoader);
}
@Test
- public void testExceptionObjectFromNewClassLoaderClassLoaders() throws
Exception {
+ void testExceptionObjectFromNewClassLoaderClassLoaders() {
testObjectFromNewClassLoaderClassLoaders(
ClassLoaderUtils::createExceptionObjectFromNewClassLoader);
}
@@ -58,7 +57,7 @@ public class ClassLoaderUtilsTest {
final ClassLoaderUtils.ObjectAndClassLoader<X> objectAndClassLoader =
supplier.get();
final Object o = objectAndClassLoader.getObject();
- assertNotEquals(ClassLoader.getSystemClassLoader(),
o.getClass().getClassLoader());
+
assertThat(ClassLoader.getSystemClassLoader()).isNotEqualTo(o.getClass().getClassLoader());
try {
Class.forName(o.getClass().getName());
@@ -71,9 +70,8 @@ public class ClassLoaderUtilsTest {
Supplier<ClassLoaderUtils.ObjectAndClassLoader<X>> supplier) {
final ClassLoaderUtils.ObjectAndClassLoader<X> objectAndClassLoader =
supplier.get();
- assertNotEquals(ClassLoader.getSystemClassLoader(),
objectAndClassLoader.getClassLoader());
- assertEquals(
- ClassLoader.getSystemClassLoader(),
- objectAndClassLoader.getClassLoader().getParent());
+ assertThat(ClassLoader.getSystemClassLoader())
+ .isNotEqualTo(objectAndClassLoader.getClassLoader())
+ .isEqualTo(objectAndClassLoader.getClassLoader().getParent());
}
}