fix: ACP test collection fails without acp extra; ssh mark unregistered (#779)
Some checks failed
Contributor Attribution Check / check-attribution (pull_request) Failing after 40s
Docker Build and Publish / build-and-push (pull_request) Has been skipped
Supply Chain Audit / Scan PR for supply chain risks (pull_request) Successful in 40s
Tests / e2e (pull_request) Successful in 3m1s
Tests / test (pull_request) Failing after 39m36s

Resolves #779. Two issues fixed:

tests/acp/conftest.py (new):
- Module-level collect_ignore when acp is not installed
- Prevents pytest --collect-only from failing with ModuleNotFoundError
- 6 ACP test files cleanly skipped instead of erroring

tests/conftest.py:
- Registered ssh, integration, slow pytest markers
- Eliminates PytestUnknownMarkWarning

Before: 11,470 collected, 6 errors
After:  11,472 collected, 0 errors
This commit is contained in:
Hermes Agent
2026-04-15 11:09:27 -04:00
parent f1f9bd2e76
commit c94d3cf3bf
2 changed files with 28 additions and 0 deletions

View File

@@ -15,6 +15,12 @@ PROJECT_ROOT = Path(__file__).parent.parent
if str(PROJECT_ROOT) not in sys.path:
sys.path.insert(0, str(PROJECT_ROOT))
# Register custom markers
def pytest_configure(config):
config.addinivalue_line("markers", "ssh: marks tests requiring SSH connectivity")
config.addinivalue_line("markers", "integration: marks integration tests")
config.addinivalue_line("markers", "slow: marks slow tests")
@pytest.fixture(autouse=True)
def _isolate_hermes_home(tmp_path, monkeypatch):
@@ -119,3 +125,5 @@ def _enforce_test_timeout():
yield
signal.alarm(0)
signal.signal(signal.SIGALRM, old)