Add more tests to cover some partial hits

snowflakedb · Oct 19, 2023 · 470e8c0 · 470e8c0
1 parent e5bee13
commit 470e8c0
Show file tree

Hide file tree

Showing 2 changed files with 34 additions and 0 deletions.
diff --git a/src/test/java/net/snowflake/client/core/CoreUtilsMiscellaneousTest.java b/src/test/java/net/snowflake/client/core/CoreUtilsMiscellaneousTest.java
@@ -273,4 +273,27 @@ public void testSizeOfHttpClientMapWithGzipAndUserAgentSuffix() {
     // Assert there are 3 entries because userAgentSuffix has changed
     assertEquals(3, HttpUtil.httpClient.size());
   }
+
+  @Test
+  public void testNullAndEmptyProxySettingsForS3() {
+    HttpClientSettingsKey testKey =
+        new HttpClientSettingsKey(
+            OCSPMode.FAIL_OPEN,
+            null,
+            443,
+            null,
+            null,
+            null,
+            "",
+            "",
+            false);
+    ClientConfiguration clientConfig = new ClientConfiguration();
+    HttpUtil.setProxyForS3(testKey, clientConfig);
+    assertEquals(Protocol.HTTP, clientConfig.getProxyProtocol());
+    assertEquals("", clientConfig.getProxyHost());
+    assertEquals(443, clientConfig.getProxyPort());
+    assertEquals("", clientConfig.getNonProxyHosts());
+    assertNull(clientConfig.getProxyUsername());
+    assertNull(clientConfig.getProxyPassword());
+  }
 }
diff --git a/src/test/java/net/snowflake/client/jdbc/SnowflakeDriverLatestIT.java b/src/test/java/net/snowflake/client/jdbc/SnowflakeDriverLatestIT.java
@@ -416,6 +416,17 @@ public void testGetPropertyInfo() throws SQLException {
         "server URL in form of <protocol>://<host or domain>:<port number>/<path of resource>",
         info[0].description);
 
+    // Test with null URL and no properties. ServerURL is needed.
+    url = null;
+    props = new Properties();
+    driver = DriverManager.getDriver("jdbc:snowflake://snowflake.reg.local:8082");
+    info = driver.getPropertyInfo(url, props);
+    assertEquals(1, info.length);
+    assertEquals("serverURL", info[0].name);
+    assertEquals(
+        "server URL in form of <protocol>://<host or domain>:<port number>/<path of resource>",
+        info[0].description);
+
     // Test with URL that requires username and password.
     url = "jdbc:snowflake://snowflake.reg.local:8082";
     info = driver.getPropertyInfo(url, props);