ros/autocoder
1
0
Fork 0
mirror of https://github.com/leonvanzyl/autocoder.git synced 2026-01-30 14:22:04 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: rename test_hook to check_hook to fix pytest fixture error

Browse Source 
The test_hook helper function was being incorrectly interpreted by pytest
as a test function due to the 'test_' prefix. Pytest attempted to inject
fixtures for its parameters (command, should_block), causing an error.

Changes:
- Renamed test_hook() to check_hook() in test_security.py
- Updated all call sites (lines 206 and 276)
- Updated docstring to clarify it's a helper function

This fixes the "fixture 'command' not found" error when running pytest.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
Auto
2026-01-10 11:04:36 +02:00
parent b633e6d3f4
commit d5d81919bf
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
test_security.py
View File

@@ -18,8 +18,8 @@ from security import (
)
def test_hook(command: str, should_block: bool) -> bool:
"""Test a single command against the security hook."""
def check_hook(command: str, should_block: bool) -> bool:
"""Check a single command against the security hook (helper function)."""
input_data = {"tool_name": "Bash", "tool_input": {"command": command}}
result = asyncio.run(bash_security_hook(input_data))
was_blocked = result.get("decision") == "block"
@@ -203,7 +203,7 @@ def main():
]
for cmd in dangerous:
if test_hook(cmd, should_block=True):
if check_hook(cmd, should_block=True):
passed += 1
else:
failed += 1
@@ -273,7 +273,7 @@ def main():
]
for cmd in safe:
if test_hook(cmd, should_block=False):
if check_hook(cmd, should_block=False):
passed += 1
else:
failed += 1