FIX (restores): Fix order of temp files closing that causes flaky tests

Author: RostislavDugin

backend/internal/features/backups/backups/usecases/postgresql/create_backup_uc.go

@@ -255,10 +255,10 @@ func (uc *CreatePostgresqlBackupUsecase) streamToStorage(
 		copyResultCh <- err
 	}()
 
-	// Wait for the dump and copy to finish
-	waitErr := cmd.Wait()
+	// Wait for the copy to finish first, then the dump process
 	copyErr := <-copyResultCh
 	bytesWritten := <-bytesWrittenCh
+	waitErr := cmd.Wait()
 
 	// Check for shutdown before finalizing
 	if config.IsShouldShutdown() {

backend/internal/features/storages/models/local/model.go

@@ -62,6 +62,12 @@ func (l *LocalStorage) SaveFile(logger *slog.Logger, fileID uuid.UUID, file io.R
 		return fmt.Errorf("failed to sync temp file: %w", err)
 	}
 
+	// Close the temp file explicitly before moving it (required on Windows)
+	if err = tempFile.Close(); err != nil {
+		logger.Error("Failed to close temp file", "fileId", fileID.String(), "error", err)
+		return fmt.Errorf("failed to close temp file: %w", err)
+	}
+
 	finalPath := filepath.Join(config.GetEnv().DataFolder, fileID.String())
 	logger.Debug(
 		"Moving file from temp to final location",