Always close the database whether there is error when initializing

Author: pan93412

lib/sqlrunner.go

@@ -254,6 +254,11 @@ func initialize(schema string) (filename string, err error) {
 	if err != nil {
 		return "", fmt.Errorf("open sqlite: %w", err)
 	}
+	defer func() {
+		if err := drv.Close(); err != nil {
+			slog.Warn("close sqlite", slog.Any("error", err))
+		}
+	}()
 
 	if _, err := drv.Exec("PRAGMA foreign_keys = ON;"); err != nil {
 		return "", fmt.Errorf("enable foreign keys: %w", err)
@@ -263,10 +268,6 @@ func initialize(schema string) (filename string, err error) {
 		return "", NewSchemaError(err)
 	}
 
-	if err := drv.Close(); err != nil {
-		return "", fmt.Errorf("close sqlite: %w", err)
-	}
-
 	// Rename the file to the final name
 	if err := os.Rename(schemaFilename+".tmp", schemaFilename); err != nil {
 		return "", fmt.Errorf("persistent schema: %w", err)