tests: add missing close

when writing tmp file.
ConSol-Monitoring · Jan 30, 2025 · f9685f2 · f9685f2
1 parent 88b7ab5
commit f9685f2
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/pkg/snclient/config.go b/pkg/snclient/config.go
@@ -130,9 +130,15 @@ func (config *Config) WriteINI(iniPath string) error {
 	if err != nil {
 		return fmt.Errorf("failed to write ini %s: %s", iniPath, err.Error())
 	}
-	defer file.Close()
 
 	_, err = file.WriteString(configData)
+	if err != nil {
+		LogDebug(file.Close())
+
+		return fmt.Errorf("failed to write ini %s: %s", iniPath, err.Error())
+	}
+
+	err = file.Close()
 	if err != nil {
 		return fmt.Errorf("failed to write ini %s: %s", iniPath, err.Error())
 	}

diff --git a/pkg/snclient/snclient_test.go b/pkg/snclient/snclient_test.go
@@ -72,6 +72,8 @@ use ssl = false
 password = test
 `)
 	require.NoErrorf(t, err, "tmp include created")
+	err = tmpInclude.Close()
+	require.NoErrorf(t, err, "tmp include created")
 
 	snc := StartTestAgent(t, config)