test: add unit test for cleanSetParameters

Argo-OSS · Sep 21, 2024 · bf6c691 · bf6c691
1 parent 4020700
commit bf6c691
Showing 1 changed file with 19 additions and 0 deletions.
diff --git a/util/helm/cmd_test.go b/util/helm/cmd_test.go
@@ -48,3 +48,22 @@ func TestNewCmd_withProxy(t *testing.T) {
 	assert.Equal(t, "https://proxy:8888", cmd.proxy)
 	assert.Equal(t, ".argoproj.io", cmd.noProxy)
 }
+
+func TestCleanSetParameters(t *testing.T) {
+	val := "{key=value, key2=value2}"
+	result := cleanSetParameters(val)
+	require.NotEmpty(t, result)
+	assert.Equal(t, "{key=value, key2=value2}", result)
+
+	val = "key=value,key2=value2"
+	result = cleanSetParameters(val)
+	assert.Equal(t, "key=value\\,key2=value2", result)
+
+	val = ""
+	result = cleanSetParameters(val)
+	assert.Equal(t, "", result)
+
+	val = ",,,,,,,,value"
+	result = cleanSetParameters(val)
+	assert.Equal(t, "\\,\\,\\,\\,\\,\\,\\,\\,value", result)
+}