Fix airflowctl auth login reporting success when keyring backend is unavailable (apache#61296)

Author: dheerajturaga

airflow-ctl/src/airflowctl/api/client.py

@@ -149,6 +149,9 @@ def save(self):
                 keyring.set_password("airflowctl", f"api_token_{self.api_environment}", self.api_token)
         except NoKeyringError as e:
             log.error(e)
+            raise AirflowCtlKeyringException(
+                "Keyring backend is not available. Cannot save credentials."
+            ) from e
         except TypeError as e:
             # This happens when the token is None, which is not allowed by keyring
             if self.api_token is None and self.client_kind == ClientKind.CLI:

airflow-ctl/tests/airflow_ctl/ctl/commands/test_auth_command.py

@@ -106,3 +106,36 @@ def test_login_with_username_and_password(self, mock_keyring, api_client_maker):
                     mock.call("airflowctl", "api_token_production", "TEST_TOKEN"),
                 ]
             )
+
+    @patch("airflowctl.api.client.keyring")
+    def test_login_with_username_and_password_no_keyring_backend(self, mock_keyring, api_client_maker):
+        """Test that login fails when no keyring backend is available."""
+        from keyring.errors import NoKeyringError
+
+        api_client = api_client_maker(
+            path="/auth/token/cli",
+            response_json=self.login_response.model_dump(),
+            expected_http_status_code=201,
+            kind=ClientKind.AUTH,
+        )
+
+        mock_keyring.set_password.side_effect = NoKeyringError("no backend")
+        with (
+            patch("sys.stdin", io.StringIO("test_password")),
+            patch("airflowctl.ctl.cli_config.getpass.getpass", return_value="test_password"),
+            pytest.raises(SystemExit, match="1"),
+        ):
+            auth_command.login(
+                self.parser.parse_args(
+                    [
+                        "auth",
+                        "login",
+                        "--api-url",
+                        "http://localhost:8080",
+                        "--username",
+                        "test_user",
+                        "--password",
+                    ]
+                ),
+                api_client=api_client,
+            )