tools/bumper: Remove entire test directory

The tests creates a temporary test directory on every run. Removing it after each test to prevent leftovers. Signed-off-by: Ram Lavi <[email protected]>
RamLavi · Jan 16, 2025 · 09764ad · 09764ad
1 parent 5995dfb
commit 09764ad
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/tools/bumper/cnao_repo_commands_test.go b/tools/bumper/cnao_repo_commands_test.go
@@ -30,7 +30,7 @@ var _ = Describe("Testing internal git CNAO Repo", func() {
 		gitCnaoRepo = newFakeGitCnaoRepo(githubApi, repoDir, &component{}, expectedTagCommitMap)
 
 		DeferCleanup(func() {
-			Expect(os.RemoveAll(gitCnaoRepo.gitRepo.localDir)).To(Succeed())
+			Expect(os.RemoveAll(tempDir)).To(Succeed())
 		})
 	})
 

diff --git a/tools/bumper/component_commands_test.go b/tools/bumper/component_commands_test.go
@@ -28,6 +28,7 @@ var _ = Describe("Testing internal git component", func() {
 
 		DeferCleanup(func() {
 			Expect(os.RemoveAll(gitComponent.gitRepo.localDir)).To(Succeed())
+			Expect(os.RemoveAll(tempDir)).To(Succeed())
 		})
 	})
-Original file line number
+Diff line change
@@ Expand Up / @@ -28,6 +28,7 @@ var _ = Describe("Testing internal git component", func() { @@
     		DeferCleanup(func() {
     			Expect(os.RemoveAll(gitComponent.gitRepo.localDir)).To(Succeed())
+    			Expect(os.RemoveAll(tempDir)).To(Succeed())
     		})
     	})
@@ Expand Down @@