Replace 'successfully' with check mark

Author: treysp

sqlmesh/core/console.py

@@ -713,7 +713,7 @@ def stop_evaluation_progress(self, success: bool = True) -> None:
         if self.evaluation_progress_live:
             self.evaluation_progress_live.stop()
             if success:
-                self.log_success(f"{GREEN_CHECK_MARK} Model batches executed successfully")
+                self.log_success(f"{GREEN_CHECK_MARK} Model batches executed")
 
         self.evaluation_progress_live = None
         self.evaluation_total_progress = None
@@ -760,7 +760,7 @@ def stop_creation_progress(self, success: bool = True) -> None:
             self.creation_progress.stop()
             self.creation_progress = None
             if success:
-                self.log_success(f"\n{GREEN_CHECK_MARK} Physical layer updated successfully")
+                self.log_success(f"\n{GREEN_CHECK_MARK} Physical layer updated")
 
         self.environment_naming_info = EnvironmentNamingInfo()
         self.default_catalog = None
@@ -801,7 +801,7 @@ def start_promotion_progress(
         if self.promotion_progress is None:
             self.promotion_progress = Progress(
                 TextColumn(
-                    "[bold blue]Updating virtual layer",
+                    "[bold blue]Updating virtual layer ",  # space to align with other progress bars
                     justify="right",
                 ),
                 BarColumn(bar_width=PROGRESS_BAR_WIDTH),
@@ -838,7 +838,7 @@ def stop_promotion_progress(self, success: bool = True) -> None:
             self.promotion_progress.stop()
             self.promotion_progress = None
             if success:
-                self.log_success(f"\n{GREEN_CHECK_MARK} Virtual layer updated successfully")
+                self.log_success(f"\n{GREEN_CHECK_MARK} Virtual layer updated")
 
         self.environment_naming_info = EnvironmentNamingInfo()
         self.default_catalog = None

tests/cli/test_cli.py

@@ -129,15 +129,15 @@ def assert_new_env(result, new_env="prod", from_env="prod", initialize=True) ->
 
 
 def assert_model_versions_created(result) -> None:
-    assert "Physical layer updated successfully" in result.output
+    assert "Physical layer updated" in result.output
 
 
 def assert_model_batches_evaluated(result) -> None:
-    assert "Model batches executed successfully" in result.output
+    assert "Model batches executed" in result.output
 
 
 def assert_env_views_updated(result) -> None:
-    assert "Virtual layer updated successfully" in result.output
+    assert "Virtual layer updated" in result.output
 
 
 def assert_backfill_success(result) -> None:

tests/integrations/jupyter/test_magics.py

@@ -303,7 +303,7 @@ def test_plan(
     )
     # TODO: Is this what we expect?
     assert text_output[2] == ""
-    assert text_output[3] == "Virtual layer updated successfully"
+    assert text_output[3] == "✔ Virtual layer updated"
     assert convert_all_html_output_to_tags(output) == [
         ["pre", "span"],
         ["pre"] + ["span"] * 4,
@@ -326,7 +326,7 @@ def test_run_dag(
     assert not output.stderr
     assert len(output.outputs) == 2
     assert convert_all_html_output_to_text(output) == [
-        "✔ Model batches executed successfully",
+        "✔ Model batches executed",
         "Run finished for environment 'prod'",
     ]
     assert get_all_html_output(output) == [
@@ -337,7 +337,7 @@ def test_run_dag(
                 h(
                     "span",
                     {"style": SUCCESS_STYLE},
-                    "✔ Model batches executed successfully",
+                    "✔ Model batches executed",
                     autoescape=False,
                 ),
                 autoescape=False,