(hbase) branch branch-3 updated: HBASE-29898 Addendum remove junit-vintage completely in hbase-replication

[zhangduo]

This is an automated email from the ASF dual-hosted git repository.

zhangduo pushed a commit to branch branch-3
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-3 by this push:
     new 9e70a81ae49 HBASE-29898 Addendum remove junit-vintage completely in 
hbase-replication
9e70a81ae49 is described below

commit 9e70a81ae499b83b6726b3388ed0858a7887fa1f
Author: Duo Zhang <zhang...@apache.org>
AuthorDate: Wed Feb 18 23:15:53 2026 +0800

    HBASE-29898 Addendum remove junit-vintage completely in hbase-replication
    
    (cherry picked from commit b0a649d170a6fe8c2fac93fcf9dcd21a24fdbfd6)
---
 hbase-replication/pom.xml                                      |  5 -----
 .../hbase/replication/ReplicationPeerStorageTestBase.java      | 10 +++++-----
 2 files changed, 5 insertions(+), 10 deletions(-)

diff --git a/hbase-replication/pom.xml b/hbase-replication/pom.xml
index 15c5abe1107..5390bca3294 100644
--- a/hbase-replication/pom.xml
+++ b/hbase-replication/pom.xml
@@ -114,11 +114,6 @@
       <artifactId>junit-jupiter-params</artifactId>
       <scope>test</scope>
     </dependency>
-    <dependency>
-      <groupId>org.junit.vintage</groupId>
-      <artifactId>junit-vintage-engine</artifactId>
-      <scope>test</scope>
-    </dependency>
     <dependency>
       <groupId>org.hamcrest</groupId>
       <artifactId>hamcrest-core</artifactId>
diff --git 
a/hbase-replication/src/test/java/org/apache/hadoop/hbase/replication/ReplicationPeerStorageTestBase.java
 
b/hbase-replication/src/test/java/org/apache/hadoop/hbase/replication/ReplicationPeerStorageTestBase.java
index b596c3290f9..53e16049059 100644
--- 
a/hbase-replication/src/test/java/org/apache/hadoop/hbase/replication/ReplicationPeerStorageTestBase.java
+++ 
b/hbase-replication/src/test/java/org/apache/hadoop/hbase/replication/ReplicationPeerStorageTestBase.java
@@ -19,9 +19,9 @@ package org.apache.hadoop.hbase.replication;
 
 import static 
org.apache.hadoop.hbase.replication.ReplicationPeerConfigTestUtil.assertConfigEquals;
 import static 
org.apache.hadoop.hbase.replication.ReplicationPeerConfigTestUtil.getConfig;
-import static org.junit.Assert.assertThrows;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertThrows;
 import static org.junit.jupiter.api.Assertions.fail;
 
 import java.util.List;
@@ -85,10 +85,10 @@ public abstract class ReplicationPeerStorageTestBase {
   public void testNoSyncReplicationState() throws Exception {
     // This could happen for a peer created before we introduce sync 
replication.
     String peerId = "testNoSyncReplicationState";
-    assertThrows("Should throw a ReplicationException when getting state of 
inexist peer",
-      ReplicationException.class, () -> 
STORAGE.getPeerSyncReplicationState(peerId));
-    assertThrows("Should throw a ReplicationException when getting state of 
inexist peer",
-      ReplicationException.class, () -> 
STORAGE.getPeerNewSyncReplicationState(peerId));
+    assertThrows(ReplicationException.class, () -> 
STORAGE.getPeerSyncReplicationState(peerId),
+      "Should throw a ReplicationException when getting state of inexist 
peer");
+    assertThrows(ReplicationException.class, () -> 
STORAGE.getPeerNewSyncReplicationState(peerId),
+      "Should throw a ReplicationException when getting state of inexist 
peer");
 
     STORAGE.addPeer(peerId, getConfig(0), true, SyncReplicationState.NONE);
     // delete the sync replication state node to simulate