Pass fixtures via the absolute path to address new filepath-securejoi…

…n library constraint (#3402) more details https://github.com/cyphar/filepath-securejoin/releases/tag/v0.4.1
cloudfoundry · Feb 13, 2025 · 5f20bec · 5f20bec
1 parent 51a29ac
commit 5f20bec
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/cf/appfiles/app_files_test.go b/cf/appfiles/app_files_test.go
@@ -108,7 +108,8 @@ var _ = Describe("AppFiles", func() {
 
 	Describe("CopyFiles", func() {
 		It("copies only the files specified", func() {
-			copyDir := filepath.Join(fixturePath, "app-copy-test")
+			copyDir, err := filepath.Abs(filepath.Join(fixturePath, "app-copy-test"))
+			Expect(err).NotTo(HaveOccurred())
 
 			filesToCopy := []models.AppFileFields{
 				{Path: filepath.Join("dir1")},
@@ -117,7 +118,8 @@ var _ = Describe("AppFiles", func() {
 
 			files := []string{}
 
-			fileutils.TempDir("copyToDir", func(tmpDir string, err error) {
+			fileutils.TempDir("copyToDir", func(tmpDir string, tmpErr error) {
+				Expect(tmpErr).NotTo(HaveOccurred())
 				copyErr := appFiles.CopyFiles(filesToCopy, copyDir, tmpDir)
 				Expect(copyErr).ToNot(HaveOccurred())