Fix #10 - wrong number of args for required

Fixes issue reported at #10:
shipmight · Jan 21, 2024 · 03935e5 · 03935e5
1 parent ec528b4
commit 03935e5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/go/lib/lib.go b/go/lib/lib.go
@@ -130,7 +130,7 @@ func GetYaml(templateYaml string, valuesYaml string) string {
 
 	// If the template contains `required`, we don't want to return an error which
 	// would prevent previewing the entire template. Simply pass the value through.
-	funcMap["required"] = func(val interface{}) (interface{}, error) {
+	funcMap["required"] = func(warn string, val interface{}) (interface{}, error) {
 		return val, nil
 	}
 

diff --git a/go/lib/lib_test.go b/go/lib/lib_test.go
@@ -54,7 +54,7 @@ func TestGetYaml(t *testing.T) {
 			expectedReturnValue: `{"yaml":"name: \n  first: 1\n  second: 2","err":"","warning":""}`,
 		},
 		{
-			templateYaml:        "name: {{ .Values.foobar | required }}",
+			templateYaml:        "name: {{ .Values.foobar | required \".foobar must be set\" }}",
 			valuesYaml:          "",
 			expectedReturnValue: `{"yaml":"name: \u003cno value\u003e","err":"","warning":""}`,
 		},