fix(test): update _FailingConfigService.set_value() to match scoped signature
CI / security (pull_request) Failing after 2s
CI / quality (pull_request) Failing after 2s
CI / e2e_tests (pull_request) Failing after 2s
CI / build (pull_request) Failing after 1s
CI / helm (pull_request) Failing after 1s
CI / typecheck (pull_request) Has been cancelled
CI / integration_tests (pull_request) Has been cancelled
CI / unit_tests (pull_request) Has been cancelled
CI / lint (pull_request) Has been cancelled
CI / status-check (pull_request) Has been cancelled
CI / docker (pull_request) Has been cancelled
CI / coverage (pull_request) Has been cancelled
CI / benchmark-regression (pull_request) Has been cancelled
CI / benchmark-publish (pull_request) Has been cancelled
CI / security (pull_request) Failing after 2s
CI / quality (pull_request) Failing after 2s
CI / e2e_tests (pull_request) Failing after 2s
CI / build (pull_request) Failing after 1s
CI / helm (pull_request) Failing after 1s
CI / typecheck (pull_request) Has been cancelled
CI / integration_tests (pull_request) Has been cancelled
CI / unit_tests (pull_request) Has been cancelled
CI / lint (pull_request) Has been cancelled
CI / status-check (pull_request) Has been cancelled
CI / docker (pull_request) Has been cancelled
CI / coverage (pull_request) Has been cancelled
CI / benchmark-regression (pull_request) Has been cancelled
CI / benchmark-publish (pull_request) Has been cancelled
Update the test double to accept the scope keyword argument added by master's scoped config changes, and pass project_root=None to avoid auto-discovery in test environments. ISSUES CLOSED: #993
This commit is contained in:
@@ -23,7 +23,7 @@ from unittest.mock import patch
|
||||
from behave import given, then, when
|
||||
from behave.runner import Context
|
||||
|
||||
from cleveragents.application.services.config_service import ConfigService
|
||||
from cleveragents.application.services.config_service import ConfigScope, ConfigService
|
||||
from cleveragents.infrastructure.events.reactive import ReactiveEventBus
|
||||
from cleveragents.infrastructure.events.types import EventType
|
||||
|
||||
@@ -62,11 +62,14 @@ class _FailingConfigService(ConfigService):
|
||||
config_dir=config_dir,
|
||||
config_path=config_path,
|
||||
event_bus=event_bus,
|
||||
project_root=None,
|
||||
)
|
||||
self._fail_on_call = fail_on_call_number
|
||||
self._call_count = 0
|
||||
|
||||
def set_value(self, key: str, value: Any) -> None:
|
||||
def set_value(
|
||||
self, key: str, value: Any, *, scope: ConfigScope | None = None
|
||||
) -> None:
|
||||
"""Override set_value to fail on a specific call number."""
|
||||
self._call_count += 1
|
||||
if self._call_count == self._fail_on_call:
|
||||
@@ -74,7 +77,7 @@ class _FailingConfigService(ConfigService):
|
||||
f"Simulated disk failure on set_value call #{self._call_count} "
|
||||
f"(key={key!r})"
|
||||
)
|
||||
super().set_value(key, value)
|
||||
super().set_value(key, value, scope=scope)
|
||||
|
||||
|
||||
# ---------------------------------------------------------------------------
|
||||
@@ -123,6 +126,7 @@ def step_pre_existing_config(context: Context) -> None:
|
||||
svc = ConfigService(
|
||||
config_dir=context.atomic_config_dir,
|
||||
config_path=context.atomic_config_path,
|
||||
project_root=None,
|
||||
)
|
||||
svc.set_value("server.url", _ORIGINAL_URL)
|
||||
svc.set_value("server.namespace", _ORIGINAL_NAMESPACE)
|
||||
|
||||
Reference in New Issue
Block a user