Simplify-code findings: - _authenticated_json: replace manual header construction with self._headers(include_tenant=False) — eliminates duplication with _headers() and includes Content-Type consistently. - _health_requires_credentials: replace getattr(exc, 'status_code') with _status_code_from_error(exc) for consistency with the existing error-classification utility. Drop the fragile string-matching fallback — _parse_response always sets status_code on _OpenVikingHTTPError, so 401/403 check is sufficient. - Relax test header assertions to check presence/absence of specific headers rather than exact dict equality, so they survive the header-construction refactor. |
||
|---|---|---|
| .. | ||
| __init__.py | ||
| test_byterover_provider.py | ||
| test_config_schema.py | ||
| test_hindsight_config_schema.py | ||
| test_hindsight_env_perms.py | ||
| test_hindsight_provider.py | ||
| test_holographic_auto_extract.py | ||
| test_holographic_retrieval.py | ||
| test_holographic_shutdown_closes_db.py | ||
| test_holographic_store.py | ||
| test_honcho_config_schema.py | ||
| test_mem0_backend.py | ||
| test_mem0_providers.py | ||
| test_mem0_setup.py | ||
| test_mem0_v3.py | ||
| test_memory_lazy_install.py | ||
| test_openviking_endpoint_always_blocked.py | ||
| test_openviking_provider.py | ||
| test_openviking_shutdown.py | ||
| test_retaindb_provider.py | ||
| test_supermemory_provider.py | ||