fix: set-file was not reading file content

pkg/flags/setfile.go

@@ -25,8 +25,8 @@ type SetFile struct {
 	Data *values.Map
 }
 
-func NewSetFile(data *values.Map) SetString {
-	return SetString{Data: data}
+func NewSetFile(data *values.Map) SetFile {
+	return SetFile{Data: data}
 }
 
 func (v *SetFile) Type() string {
@@ -48,6 +48,6 @@ func (v *SetFile) Set(entry string) error {
 		return errors.New(err)
 	}
 
-	v.Data.Set(key, fileText)
+	v.Data.Set(key, string(fileText))
 	return nil
 }

pkg/render/render_test.go

@@ -22,6 +22,7 @@ import (
 	"os"
 	"path"
 	"path/filepath"
+	"strings"
 	"testing"
 )
 
@@ -32,27 +33,39 @@ func TestRenderGeneric(t *testing.T) {
 		templateFile string
 		valuesFile   string
 		includesFile string
+		setFileFlag  string
 		wantErr      error
 	}{
 		{
 			"using-files",
 			"input.yaml",
 			"",
 			"",
+			"",
 			nil,
 		},
 		{
 			"using-helpers",
 			"input.yaml",
 			"",
 			"_helpers.tmpl",
+			"",
 			nil,
 		},
 		{
 			"policies",
 			"apiproxy.yaml",
 			"values.yaml",
 			"",
+			"",
+			nil,
+		},
+		{
+			"set-file",
+			"input.yaml",
+			"",
+			"",
+			"data=./data.json",
 			nil,
 		},
 	}
@@ -87,6 +100,14 @@ func TestRenderGeneric(t *testing.T) {
 				require.NoError(t, err)
 			}
 
+			if tt.setFileFlag != "" {
+				key, filePath, _ := strings.Cut(tt.setFileFlag, "=")
+				tt.setFileFlag = fmt.Sprintf("%s=%s", key, path.Join(testDir, filePath))
+				f := flags.NewSetFile(cFlags.Values)
+				err := f.Set(tt.setFileFlag)
+				require.NoError(t, err)
+			}
+
 			err = RenderGenericTemplate(cFlags, false)
 
 			if tt.wantErr != nil {

pkg/render/testdata/render/set-file/data.json

@@ -0,0 +1,5 @@
+[
+  {
+    "name": "hello world"
+  }
+]

pkg/render/testdata/render/set-file/exp-input.yaml

@@ -0,0 +1,15 @@
+#  Copyright 2024 Google LLC
+#
+#  Licensed under the Apache License, Version 2.0 (the "License");
+#  you may not use this file except in compliance with the License.
+#  You may obtain a copy of the License at
+#
+#       http:#www.apache.org/licenses/LICENSE-2.0
+#
+#  Unless required by applicable law or agreed to in writing, software
+#  distributed under the License is distributed on an "AS IS" BASIS,
+#  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+#  See the License for the specific language governing permissions and
+#  limitations under the License.
+
+name: hello world

pkg/render/testdata/render/set-file/input.yaml

@@ -0,0 +1,16 @@
+#  Copyright 2024 Google LLC
+#
+#  Licensed under the Apache License, Version 2.0 (the "License");
+#  you may not use this file except in compliance with the License.
+#  You may obtain a copy of the License at
+#
+#       http:#www.apache.org/licenses/LICENSE-2.0
+#
+#  Unless required by applicable law or agreed to in writing, software
+#  distributed under the License is distributed on an "AS IS" BASIS,
+#  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+#  See the License for the specific language governing permissions and
+#  limitations under the License.
+
+#{{ $data := index ($.Values.data | fromJson) 0 }}
+name: {{ $data.name }}