(shardingsphere) branch master updated: Refactor IndexReviseEngineTest (#30904)

[jianglongtao]

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

jianglongtao pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/shardingsphere.git


The following commit(s) were added to refs/heads/master by this push:
     new ff0da07d640 Refactor IndexReviseEngineTest (#30904)
ff0da07d640 is described below

commit ff0da07d640860b1c0668d5b22c361e0160f6aae
Author: yx9o <yangx_s...@163.com>
AuthorDate: Tue Apr 16 00:05:33 2024 +0800

    Refactor IndexReviseEngineTest (#30904)
---
 .../schema/reviser/index/IndexReviseEngineTest.java         | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git 
a/infra/common/src/test/java/org/apache/shardingsphere/infra/metadata/database/schema/reviser/index/IndexReviseEngineTest.java
 
b/infra/common/src/test/java/org/apache/shardingsphere/infra/metadata/database/schema/reviser/index/IndexReviseEngineTest.java
index 85a486ec79d..4c79f9c4d53 100644
--- 
a/infra/common/src/test/java/org/apache/shardingsphere/infra/metadata/database/schema/reviser/index/IndexReviseEngineTest.java
+++ 
b/infra/common/src/test/java/org/apache/shardingsphere/infra/metadata/database/schema/reviser/index/IndexReviseEngineTest.java
@@ -18,10 +18,8 @@
 package org.apache.shardingsphere.infra.metadata.database.schema.reviser.index;
 
 import 
org.apache.shardingsphere.infra.database.core.metadata.data.model.IndexMetaData;
-import org.apache.shardingsphere.infra.rule.ShardingSphereRule;
 import 
org.apache.shardingsphere.infra.metadata.database.schema.reviser.MetaDataReviseEntry;
-
-import org.junit.jupiter.api.Assertions;
+import org.apache.shardingsphere.infra.rule.ShardingSphereRule;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.mockito.InjectMocks;
@@ -37,18 +35,16 @@ import static org.hamcrest.CoreMatchers.equalTo;
 import static org.hamcrest.CoreMatchers.equalToObject;
 import static org.hamcrest.CoreMatchers.is;
 import static org.hamcrest.MatcherAssert.assertThat;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.ArgumentMatchers.anyString;
-import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.doReturn;
+import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.when;
 
 class IndexReviseEngineTest<T extends ShardingSphereRule> {
     
-    @Mock
-    private T rule;
-    
     @Mock
     private MetaDataReviseEntry<T> metaDataReviseEntry;
     
@@ -65,7 +61,7 @@ class IndexReviseEngineTest<T extends ShardingSphereRule> {
         when(metaDataReviseEntry.getIndexReviser(any(), 
anyString())).thenReturn(Optional.empty());
         Collection<IndexMetaData> indexMetaDataCollection = 
Collections.singletonList(new IndexMetaData("index"));
         Collection<IndexMetaData> actual = 
indexReviseEngine.revise("tableName", indexMetaDataCollection);
-        Assertions.assertNotNull(actual);
+        assertNotNull(actual);
         assertThat(actual.size(), is(1));
         assertThat(actual, equalToObject(indexMetaDataCollection));
     }
@@ -81,5 +77,4 @@ class IndexReviseEngineTest<T extends ShardingSphereRule> {
         assertThat(actual.size(), equalTo(1));
         assertTrue(actual.contains(indexMetaData));
     }
-    
 }