fix(tests): drop the unread detect_hardline_command patch in test_shell_exec_security
tui_gateway.server never reads that name (the hardline check lives in methods_tools via the approval import); also add the trailing newline.
This commit is contained in:
@@ -24,7 +24,6 @@ def test_shell_exec_scrubs_child_env_and_force_redacts_rpc_output(monkeypatch):
|
||||
monkeypatch.setenv("OPENROUTER_API_KEY", secret)
|
||||
monkeypatch.setattr(redact, "_REDACT_ENABLED", False)
|
||||
monkeypatch.setattr(server.subprocess, "run", fake_run)
|
||||
monkeypatch.setattr(server, "detect_hardline_command", lambda _command: (False, ""), raising=False)
|
||||
|
||||
response = server.handle_request(
|
||||
{"id": "security", "method": "shell.exec", "params": {"command": "echo safe"}}
|
||||
@@ -33,4 +32,4 @@ def test_shell_exec_scrubs_child_env_and_force_redacts_rpc_output(monkeypatch):
|
||||
assert "OPENROUTER_API_KEY" not in captured["env"]
|
||||
assert secret not in response["result"]["stdout"]
|
||||
assert secret not in response["result"]["stderr"]
|
||||
assert "visible" in response["result"]["stdout"]
|
||||
assert "visible" in response["result"]["stdout"]
|
||||
|
||||
Reference in New Issue
Block a user