[shardingsphere] branch master updated: Add more assertions of GovernanceFacadeTest.assertInit() (#10355)

[zhangliang]

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

zhangliang 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 d0d3b37  Add more assertions of GovernanceFacadeTest.assertInit() 
(#10355)
d0d3b37 is described below

commit d0d3b3794093b3532d2f4d360f07f47d4f66c724
Author: JooKS <jookunsh...@gmail.com>
AuthorDate: Tue May 18 09:51:44 2021 -0500

    Add more assertions of GovernanceFacadeTest.assertInit() (#10355)
    
    * Add more assertions of GovernanceFacadeTest.assertInit()
    
    * Add more asserts of attributes
    
    * Add a " " to fix build error
---
 .../core/facade/GovernanceFacadeTest.java          | 22 +++++++++++++++++++++-
 1 file changed, 21 insertions(+), 1 deletion(-)

diff --git 
a/shardingsphere-governance/shardingsphere-governance-core/src/test/java/org/apache/shardingsphere/governance/core/facade/GovernanceFacadeTest.java
 
b/shardingsphere-governance/shardingsphere-governance-core/src/test/java/org/apache/shardingsphere/governance/core/facade/GovernanceFacadeTest.java
index af0197b..072b347 100644
--- 
a/shardingsphere-governance/shardingsphere-governance-core/src/test/java/org/apache/shardingsphere/governance/core/facade/GovernanceFacadeTest.java
+++ 
b/shardingsphere-governance/shardingsphere-governance-core/src/test/java/org/apache/shardingsphere/governance/core/facade/GovernanceFacadeTest.java
@@ -34,7 +34,12 @@ import java.util.Collections;
 import java.util.Map;
 import java.util.Properties;
 
+import static org.hamcrest.CoreMatchers.instanceOf;
+import static org.hamcrest.CoreMatchers.is;
+import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertThat;
+import static org.junit.Assert.assertEquals;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.verify;
 
@@ -47,7 +52,15 @@ public final class GovernanceFacadeTest {
         GovernanceConfiguration config = new 
GovernanceConfiguration("test_name", new RegistryCenterConfiguration("TEST", 
"127.0.0.1", new Properties()), false);
         governanceFacade.init(config, Arrays.asList("schema_0", "schema_1"));
         assertNotNull(governanceFacade.getRegistryCenter());
-        // TODO use reflection to assert attributes of GovernanceFacade
+        assertThat(getField(governanceFacade, "isOverwrite"), 
instanceOf(Boolean.class));
+        assertFalse((Boolean) getField(governanceFacade, "isOverwrite"));
+        assertThat(getField(governanceFacade, "registryCenterRepository"), 
instanceOf(RegistryCenterRepository.class));
+        RegistryCenterRepository registryCenterRepository = 
(RegistryCenterRepository) getField(governanceFacade, 
"registryCenterRepository");
+        assertEquals(registryCenterRepository.getType(), "TEST");
+        assertThat(getField(governanceFacade, "listenerManager"), 
instanceOf(GovernanceListenerManager.class));
+        GovernanceListenerManager listenerManager = 
(GovernanceListenerManager) getField(governanceFacade, "listenerManager");
+        assertThat(getField(listenerManager, "registryCenterRepository"), 
is(registryCenterRepository));
+        assertThat(getField(listenerManager, "schemaNames"), 
is(Arrays.asList("schema_0", "schema_1")));
     }
     
     @Test
@@ -83,4 +96,11 @@ public final class GovernanceFacadeTest {
         field.setAccessible(true);
         field.set(target, fieldValue);
     }
+
+    @SneakyThrows(ReflectiveOperationException.class)
+    private static Object getField(final Object target, final String 
fieldName) {
+        Field field = target.getClass().getDeclaredField(fieldName);
+        field.setAccessible(true);
+        return field.get(target);
+    }
 }