You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by ji...@apache.org on 2022/07/20 02:27:18 UTC

[shardingsphere] branch master updated: Refactor ShardingSphereDataSourceFactoryTest (#19373)

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 63283d590d3 Refactor ShardingSphereDataSourceFactoryTest (#19373)
63283d590d3 is described below

commit 63283d590d3d35a7578e80e1071df07d64a6186e
Author: Liang Zhang <zh...@apache.org>
AuthorDate: Wed Jul 20 10:27:13 2022 +0800

    Refactor ShardingSphereDataSourceFactoryTest (#19373)
---
 .../api/ShardingSphereDataSourceFactoryTest.java   | 78 ++++++++--------------
 1 file changed, 29 insertions(+), 49 deletions(-)

diff --git a/shardingsphere-jdbc/shardingsphere-jdbc-core/src/test/java/org/apache/shardingsphere/driver/api/ShardingSphereDataSourceFactoryTest.java b/shardingsphere-jdbc/shardingsphere-jdbc-core/src/test/java/org/apache/shardingsphere/driver/api/ShardingSphereDataSourceFactoryTest.java
index 5c75cfea41e..0d8dae9ebd6 100644
--- a/shardingsphere-jdbc/shardingsphere-jdbc-core/src/test/java/org/apache/shardingsphere/driver/api/ShardingSphereDataSourceFactoryTest.java
+++ b/shardingsphere-jdbc/shardingsphere-jdbc-core/src/test/java/org/apache/shardingsphere/driver/api/ShardingSphereDataSourceFactoryTest.java
@@ -35,81 +35,61 @@ import static org.hamcrest.CoreMatchers.is;
 import static org.junit.Assert.assertThat;
 
 public final class ShardingSphereDataSourceFactoryTest {
-
+    
     @Test
     public void assertCreateDataSourceWithModeConfiguration() throws SQLException {
-        DataSource dataSource = ShardingSphereDataSourceFactory.createDataSource(new ModeConfiguration("Standalone", null, false));
-        assertDataSource(dataSource, DefaultDatabase.LOGIC_NAME);
+        assertDataSource(ShardingSphereDataSourceFactory.createDataSource(new ModeConfiguration("Standalone", null, false)), DefaultDatabase.LOGIC_NAME);
     }
-
+    
     @Test
     public void assertCreateDataSourceWithDatabaseNameAndModeConfiguration() throws SQLException {
-        DataSource dataSource = ShardingSphereDataSourceFactory.createDataSource("test_db", new ModeConfiguration("Standalone", null, false), new HashMap<>(), null, null);
-        assertDataSource(dataSource, "test_db");
+        assertDataSource(ShardingSphereDataSourceFactory.createDataSource("test_db", new ModeConfiguration("Standalone", null, false), new HashMap<>(), null, null), "test_db");
     }
-
+    
     @Test
-    public void assertCreateDataSourceWithAllParametersForMultipleDataSourcesWithDefaultDatabaseName()
-            throws SQLException {
-        DataSource dataSource =
-                ShardingSphereDataSourceFactory.createDataSource(new ModeConfiguration("Standalone", null, false),
-                        new HashMap<>(), new LinkedList<>(), new Properties());
-        assertDataSource(dataSource, DefaultDatabase.LOGIC_NAME);
+    public void assertCreateDataSourceWithAllParametersForMultipleDataSourcesWithDefaultDatabaseName() throws SQLException {
+        assertDataSource(ShardingSphereDataSourceFactory.createDataSource(
+                new ModeConfiguration("Standalone", null, false), new HashMap<>(), new LinkedList<>(), new Properties()), DefaultDatabase.LOGIC_NAME);
     }
-
+    
     @Test
     public void assertCreateDataSourceWithAllParametersForMultipleDataSources() throws SQLException {
-        DataSource dataSource =
-                ShardingSphereDataSourceFactory.createDataSource("test_db", new ModeConfiguration("Standalone", null, false),
-                        new HashMap<>(), new LinkedList<>(), new Properties());
-        assertDataSource(dataSource, "test_db");
+        assertDataSource(ShardingSphereDataSourceFactory.createDataSource(
+                "test_db", new ModeConfiguration("Standalone", null, false), new HashMap<>(), new LinkedList<>(), new Properties()), "test_db");
     }
-
+    
     @Test
-    public void assertCreateDataSourceWithAllParametersForSingleDataSourceWithDefaultDatabaseName()
-            throws SQLException {
-        DataSource dataSource =
-                ShardingSphereDataSourceFactory.createDataSource(new ModeConfiguration("Standalone", null, false),
-                        new MockedDataSource(), new LinkedList<>(), new Properties());
-        assertDataSource(dataSource, DefaultDatabase.LOGIC_NAME);
+    public void assertCreateDataSourceWithAllParametersForSingleDataSourceWithDefaultDatabaseName() throws SQLException {
+        assertDataSource(ShardingSphereDataSourceFactory.createDataSource(
+                new ModeConfiguration("Standalone", null, false), new MockedDataSource(), new LinkedList<>(), new Properties()), DefaultDatabase.LOGIC_NAME);
     }
-
+    
     @Test
     public void assertCreateDataSourceWithAllParametersForSingleDataSource() throws SQLException {
-        DataSource dataSource =
-                ShardingSphereDataSourceFactory.createDataSource("test_db", new ModeConfiguration("Standalone", null, false),
-                        new MockedDataSource(), new LinkedList<>(), new Properties());
-        assertDataSource(dataSource, "test_db");
+        assertDataSource(ShardingSphereDataSourceFactory.createDataSource("test_db",
+                new ModeConfiguration("Standalone", null, false), new MockedDataSource(), new LinkedList<>(), new Properties()), "test_db");
     }
-
+    
     @Test
     public void assertCreateDataSourceWithDefaultModeConfigurationForMultipleDataSources() throws SQLException {
-        DataSource dataSource = ShardingSphereDataSourceFactory.createDataSource(null);
-        assertDataSource(dataSource, DefaultDatabase.LOGIC_NAME);
+        assertDataSource(ShardingSphereDataSourceFactory.createDataSource(null), DefaultDatabase.LOGIC_NAME);
     }
-
+    
     @Test
-    public void assertCreateDataSourceWithDatabaseNameAndDefaultModeConfigurationForMultipleDataSources()
-            throws SQLException {
-        DataSource dataSource = ShardingSphereDataSourceFactory.createDataSource("test_db", null);
-        assertDataSource(dataSource, "test_db");
+    public void assertCreateDataSourceWithDatabaseNameAndDefaultModeConfigurationForMultipleDataSources() throws SQLException {
+        assertDataSource(ShardingSphereDataSourceFactory.createDataSource("test_db", null), "test_db");
     }
-
+    
     @Test
     public void assertCreateDataSourceWithDefaultModeConfigurationForSingleDataSource() throws SQLException {
-        DataSource dataSource =
-                ShardingSphereDataSourceFactory.createDataSource((ModeConfiguration) null, new MockedDataSource(), new LinkedList<>(), new Properties());
-        assertDataSource(dataSource, DefaultDatabase.LOGIC_NAME);
+        assertDataSource(ShardingSphereDataSourceFactory.createDataSource((ModeConfiguration) null, new MockedDataSource(), new LinkedList<>(), new Properties()), DefaultDatabase.LOGIC_NAME);
     }
-
+    
     @Test
-    public void assertCreateDataSourceWithDatabaseNameAndDefaultModeConfigurationForSingleDataSource()
-            throws SQLException {
-        DataSource dataSource =
-                ShardingSphereDataSourceFactory.createDataSource("test_db", null, new MockedDataSource(), new LinkedList<>(), new Properties());
-        assertDataSource(dataSource, "test_db");
+    public void assertCreateDataSourceWithDatabaseNameAndDefaultModeConfigurationForSingleDataSource() throws SQLException {
+        assertDataSource(ShardingSphereDataSourceFactory.createDataSource("test_db", null, new MockedDataSource(), new LinkedList<>(), new Properties()), "test_db");
     }
-
+    
     @SneakyThrows(ReflectiveOperationException.class)
     private void assertDataSource(final DataSource actualDataSource, final String expectedDataSourceName) {
         Field field = ShardingSphereDataSource.class.getDeclaredField("databaseName");