From 6ce160a5a0cb6bbc131ac892c1de828505405bc0 Mon Sep 17 00:00:00 2001 From: ethernet Date: Mon, 13 Jul 2026 16:42:27 -0400 Subject: [PATCH] Revert "fix(tests): force manual approval mode in E2E blocking tests" This reverts commit 55624e10b93fc2995be275f8d1919088ede29eab. --- tests/gateway/test_approve_deny_commands.py | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/tests/gateway/test_approve_deny_commands.py b/tests/gateway/test_approve_deny_commands.py index 18eef7b65b26..7976838ef05f 100644 --- a/tests/gateway/test_approve_deny_commands.py +++ b/tests/gateway/test_approve_deny_commands.py @@ -421,18 +421,6 @@ class TestBlockingApprovalE2E: os.environ.pop("HERMES_GATEWAY_SESSION", None) os.environ.pop("HERMES_EXEC_ASK", None) os.environ.pop("HERMES_SESSION_KEY", None) - # Force manual approval mode — these tests exercise the blocking - # gateway flow, not smart approval. The default became "smart" - # (#62661), which routes through _smart_approve() first and - # delays the notify callback past the test's wait window. - self._approval_mode_patch = patch( - "tools.approval._get_approval_config", - return_value={"mode": "manual"}, - ) - self._approval_mode_patch.start() - - def teardown_method(self): - self._approval_mode_patch.stop() def test_blocking_approval_approve_once(self): """check_all_command_guards blocks until resolve_gateway_approval is called."""