Compare commits

...

1 Commits

Author SHA1 Message Date
freemo 846e007738 fix(v3.7.0): resolve issue #1426
CI / benchmark-publish (pull_request) Has been skipped
CI / lint (pull_request) Failing after 19s
CI / build (pull_request) Successful in 16s
CI / quality (pull_request) Successful in 34s
CI / helm (pull_request) Successful in 23s
CI / security (pull_request) Failing after 50s
CI / typecheck (pull_request) Failing after 57s
CI / coverage (pull_request) Has been skipped
CI / benchmark-regression (pull_request) Has been skipped
CI / unit_tests (pull_request) Failing after 2m9s
CI / docker (pull_request) Has been skipped
CI / e2e_tests (pull_request) Failing after 17m50s
CI / integration_tests (pull_request) Failing after 22m13s
CI / status-check (pull_request) Failing after 1s
2026-04-02 19:31:48 +00:00
8 changed files with 13 additions and 13 deletions
+2 -2
View File
@@ -564,9 +564,9 @@ def step_try_create_action_model_from_config_missing(
context.action_model_config_error = e
@then('an action model config error should be raised with "{field}"')
@then('an action model configured should be raised with "{field}"')
def step_check_action_model_config_error(context: Context, field: str) -> None:
"""Check that a config error mentioning the field was raised."""
"""Check that a configured mentioning the field was raised."""
assert context.action_model_config_error is not None, (
f"Expected ValueError for missing '{field}'"
)
+2 -2
View File
@@ -351,9 +351,9 @@ def step_try_load_profile_config_missing_name(
context.profile_config_error = e
@then('a profile config error should be raised with "{field}"')
@then('a profile configured should be raised with "{field}"')
def step_check_profile_config_error(context: Context, field: str) -> None:
"""Check config error mentions field."""
"""Check configured mentions field."""
assert context.profile_config_error is not None, (
f"Expected ValueError for missing '{field}'"
)
@@ -224,7 +224,7 @@ def step_workspace_langsmith_enabled(context):
@given("a coverage gap workspace with a failing vector store")
def step_workspace_failing_vector_store(context):
stub = _VectorStoreStub(enabled=True)
stub.refresh_error = ConfigurationError("Simulated config error for coverage")
stub.refresh_error = ConfigurationError("Simulated configured for coverage")
_make_workspace(context, vector_stub=stub)
context.cov_vector_stub = stub
@@ -275,7 +275,7 @@ def step_vector_stub_has_results(context):
@given("the vector store search raises a ConfigurationError")
def step_vector_search_config_error(context):
context.cov_vector_stub.search_error = ConfigurationError(
"Simulated search config error"
"Simulated search configured"
)
@@ -366,7 +366,7 @@ def step_svc_vector_config_error(context: Context) -> None:
context.uow = _make_uow(plan=plan)
vs = MagicMock()
vs.is_enabled.return_value = True
vs.search.side_effect = ConfigurationError(message="config error")
vs.search.side_effect = ConfigurationError(message="configured")
context.svc = ContextService(context.settings, context.uow, vector_store_service=vs)
@@ -396,7 +396,7 @@ def step_check_config_error(context):
@then("the command should abort with ConfigurationError")
def step_check_abort_config(context):
"""Check command aborted with config error."""
"""Check command aborted with configured."""
assert context.result.exit_code != 0
@@ -4,7 +4,7 @@ These steps target specific uncovered lines in strategy_registry.py:
- Lines 65-68: CustomStrategyConfig.__init__ assignment block
- Line 167: register_from_config delegation
- Lines 196-200: register_all_from_config skip on missing module/class
- Lines 205-209: register_all_from_config error handling on load failure
- Lines 205-209: register_all_from_configured handling on load failure
"""
import logging
+2 -2
View File
@@ -467,9 +467,9 @@ def skill_model_try_load_missing(context: Context, field: str) -> None:
context.skill_model_config_error = e
@then('a skill_model config error should be raised with "{field}"')
@then('a skill_model configured should be raised with "{field}"')
def skill_model_check_config_error(context: Context, field: str) -> None:
"""Check that a config error mentioning field was raised."""
"""Check that a configured mentioning field was raised."""
assert context.skill_model_config_error is not None, (
f"Expected ValueError for missing '{field}'"
)
+2 -2
View File
@@ -561,9 +561,9 @@ def step_try_load_tool_config_missing(context: Context, field: str) -> None:
context.tool_model_config_error = e
@then('a tool model config error should be raised with "{field}"')
@then('a tool model configured should be raised with "{field}"')
def step_check_tool_config_error(context: Context, field: str) -> None:
"""Check that a config error mentioning field was raised."""
"""Check that a configured mentioning field was raised."""
assert context.tool_model_config_error is not None, (
f"Expected ValueError for missing '{field}'"
)