Merge pull request #7 from racklet/quote_output

Quote the files when output

Dockerfile

@@ -1,6 +1,8 @@
 FROM golang:1.16-alpine as builder
 WORKDIR /build
 COPY . .
+# Make sure tests pass before building :)
+RUN CGO_ENABLED=0 go test ./...
 RUN CGO_ENABLED=0 go build -a -o render-drawio ./cmd/render-drawio
 
 FROM rlespinasse/drawio-desktop-headless:1.0.1

pkg/render/render.go

@@ -257,6 +257,13 @@ func GitHubActionSetFilesOutput(key, rootDir string, files []string) error {
 		}
 	}
 	// Run the generic function
-	GitHubActionSetOutput(key, strings.Join(files, " "))
+	GitHubActionSetOutput(key, joinQuotedStrings(files))
 	return nil
 }
+
+func joinQuotedStrings(strs []string) string {
+	for i := range strs {
+		strs[i] = fmt.Sprintf("%q", strs[i])
+	}
+	return strings.Join(strs, " ")
+}

pkg/render/render_test.go

@@ -0,0 +1,24 @@
+package render
+
+import "testing"
+
+func Test_joinQuotedStrings(t *testing.T) {
+	tests := []struct {
+		name string
+		strs []string
+		want string
+	}{
+		{
+			name: "simple",
+			strs: []string{"foo bar", "bar baz"},
+			want: `"foo bar" "bar baz"`,
+		},
+	}
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			if got := joinQuotedStrings(tt.strs); got != tt.want {
+				t.Errorf("joinQuotedStrings() = %v, want %v", got, tt.want)
+			}
+		})
+	}
+}