Chore: Improve the missing dates output by wrapping them into square brackets (#3693)

Author: izeigerman

sqlmesh/core/console.py

@@ -889,7 +889,7 @@ def _show_missing_dates(self, plan: Plan, default_catalog: t.Optional[str]) -> N
         missing_intervals = plan.missing_intervals
         if not missing_intervals:
             return
-        backfill = Tree("[bold]Models needing backfill (missing dates):")
+        backfill = Tree("[bold]Models needing backfill \[missing dates]:")
         for missing in missing_intervals:
             snapshot = plan.context_diff.snapshots[missing.snapshot_id]
             if not snapshot.is_model:
@@ -903,7 +903,7 @@ def _show_missing_dates(self, plan: Plan, default_catalog: t.Optional[str]) -> N
                 plan.environment_naming_info, default_catalog, dialect=self.dialect
             )
             backfill.add(
-                f"{display_name}: {_format_missing_intervals(snapshot, missing)}{preview_modifier}"
+                f"{display_name}: \[{_format_missing_intervals(snapshot, missing)}]{preview_modifier}"
             )
 
         if backfill:
@@ -1723,7 +1723,7 @@ def _show_missing_dates(self, plan: Plan, default_catalog: t.Optional[str]) -> N
         missing_intervals = plan.missing_intervals
         if not missing_intervals:
             return
-        self._print("\n**Models needing backfill (missing dates):**")
+        self._print("\n**Models needing backfill \[missing dates]:**")
         snapshots = []
         for missing in missing_intervals:
             snapshot = plan.context_diff.snapshots[missing.snapshot_id]
@@ -1738,7 +1738,7 @@ def _show_missing_dates(self, plan: Plan, default_catalog: t.Optional[str]) -> N
                 plan.environment_naming_info, default_catalog, dialect=self.dialect
             )
             snapshots.append(
-                f"* `{display_name}`: {_format_missing_intervals(snapshot, missing)}{preview_modifier}"
+                f"* `{display_name}`: [{_format_missing_intervals(snapshot, missing)}]{preview_modifier}"
             )
 
         length = len(snapshots)

tests/cli/test_cli.py

@@ -294,8 +294,8 @@ def test_plan_dev_start_date(runner, tmp_path):
         input="\ny\n",
     )
     assert_plan_success(result, "dev")
-    assert "sqlmesh_example__dev.full_model: full refresh" in result.output
-    assert "sqlmesh_example__dev.incremental_model: 2023-01-01" in result.output
+    assert "sqlmesh_example__dev.full_model: [full refresh]" in result.output
+    assert "sqlmesh_example__dev.incremental_model: [2023-01-01" in result.output
 
 
 def test_plan_dev_end_date(runner, tmp_path):
@@ -308,8 +308,8 @@ def test_plan_dev_end_date(runner, tmp_path):
         input="\ny\n",
     )
     assert_plan_success(result, "dev")
-    assert "sqlmesh_example__dev.full_model: full refresh" in result.output
-    assert "sqlmesh_example__dev.incremental_model: 2020-01-01 - 2023-01-01" in result.output
+    assert "sqlmesh_example__dev.full_model: [full refresh]" in result.output
+    assert "sqlmesh_example__dev.incremental_model: [2020-01-01 - 2023-01-01]" in result.output
 
 
 def test_plan_dev_create_from_virtual(runner, tmp_path):

tests/integrations/github/cicd/test_integration.py

@@ -379,8 +379,8 @@ def test_merge_pr_has_non_breaking_change_diff_start(
 - `sushi.top_waiters`
 
 
-**Models needing backfill (missing dates):**
-* `sushi.waiter_revenue_by_day`: 2022-12-25 - 2022-12-28
+**Models needing backfill [missing dates]:**
+* `sushi.waiter_revenue_by_day`: [2022-12-25 - 2022-12-28]
 """
     assert prod_plan_preview_checks_runs[2]["output"]["summary"] == expected_prod_plan
 
@@ -1066,8 +1066,8 @@ def test_no_merge_since_no_deploy_signal_no_approvers_defined(
 - `sushi.top_waiters`
 
 
-**Models needing backfill (missing dates):**
-* `sushi.waiter_revenue_by_day`: 2022-12-25 - 2022-12-29
+**Models needing backfill [missing dates]:**
+* `sushi.waiter_revenue_by_day`: [2022-12-25 - 2022-12-29]
 """
     assert prod_plan_preview_checks_runs[2]["output"]["title"] == "Prod Plan Preview"
     assert prod_plan_preview_checks_runs[2]["output"]["summary"] == expected_prod_plan