This is an automated email from the ASF dual-hosted git repository.
snuyanzin 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 631e2e2b09b [FLINK-37019][tests] Bump junit5 to 5.11.4
631e2e2b09b is described below
commit 631e2e2b09b2067328c8813d5d22259783ff55f9
Author: Sergey Nuyanzin <snuyan...@gmail.com>
AuthorDate: Sun Jan 12 21:28:50 2025 +0100
[FLINK-37019][tests] Bump junit5 to 5.11.4
---
flink-runtime/pom.xml | 3 ++-
flink-table/flink-sql-gateway/pom.xml | 6 +++++-
flink-table/flink-sql-parser/pom.xml | 6 ++++++
flink-table/flink-table-planner/pom.xml | 6 +++++-
flink-yarn-tests/pom.xml | 6 +++++-
pom.xml | 2 +-
6 files changed, 24 insertions(+), 5 deletions(-)
diff --git a/flink-runtime/pom.xml b/flink-runtime/pom.xml
index 6837595bf4d..5f0063d7bdc 100644
--- 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
index 7a3f9bc918b..6511fd4c6bf 100644
--- 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
index 866fa4f97e3..d355726f777 100644
--- 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
index 2458256fb40..8cc81062099 100644
--- 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
index ce5bfca0df2..a53107346e8 100644
--- 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
index 6bb6ed9ab2c..b04a3aefc03 100644
--- 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>
