sweepbatcher: store batch status before monitoring

If monitorConfirmations fails, we still want to persist the state to DB.

Author: starius

sweepbatcher/sweep_batch.go

@@ -2079,16 +2079,20 @@ func (b *batch) handleSpend(ctx context.Context, spendTx *wire.MsgTx) error {
 	b.Infof("spent, confirmed sweeps: %v, purged sweeps: %v, "+
 		"purged swaps: %v", confirmedSweeps, purgedSweeps, purgedSwaps)
 
-	err = b.monitorConfirmations(ctx)
-	if err != nil {
-		return err
-	}
-
 	// We are no longer able to accept new sweeps, so we mark the batch as
 	// closed and persist on storage.
 	b.state = Closed
 
-	return b.persist(ctx)
+	if err := b.persist(ctx); err != nil {
+		return fmt.Errorf("saving batch failed: %w", err)
+	}
+
+	err = b.monitorConfirmations(ctx)
+	if err != nil {
+		return fmt.Errorf("monitorConfirmations failed: %w", err)
+	}
+
+	return nil
 }
 
 // handleConf handles a confirmation notification. This is the final step of the