Skip to content

Commit 2cb86f2

Browse files
committed
fix(cli): improve temporary directory cleanup in downloader tests
1 parent 9b29474 commit 2cb86f2

File tree

1 file changed

+15
-3
lines changed

1 file changed

+15
-3
lines changed

src/client/acontext-cli/internal/template/downloader_test.go

Lines changed: 15 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,11 @@ func TestReplacePyProjectName(t *testing.T) {
6666
// Create temporary directory
6767
tempDir, err := os.MkdirTemp("", "test-pyproject-*")
6868
require.NoError(t, err)
69-
defer os.RemoveAll(tempDir)
69+
defer func() {
70+
if err := os.RemoveAll(tempDir); err != nil {
71+
t.Logf("Failed to clean up temp dir: %v", err)
72+
}
73+
}()
7074

7175
// Create a test pyproject.toml
7276
pyprojectPath := filepath.Join(tempDir, "pyproject.toml")
@@ -101,7 +105,11 @@ func TestReplacePackageJsonName(t *testing.T) {
101105
// Create temporary directory
102106
tempDir, err := os.MkdirTemp("", "test-package-json-*")
103107
require.NoError(t, err)
104-
defer os.RemoveAll(tempDir)
108+
defer func() {
109+
if err := os.RemoveAll(tempDir); err != nil {
110+
t.Logf("Failed to clean up temp dir: %v", err)
111+
}
112+
}()
105113

106114
// Create a test package.json
107115
packageJsonPath := filepath.Join(tempDir, "package.json")
@@ -131,7 +139,11 @@ func TestReplaceTemplateVars(t *testing.T) {
131139
// Create temporary directory
132140
tempDir, err := os.MkdirTemp("", "test-template-vars-*")
133141
require.NoError(t, err)
134-
defer os.RemoveAll(tempDir)
142+
defer func() {
143+
if err := os.RemoveAll(tempDir); err != nil {
144+
t.Logf("Failed to clean up temp dir: %v", err)
145+
}
146+
}()
135147

136148
// Create test files
137149
pyprojectPath := filepath.Join(tempDir, "pyproject.toml")

0 commit comments

Comments
 (0)