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

menghaoran 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 61f05d03ef1 Add more test cases on TableMetaDataNode (#32973)
61f05d03ef1 is described below

commit 61f05d03ef1ada6c151f3967c3276a2169818e80
Author: Liang Zhang <[email protected]>
AuthorDate: Tue Sep 24 13:21:22 2024 +0800

    Add more test cases on TableMetaDataNode (#32973)
---
 .../node/metadata/TableMetaDataNodeTest.java       | 51 ++++++++++++++++------
 1 file changed, 37 insertions(+), 14 deletions(-)

diff --git 
a/kernel/metadata/core/src/test/java/org/apache/shardingsphere/metadata/persist/node/metadata/TableMetaDataNodeTest.java
 
b/kernel/metadata/core/src/test/java/org/apache/shardingsphere/metadata/persist/node/metadata/TableMetaDataNodeTest.java
index abbaf8333f9..7d44d7a3593 100644
--- 
a/kernel/metadata/core/src/test/java/org/apache/shardingsphere/metadata/persist/node/metadata/TableMetaDataNodeTest.java
+++ 
b/kernel/metadata/core/src/test/java/org/apache/shardingsphere/metadata/persist/node/metadata/TableMetaDataNodeTest.java
@@ -23,25 +23,14 @@ import java.util.Optional;
 
 import static org.hamcrest.CoreMatchers.is;
 import static org.hamcrest.MatcherAssert.assertThat;
+import static org.junit.jupiter.api.Assertions.assertFalse;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
 class TableMetaDataNodeTest {
     
     @Test
-    void assertGetTableNameByActiveVersionNode() {
-        Optional<String> actual = 
TableMetaDataNode.getTableNameByActiveVersionNode("/metadata/foo_db/schemas/foo_schema/tables/foo_table/active_version");
-        assertTrue(actual.isPresent());
-        assertThat(actual.get(), is("foo_table"));
-    }
-    
-    @Test
-    void assertIsTableActiveVersionNode() {
-        
assertTrue(TableMetaDataNode.isTableActiveVersionNode("/metadata/foo_db/schemas/foo_schema/tables/foo_table/active_version"));
-    }
-    
-    @Test
-    void assertIsTableNode() {
-        
assertTrue(TableMetaDataNode.isTableNode("/metadata/foo_db/schemas/foo_schema/tables/foo_table"));
+    void assertGetMetaDataTablesNode() {
+        assertThat(TableMetaDataNode.getMetaDataTablesNode("foo_db", 
"foo_schema"), is("/metadata/foo_db/schemas/foo_schema/tables"));
     }
     
     @Test
@@ -63,4 +52,38 @@ class TableMetaDataNodeTest {
     void assertGetTableNode() {
         assertThat(TableMetaDataNode.getTableNode("foo_db", "foo_schema", 
"foo_table"), is("/metadata/foo_db/schemas/foo_schema/tables/foo_table"));
     }
+    
+    @Test
+    void assertGetTableNameByActiveVersionNode() {
+        Optional<String> actual = 
TableMetaDataNode.getTableNameByActiveVersionNode("/metadata/foo_db/schemas/foo_schema/tables/foo_table/active_version");
+        assertTrue(actual.isPresent());
+        assertThat(actual.get(), is("foo_table"));
+    }
+    
+    @Test
+    void assertGetTableNameByActiveVersionNodeIfNotFound() {
+        
assertFalse(TableMetaDataNode.getTableNameByActiveVersionNode("/xxx/foo_db/schemas/foo_schema/tables/foo_table/active_version").isPresent());
+    }
+    
+    @Test
+    void assertGetTableName() {
+        Optional<String> actual = 
TableMetaDataNode.getTableName("/metadata/foo_db/schemas/foo_schema/tables/foo_table");
+        assertTrue(actual.isPresent());
+        assertThat(actual.get(), is("foo_table"));
+    }
+    
+    @Test
+    void assertGetTableNameIfNotFound() {
+        
assertFalse(TableMetaDataNode.getTableName("/xxx/foo_db/schemas/foo_schema/tables/foo_table").isPresent());
+    }
+    
+    @Test
+    void assertIsTableActiveVersionNode() {
+        
assertTrue(TableMetaDataNode.isTableActiveVersionNode("/metadata/foo_db/schemas/foo_schema/tables/foo_table/active_version"));
+    }
+    
+    @Test
+    void assertIsTableNode() {
+        
assertTrue(TableMetaDataNode.isTableNode("/metadata/foo_db/schemas/foo_schema/tables/foo_table"));
+    }
 }

Reply via email to