🧪 Fix flaky daemon stop test (#7319)

Wait for the daemon PID file to disappear before asserting that
`is_daemon_running` is false in `test_daemon_stop`.

The CLI stop command can return before Circus removes the PID
file, which makes an immediate assertion timing-sensitive in CI.
This aligns the test with the existing daemon fixtures, which
already apply the same extra wait to avoid this race.

Reviewed-by: Julian Geiger <julian.geiger@psi.ch>

Author: agoscinski

tests/cmdline/commands/test_daemon.py

@@ -15,7 +15,7 @@
 
 from aiida import get_profile
 from aiida.cmdline.commands import cmd_daemon
-from aiida.engine.daemon.client import DaemonClient
+from aiida.engine.daemon.client import DaemonClient, DaemonTimeoutException
 
 pytestmark = pytest.mark.requires_rmq
 
@@ -100,6 +100,11 @@ def test_daemon_stop(run_cli_command, started_daemon_client):
     """Test ``verdi daemon stop``."""
     result = run_cli_command(cmd_daemon.stop)
 
+    started_daemon_client._await_condition(
+        lambda: not started_daemon_client.is_daemon_running,
+        DaemonTimeoutException('The daemon failed to stop.'),
+    )
+
     assert 'Stopping the daemon' in result.output
     assert not started_daemon_client.is_daemon_running