CLI - add warning when refreshing workouts and Open Elevation URL is set

Author: SamR1

fittrackee/tests/workouts/test_workouts_commands.py

@@ -536,7 +536,8 @@ def test_it_displays_message_when_no_workouts_to_refresh(
     ) -> None:
         runner = CliRunner()
 
-        result = runner.invoke(cli, ["workouts", "refresh"])
+        with patch("click.confirm"):
+            result = runner.invoke(cli, ["workouts", "refresh"])
 
         assert result.exit_code == 0
         assert caplog.messages == ["No workouts to refresh.", "\nDone."]
@@ -546,7 +547,8 @@ def test_it_displays_message_when_no_workouts_to_refresh_and_verbose_is_true(  #
     ) -> None:
         runner = CliRunner()
 
-        result = runner.invoke(cli, ["workouts", "refresh", "--verbose"])
+        with patch("click.confirm"):
+            result = runner.invoke(cli, ["workouts", "refresh", "--verbose"])
 
         assert result.exit_code == 0
         assert caplog.messages == ["No workouts to refresh.", "\nDone."]
@@ -651,9 +653,12 @@ def test_it_calls_workouts_refresh_service_whit_default_values(
     ) -> None:
         runner = CliRunner()
 
-        with patch(
-            "fittrackee.workouts.commands.WorkoutsFromFileRefreshService"
-        ) as service_mock:
+        with (
+            patch("click.confirm"),
+            patch(
+                "fittrackee.workouts.commands.WorkoutsFromFileRefreshService"
+            ) as service_mock,
+        ):
             runner.invoke(cli, ["workouts", "refresh"])
 
         service_mock.assert_called_once_with(
@@ -677,6 +682,7 @@ def test_it_calls_workouts_refresh_service_with_all_values(
         runner = CliRunner()
 
         with (
+            patch("click.confirm"),
             patch(
                 "fittrackee.workouts.commands.WorkoutsFromFileRefreshService"
             ) as service_mock,
@@ -729,6 +735,7 @@ def test_it_displays_error_when_service_raises_error(
         runner = CliRunner()
 
         with (
+            patch("click.confirm"),
             patch(
                 "fittrackee.workouts.commands.WorkoutsFromFileRefreshService"
             ) as service_mock,

fittrackee/workouts/commands.py

@@ -260,6 +260,19 @@ def refresh_workouts(
             refresh_logger.setLevel(logging.DEBUG)
             refresh_logger.addHandler(handler)
 
+        if app.config["OPEN_ELEVATION_API_URL"]:
+            click.secho(
+                "\nWarning: Open Elevation API is set.\nIf users have "
+                "enabled missing elevation processing, multiple calls will be "
+                "made to Open Elevation depending on the number of workouts "
+                "to be refreshed, which may result in rate limit errors.",
+                fg="yellow",
+                bold=True,
+            )
+            if not click.confirm("Do you want to continue?"):
+                click.echo("Aborted!")
+                sys.exit(0)
+
         try:
             service = WorkoutsFromFileRefreshService(
                 sport_id=sport_id,