Added tests for function parseCommaSeparatedStringMapFromString with …

…valid and invalid JSON arguments
linkedin · Nov 3, 2023 · cec72b6 · cec72b6
1 parent ff7de17
commit cec72b6
Showing 1 changed file with 17 additions and 0 deletions.
diff --git a/internal/venice-common/src/test/java/com/linkedin/venice/utils/UtilsTest.java b/internal/venice-common/src/test/java/com/linkedin/venice/utils/UtilsTest.java
@@ -183,4 +183,21 @@ public void testParseMap() {
   public void testSanitizingStringForLogger() {
     Assert.assertEquals(Utils.getSanitizedStringForLogger(".abc.123."), "_abc_123_");
   }
+
+  @Test
+  public void testParseCommaSeparatedStringMapFromString() {
+    try {
+      Utils.parseCommaSeparatedStringMapFromString(
+          "{\"changeCaptureView\": {\"viewClassName\": \"com.linkedin.venice.views.ChangeCaptureView\",\"params\": {}}}",
+          "someField"); // Method that should not throw an exception
+    } catch (Exception e) {
+      fail("Expected no exception to be thrown, but got: " + e.getMessage());
+    }
+
+    Assert.assertThrows(
+        VeniceException.class,
+        () -> Utils.parseCommaSeparatedStringMapFromString(
+            "{\"changeCaptureView\": {\"viewClassName\": \"com.linkedin.venice.views.ChangeCaptureView\",\"params\":",
+            "someField"));
+  }
 }