Fix: Improve the condition under which sqlmesh raises an error about snapshot being unrevertable (#1879)

Author: izeigerman

sqlmesh/core/plan/definition.py

@@ -666,6 +666,7 @@ def _ensure_no_forward_only_revert(self) -> None:
             if (
                 candidate.snapshot_id not in self.context_diff.new_snapshots
                 and promoted.is_forward_only
+                and not promoted.is_paused
                 and not candidate.is_forward_only
                 and not candidate.is_indirect_non_breaking
                 and (

tests/core/test_plan.py

@@ -16,7 +16,14 @@
     SnapshotFingerprint,
 )
 from sqlmesh.utils.dag import DAG
-from sqlmesh.utils.date import now, to_date, to_datetime, to_timestamp, yesterday_ds
+from sqlmesh.utils.date import (
+    now,
+    now_timestamp,
+    to_date,
+    to_datetime,
+    to_timestamp,
+    yesterday_ds,
+)
 from sqlmesh.utils.errors import PlanError
 
 
@@ -263,6 +270,7 @@ def test_forward_only_revert_not_allowed(make_snapshot, mocker: MockerFixture):
     forward_only_snapshot = make_snapshot(SqlModel(name="a", query=parse_one("select 2, ds")))
     forward_only_snapshot.categorize_as(SnapshotChangeCategory.FORWARD_ONLY)
     forward_only_snapshot.version = snapshot.version
+    forward_only_snapshot.unpaused_ts = now_timestamp()
     assert forward_only_snapshot.is_forward_only
 
     context_diff_mock = mocker.Mock()