✨ Add support for creating secrets

Author: patrick91

src/fastapi_cloud_cli/commands/env.py

@@ -206,6 +206,13 @@ def set(
             help="A path to the folder containing the app you want to deploy"
         ),
     ] = None,
+    secret: Annotated[
+        bool,
+        typer.Option(
+            "--secret",
+            help="Mark the environment variable as secret (value will be hidden)",
+        ),
+    ] = False,
 ) -> Any:
     """
     Set an environment variable for the app.
@@ -247,6 +254,9 @@ def set(
             assert value is not None
 
             with handle_http_errors(progress):
-                _set_environment_variable(app_config.app_id, name, value)
+                _set_environment_variable(app_config.app_id, name, value, secret)
 
-        toolkit.print(f"Environment variable [bold]{name}[/] set.")
+        if secret:
+            toolkit.print(f"Secret environment variable [bold]{name}[/] set.")
+        else:
+            toolkit.print(f"Environment variable [bold]{name}[/] set.")

tests/test_env_set.py

@@ -47,7 +47,10 @@ def test_shows_a_message_if_app_is_not_configured(logged_in_cli: None) -> None:
 def test_shows_a_message_if_something_is_wrong(
     logged_in_cli: None, respx_mock: respx.MockRouter, configured_app: Path
 ) -> None:
-    respx_mock.post("/apps/123/environment-variables/").mock(return_value=Response(500))
+    respx_mock.post(
+        "/apps/123/environment-variables/",
+        json={"name": "SOME_VAR", "value": "secret", "is_secret": False},
+    ).mock(return_value=Response(500))
 
     with changing_dir(configured_app):
         result = runner.invoke(app, ["env", "set", "SOME_VAR", "secret"])
@@ -63,7 +66,10 @@ def test_shows_a_message_if_something_is_wrong(
 def test_shows_message_when_it_sets(
     logged_in_cli: None, respx_mock: respx.MockRouter, configured_app: Path
 ) -> None:
-    respx_mock.post("/apps/123/environment-variables/").mock(return_value=Response(200))
+    respx_mock.post(
+        "/apps/123/environment-variables/",
+        json={"name": "SOME_VAR", "value": "secret", "is_secret": False},
+    ).mock(return_value=Response(200))
 
     with changing_dir(configured_app):
         result = runner.invoke(app, ["env", "set", "SOME_VAR", "secret"])
@@ -78,7 +84,10 @@ def test_asks_for_name_and_value(
 ) -> None:
     steps = [*"SOME_VAR", Keys.ENTER, *"secret", Keys.ENTER]
 
-    respx_mock.post("/apps/123/environment-variables/").mock(return_value=Response(200))
+    respx_mock.post(
+        "/apps/123/environment-variables/",
+        json={"name": "SOME_VAR", "value": "secret", "is_secret": False},
+    ).mock(return_value=Response(200))
 
     with (
         changing_dir(configured_app),
@@ -94,3 +103,19 @@ def test_asks_for_name_and_value(
     assert "Environment variable SOME_VAR set" in result.output
 
     assert "*" * 6 in result.output
+
+
+@pytest.mark.respx(base_url=settings.base_api_url)
+def test_sets_secret_flag(
+    logged_in_cli: None, respx_mock: respx.MockRouter, configured_app: Path
+) -> None:
+    respx_mock.post(
+        "/apps/123/environment-variables/",
+        json={"name": "SOME_VAR", "value": "secret", "is_secret": True},
+    ).mock(return_value=Response(200))
+
+    with changing_dir(configured_app):
+        result = runner.invoke(app, ["env", "set", "SOME_VAR", "secret", "--secret"])
+
+    assert result.exit_code == 0
+    assert "Secret environment variable SOME_VAR set" in result.output