mirrors/hermes-agent
1
0
Fork 0
mirror of https://github.com/NousResearch/hermes-agent.git synced 2026-07-29 18:46:59 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 7

test: cover both multiplex URL policy orderings

Browse source
This commit is contained in:
atakan g 2026-07-21 01:22:27 +03:00 • committed by Teknium
parent 7b18de5f40
commit d98ea211f9
2 changed files with 20 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

13
tests/tools/test_browser_ssrf_local.py
View file

@ -369,8 +369,13 @@ class TestAllowPrivateUrlsConfig:
assert browser_tool._allow_private_urls() is False
@pytest.mark.parametrize(
"profile_order",
[("allowed", "blocked"), ("blocked", "allowed")],
ids=["allowed-then-blocked", "blocked-then-allowed"],
)
def test_profile_scoped_config_does_not_reuse_another_profiles_opt_out(
self, tmp_path
self, tmp_path, profile_order
):
"""The browser's independent guard must follow the active profile."""
from hermes_constants import (
@ -396,5 +401,7 @@ class TestAllowPrivateUrlsConfig:
finally:
reset_hermes_home_override(token)
assert under_profile(allowed_home) is True
assert under_profile(blocked_home) is False
homes = {"allowed": allowed_home, "blocked": blocked_home}
expected = {"allowed": True, "blocked": False}
for profile in profile_order:
assert under_profile(homes[profile]) is expected[profile]

13
tests/tools/test_url_safety.py
View file

@ -590,8 +590,13 @@ class TestGlobalAllowPrivateUrls:
monkeypatch.setenv("HERMES_ALLOW_PRIVATE_URLS", "false")
assert _global_allow_private_urls() is True
@pytest.mark.parametrize(
"profile_order",
[("allowed", "blocked"), ("blocked", "allowed")],
ids=["allowed-then-blocked", "blocked-then-allowed"],
)
def test_profile_scoped_config_does_not_reuse_another_profiles_opt_out(
self, tmp_path, monkeypatch
self, tmp_path, monkeypatch, profile_order
):
"""Multiplexed profiles must resolve their own private-URL policy."""
from hermes_constants import (
@ -623,8 +628,10 @@ class TestGlobalAllowPrivateUrls:
finally:
reset_hermes_home_override(token)
assert under_profile(allowed_home) is True
assert under_profile(blocked_home) is False
homes = {"allowed": allowed_home, "blocked": blocked_home}
expected = {"allowed": True, "blocked": False}
for profile in profile_order:
assert under_profile(homes[profile]) is expected[profile]
class TestAllowPrivateUrlsIntegration: