test: fix unawaited coroutine warnings in test suite

Close abandoned coroutines to prevent RuntimeWarning emissions
during garbage collection:

- test_cli_commands.py: Patch async_main in TestMain tests so no
  real coroutine is created when asyncio.run is mocked
- test_tui_actions.py: Close AsyncMock coroutines in
  TestRunCommand tests after guard-clause early returns and
  captured-but-unrun workers
- test_tui_screens.py: Close _do_credential_login coroutine
  passed to mocked run_worker in TestAuthScreen tests

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: deviantintegral

tests/test_cli_commands.py

@@ -1422,6 +1422,7 @@ def test_main_calls_async_main(self):
         with (
             patch("flameconnect.cli.build_parser") as mock_parser_fn,
             patch("flameconnect.cli.asyncio") as mock_asyncio,
+            patch("flameconnect.cli.async_main", new=MagicMock()),
         ):
             mock_parser = MagicMock()
             mock_args = argparse.Namespace(command="list", verbose=False)
@@ -1436,6 +1437,7 @@ def test_main_verbose_logging(self):
         with (
             patch("flameconnect.cli.build_parser") as mock_parser_fn,
             patch("flameconnect.cli.asyncio"),
+            patch("flameconnect.cli.async_main", new=MagicMock()),
             patch("flameconnect.cli.logging") as mock_logging,
         ):
             import logging as real_logging
@@ -1455,6 +1457,7 @@ def test_main_no_verbose_logging(self):
         with (
             patch("flameconnect.cli.build_parser") as mock_parser_fn,
             patch("flameconnect.cli.asyncio"),
+            patch("flameconnect.cli.async_main", new=MagicMock()),
             patch("flameconnect.cli.logging") as mock_logging,
         ):
             import logging as real_logging

tests/test_tui_actions.py

@@ -2308,29 +2308,44 @@ async def test_no_op_when_not_dashboard(self, mock_client):
         non_dashboard = MagicMock(spec=[])
         app = _make_app(mock_client, non_dashboard)
 
-        coro = AsyncMock()()
+        async def _noop():
+            pass
+
+        coro = _noop()
 
         with patch.object(type(app), "screen", new_callable=PropertyMock) as prop:
             prop.return_value = non_dashboard
             app._run_command(coro, "test...", "test failed")
 
         # run_worker should not have been called
         assert len(app._captured_workers) == 0
+        coro.close()
 
     async def test_sets_write_in_progress(self, mock_client, mock_dashboard):
         app = _make_app(mock_client, mock_dashboard)
-        coro = AsyncMock()()
+
+        async def _noop():
+            pass
+
+        coro = _noop()
 
         with patch.object(type(app), "screen", new_callable=PropertyMock) as prop:
             prop.return_value = mock_dashboard
             app._run_command(coro, "test...", "test failed")
 
         assert app._write_in_progress is True
         mock_dashboard.log_message.assert_any_call("test...")
+        coro.close()
+        for w in app._captured_workers:
+            w.close()
 
     async def test_worker_clears_flag_on_success(self, mock_client, mock_dashboard):
         app = _make_app(mock_client, mock_dashboard)
-        coro = AsyncMock()()
+
+        async def _noop():
+            pass
+
+        coro = _noop()
 
         with patch.object(type(app), "screen", new_callable=PropertyMock) as prop:
             prop.return_value = mock_dashboard

tests/test_tui_screens.py

@@ -1246,6 +1246,7 @@ async def test_credential_submit_triggers_worker(self):
                 app.screen.query_one("#sign-in-btn", Button).press()
                 await pilot.pause()
                 mock_worker.assert_called_once()
+                mock_worker.call_args[0][0].close()
 
     async def test_credential_login_success_dismisses(self):
         """Successful credential login dismisses with the redirect URL."""
@@ -1311,10 +1312,11 @@ async def test_password_submitted_triggers_on_submit(self):
         async with app.run_test(size=(80, 25)) as pilot:
             app.screen.query_one("#email-input", Input).value = "user@test.com"
             app.screen.query_one("#password-input", Input).value = "secret"
-            with patch.object(app.screen, "run_worker"):
+            with patch.object(app.screen, "run_worker") as mock_worker:
                 pw_input = app.screen.query_one("#password-input", Input)
                 await pw_input.action_submit()
                 await pilot.pause()
+                mock_worker.call_args[0][0].close()
 
     async def test_button_pressed_non_sign_in_ignored(self):
         """Button press on non sign-in button should be ignored."""