wehub-resource-sync
b4fbd6fe9f
Deploy Site / deploy-vercel (push) Has been skipped
Deploy Site / deploy-docs (push) Has been skipped
Build Skills Index / build-index (push) Has been skipped
CI / Deny unrelated histories (push) Has been skipped
CI / Detect affected areas (push) Successful in 27m35s
CI / OSV scan (push) Failing after 4s
CI / Build&Test Docker image (push) Successful in 9s
CI / Supply-chain scan (push) Has been skipped
CI / Lint Docker scripts (push) Failing after 5m13s
CI / Check contributors (push) Failing after 12m8s
CI / Docs Site (push) Failing after 12m8s
CI / TypeScript (push) Failing after 12m8s
CI / Python lints (push) Failing after 12m9s
CI / Python tests (push) Failing after 12m9s
CI / Check uv.lock (push) Failing after 23m22s
CI / CI timing report (push) Has been cancelled
Build Skills Index / trigger-deploy (push) Has been cancelled
CI / All required checks pass (push) Has been cancelled
54 lines
1.8 KiB
Python
54 lines
1.8 KiB
Python
"""Unit tests for the verification-loop policy (agent/verify_hooks.py).
|
|
|
|
The `pre_verify` user-hook aggregation lives in `hermes_cli.plugins`
|
|
(`get_pre_verify_continue_message`) and is tested in
|
|
`tests/hermes_cli/test_plugins.py`, alongside `get_pre_tool_call_block_message`.
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
from agent import verify_hooks
|
|
|
|
|
|
class TestMaxVerifyNudges:
|
|
def test_default_when_unset(self):
|
|
assert (
|
|
verify_hooks.max_verify_nudges({})
|
|
== verify_hooks.DEFAULT_MAX_VERIFY_NUDGES
|
|
)
|
|
assert (
|
|
verify_hooks.max_verify_nudges({"agent": {}})
|
|
== verify_hooks.DEFAULT_MAX_VERIFY_NUDGES
|
|
)
|
|
|
|
def test_reads_and_coerces(self):
|
|
assert verify_hooks.max_verify_nudges({"agent": {"max_verify_nudges": 5}}) == 5
|
|
assert verify_hooks.max_verify_nudges({"agent": {"max_verify_nudges": "2"}}) == 2
|
|
assert verify_hooks.max_verify_nudges({"agent": {"max_verify_nudges": -1}}) == 0
|
|
|
|
def test_bad_value_falls_back(self):
|
|
assert (
|
|
verify_hooks.max_verify_nudges({"agent": {"max_verify_nudges": "x"}})
|
|
== verify_hooks.DEFAULT_MAX_VERIFY_NUDGES
|
|
)
|
|
|
|
|
|
class TestCodingVerifyGuidance:
|
|
def test_enabled_by_default(self):
|
|
assert (
|
|
verify_hooks.coding_verify_guidance({})
|
|
== verify_hooks.CODING_VERIFY_GUIDANCE
|
|
)
|
|
assert (
|
|
verify_hooks.coding_verify_guidance({"agent": {}})
|
|
== verify_hooks.CODING_VERIFY_GUIDANCE
|
|
)
|
|
|
|
def test_reads_truthy_config(self):
|
|
cfg = {"agent": {"verify_guidance": "yes"}}
|
|
assert verify_hooks.coding_verify_guidance(cfg) == verify_hooks.CODING_VERIFY_GUIDANCE
|
|
|
|
def test_opt_out_via_config(self):
|
|
off = {"agent": {"verify_guidance": False}}
|
|
assert verify_hooks.coding_verify_guidance(off) is None
|