Add ShardingSphereSchemaTest (#6990)

* Refactor ShardingSphereSchema's constructor

* Move CachedDatabaseMetaDataTest

* Add ShardingSphereSchemaTest

shardingsphere-kernel/shardingsphere-kernel-context/src/main/java/org/apache/shardingsphere/kernel/context/schema/ShardingSphereSchema.java

@@ -18,6 +18,7 @@
 package org.apache.shardingsphere.kernel.context.schema;
 
 import lombok.Getter;
+import lombok.RequiredArgsConstructor;
 import org.apache.shardingsphere.infra.config.RuleConfiguration;
 import org.apache.shardingsphere.infra.metadata.ShardingSphereMetaData;
 import org.apache.shardingsphere.infra.rule.ShardingSphereRule;
@@ -25,32 +26,23 @@ import org.apache.shardingsphere.infra.rule.ShardingSphereRule;
 import javax.sql.DataSource;
 import java.lang.reflect.Method;
 import java.util.Collection;
-import java.util.LinkedHashMap;
-import java.util.LinkedList;
 import java.util.Map;
 
 /**
  * ShardingSphere schema.
  */
+@RequiredArgsConstructor
 @Getter
 public final class ShardingSphereSchema {
     
-    private final Collection<RuleConfiguration> configurations = new LinkedList<>();
+    private final Collection<RuleConfiguration> configurations;
     
-    private final Collection<ShardingSphereRule> rules = new LinkedList<>();
+    private final Collection<ShardingSphereRule> rules;
     
-    private final Map<String, DataSource> dataSources = new LinkedHashMap<>();
+    private final Map<String, DataSource> dataSources;
     
     private final ShardingSphereMetaData metaData;
     
-    public ShardingSphereSchema(final Collection<RuleConfiguration> configurations, final Collection<ShardingSphereRule> rules,
-                                final Map<String, DataSource> dataSourceMap, final ShardingSphereMetaData shardingSphereMetaData) {
-        this.configurations.addAll(configurations);
-        this.rules.addAll(rules);
-        dataSources.putAll(dataSourceMap);
-        metaData = shardingSphereMetaData;
-    }
-    
     /**
      * Close data sources.
      * @param dataSources data sources

shardingsphere-kernel/shardingsphere-kernel-context/src/test/java/org/apache/shardingsphere/kernel/context/schema/ShardingSphereSchemaTest.java

+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.shardingsphere.kernel.context.schema;
+
+import org.apache.shardingsphere.infra.metadata.ShardingSphereMetaData;
+import org.apache.shardingsphere.kernel.context.schema.fixture.ClosableDataSource;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.mockito.Mock;
+import org.mockito.junit.MockitoJUnitRunner;
+
+import javax.sql.DataSource;
+import java.util.Collections;
+import java.util.HashMap;
+import java.util.Map;
+
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.times;
+import static org.mockito.Mockito.verify;
+
+@RunWith(MockitoJUnitRunner.class)
+public final class ShardingSphereSchemaTest {
+    
+    @Mock
+    private ClosableDataSource dataSource0;
+    
+    @Mock
+    private ClosableDataSource dataSource1;
+    
+    @Test
+    public void assertCloseDataSources() throws Exception {
+        new ShardingSphereSchema(Collections.emptyList(), Collections.emptyList(), createDataSources(), mock(ShardingSphereMetaData.class)).closeDataSources(Collections.singleton("ds_0"));
+        verify(dataSource0).close();
+        verify(dataSource1, times(0)).close();
+    }
+    
+    private Map<String, DataSource> createDataSources() {
+        Map<String, DataSource> result = new HashMap<>(2, 1);
+        result.put("ds_0", dataSource0);
+        result.put("ds_1", dataSource1);
+        return result;
+    }
+}

shardingsphere-kernel/shardingsphere-kernel-context/src/test/java/org/apache/shardingsphere/kernel/context/schema/fixture/ClosableDataSource.java

+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.shardingsphere.kernel.context.schema.fixture;
+
+import javax.sql.DataSource;
+
+public interface ClosableDataSource extends DataSource, AutoCloseable {
+}