jingshanglu commented on code in PR #19445:
URL: https://github.com/apache/shardingsphere/pull/19445#discussion_r928355901


##########
shardingsphere-infra/shardingsphere-infra-common/src/test/java/org/apache/shardingsphere/infra/datasource/props/DataSourcePropertiesValidatorTest.java:
##########
@@ -29,7 +31,12 @@ public final class DataSourcePropertiesValidatorTest {
     
     @Test
     public void assertValidateSuccess() throws InvalidResourcesException {
-        new 
DataSourcePropertiesValidator().validate(Collections.singletonMap("name", new 
DataSourceProperties(HikariDataSource.class.getName(), 
createValidProperties())));
+        new 
DataSourcePropertiesValidator().validate(Collections.singletonMap("name", new 
DataSourceProperties(HikariDataSource.class.getName(), 
createValidProperties())), new H2DatabaseType());
+    }
+    
+    @Test(expected = InvalidResourcesException.class)
+    public void assertDatabaseTypeInValidateFail() throws 
InvalidResourcesException {
+        new 
DataSourcePropertiesValidator().validate(Collections.singletonMap("name", new 
DataSourceProperties(HikariDataSource.class.getName(), 
createValidProperties())), new MySQLDatabaseType());

Review Comment:
   This is UT about the logic.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to