Add tests & fix logic in edge case

Author: mzitnik

clickhouse-jdbc/src/main/java/com/clickhouse/jdbc/ClickHouseDriver.java

@@ -55,12 +55,15 @@ public boolean isV2(String url) {
             return false;
         }
 
-        if (url != null && url.contains("clickhouse.jdbc.v1")) {
+        if (url != null && url.contains("clickhouse.jdbc.v")) {
             urlFlagSent = true;
 
             if (url.contains("clickhouse.jdbc.v1=true")) {
                 log.info("V1 driver is requested through URL.");
                 return false;
+            } if (url.contains("clickhouse.jdbc.v2=false")) {
+                log.info("V1 driver is requested through URL.");
+                return false;
             } else {
                 log.info("V2 driver is requested through URL.");
                 return true;

clickhouse-jdbc/src/test/java/com/clickhouse/jdbc/ClickHouseDriverTest.java

@@ -28,4 +28,16 @@ public void testConnect() throws SQLException {
         Connection conn = driver.connect("jdbc:clickhouse://" + address, null);
         conn.close();
     }
+    @Test(groups = "integration")
+    public void testV2Driver() {
+        ClickHouseDriver driver = new ClickHouseDriver();
+        Boolean V1 = false;
+        Boolean V2 = true;
+        Assert.assertEquals(driver.isV2("jdbc:clickhouse://localhost:8123"), V2);
+        Assert.assertEquals(driver.isV2("jdbc:clickhouse://localhost:8123?clickhouse.jdbc.v1=true"), V1);
+        Assert.assertEquals(driver.isV2("jdbc:clickhouse://localhost:8123?clickhouse.jdbc.v1=false"), V2);
+        Assert.assertEquals(driver.isV2("jdbc:clickhouse://localhost:8123?clickhouse.jdbc.v2=true"), V2);
+        Assert.assertEquals(driver.isV2("jdbc:clickhouse://localhost:8123?clickhouse.jdbc.v2=false"), V1);
+
+    }
 }