mirrors/hermes-agent
1
0
Fork 0
mirror of https://github.com/NousResearch/hermes-agent.git synced 2026-04-25 00:51:20 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

test(approval): clear leaked bypass state

Browse source
This commit is contained in:
Kenny Xie 2026-04-09 22:42:08 -07:00 • committed by Teknium
parent eb8071bbc1
commit ae9a713a0a
1 changed files with 8 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

10
tests/tools/test_approval.py
View file

@ -155,7 +155,7 @@ class TestSessionKeyContext:
assert "set_current_session_key" in called_names
assert "reset_current_session_key" in called_names
def test_context_keeps_pending_approval_attached_to_originating_session(self):
def test_context_keeps_pending_approval_attached_to_originating_session(self, monkeypatch):
import os
import threading
@ -164,6 +164,11 @@ class TestSessionKeyContext:
pop_pending("alice")
pop_pending("bob")
approval_module._permanent_approved.clear()
monkeypatch.delenv("HERMES_YOLO_MODE", raising=False)
monkeypatch.delenv("HERMES_INTERACTIVE", raising=False)
monkeypatch.delenv("HERMES_GATEWAY_SESSION", raising=False)
monkeypatch.delenv("HERMES_EXEC_ASK", raising=False)
monkeypatch.delenv("HERMES_SESSION_KEY", raising=False)
alice_ready = threading.Event()
bob_ready = threading.Event()
@ -190,7 +195,8 @@ class TestSessionKeyContext:
t1 = threading.Thread(target=worker_alice)
t2 = threading.Thread(target=worker_bob)
with mock_patch.dict("os.environ", {"HERMES_GATEWAY_SESSION": "1"}, clear=False):
with mock_patch("tools.approval._get_approval_mode", return_value="manual"), \
mock_patch.dict("os.environ", {"HERMES_GATEWAY_SESSION": "1"}, clear=False):
t1.start()
t2.start()
t1.join()