[FLINK-37019][tests] Bump junit5 to 5.11.4

apache · Jan 12, 2025 · 631e2e2 · 631e2e2
1 parent 78ea6ec
commit 631e2e2
Show file tree

Hide file tree

Showing 6 changed files with 24 additions and 5 deletions.
diff --git a/flink-runtime/pom.xml b/flink-runtime/pom.xml
@@ -50,7 +50,8 @@ under the License.
 			OperatorStateBackendTest
 			-->--add-opens=java.base/java.io=ALL-UNNAMED <!--
 			AsynchronousFileIOChannelTest
-			-->--add-opens=java.base/java.util.concurrent=ALL-UNNAMED
+			-->--add-opens=java.base/java.util.concurrent=ALL-UNNAMED <!--
+			-->-Djunit.platform.reflection.search.useLegacySemantics=true
 		</surefire.module.config>
 	</properties>
 

diff --git a/flink-table/flink-sql-gateway/pom.xml b/flink-table/flink-sql-gateway/pom.xml
@@ -39,7 +39,11 @@
 			SqlGatewayTest/CommonTestUtils.setEnv
 			-->--add-opens=java.base/java.util=ALL-UNNAMED <!--
 			OperationManager#getThreadInFuture
-			-->--add-opens=java.base/java.util.concurrent=ALL-UNNAMED
+			-->--add-opens=java.base/java.util.concurrent=ALL-UNNAMED <!--
+			SqlGatewayRestEndpointStatementITCase has to override AbstractSqlGatewayStatementITCase methods
+			JUnit changed this behavior since 5.11.x, to restore it back need this flag as mentioned at
+			https://junit.org/junit5/docs/current/user-guide/#extensions-supported-utilities-search-semantics
+			-->-Djunit.platform.reflection.search.useLegacySemantics=true
 		</surefire.module.config>
 	</properties>
 

diff --git a/flink-table/flink-sql-parser/pom.xml b/flink-table/flink-sql-parser/pom.xml
@@ -35,6 +35,12 @@ under the License.
 	<properties>
 		<!-- override parent pom -->
 		<test.excludedGroups/>
+		<surefire.module.config><!--
+			FlinkSqlParserImplTest has to override Calcite's SqlParserTest
+			JUnit changed this behavior since 5.11.x, to restore it back need this flag as mentioned at
+			https://junit.org/junit5/docs/current/user-guide/#extensions-supported-utilities-search-semantics
+			-->-Djunit.platform.reflection.search.useLegacySemantics=true
+		</surefire.module.config>
 	</properties>
 
 	<dependencies>

diff --git a/flink-table/flink-table-planner/pom.xml b/flink-table/flink-table-planner/pom.xml
@@ -50,7 +50,11 @@ under the License.
 			kryo MathContext
 			-->--add-opens=java.base/java.math=ALL-UNNAMED <!--
 			kryo ByteBuffer
-			-->--add-opens=java.base/java.nio=ALL-UNNAMED
+			-->--add-opens=java.base/java.nio=ALL-UNNAMED <!--
+			Multiple tests have to override TableTestBase methods and others
+			JUnit changed this behavior since 5.11.x, to restore it back need this flag as mentioned at
+			https://junit.org/junit5/docs/current/user-guide/#extensions-supported-utilities-search-semantics
+			-->-Djunit.platform.reflection.search.useLegacySemantics=true
 		</surefire.module.config>
 	</properties>
 

diff --git a/flink-yarn-tests/pom.xml b/flink-yarn-tests/pom.xml
@@ -44,7 +44,11 @@ under the License.
 			MiniYARNCluster
 			-->--add-opens=java.base/java.lang=ALL-UNNAMED <!--
 			YARNSessionFIFOSecuredITCase
-			-->--add-exports=java.security.jgss/sun.security.krb5=ALL-UNNAMED
+			-->--add-exports=java.security.jgss/sun.security.krb5=ALL-UNNAMED <!--
+			YARNSessionCapacitySchedulerITCase has to override YarnTestBase methods
+			JUnit changed this behavior since 5.11.x, to restore it back need this flag as mentioned at
+			https://junit.org/junit5/docs/current/user-guide/#extensions-supported-utilities-search-semantics
+			-->-Djunit.platform.reflection.search.useLegacySemantics=true
 		</surefire.module.config>
 	</properties>
 

diff --git a/pom.xml b/pom.xml
@@ -149,7 +149,7 @@ under the License.
 		<javax.activation.api.version>1.2.0</javax.activation.api.version>
 		<jaxb.api.version>2.3.1</jaxb.api.version>
 		<junit4.version>4.13.2</junit4.version>
-		<junit5.version>5.10.1</junit5.version>
+		<junit5.version>5.11.4</junit5.version>
 		<archunit.version>1.2.0</archunit.version>
 		<mockito.version>5.14.2</mockito.version>
 		<hamcrest.version>1.3</hamcrest.version>