From 846e007738a2b8784db94c08e1b71c4a7492933f Mon Sep 17 00:00:00 2001 From: Jeffrey Phillips Freeman Date: Thu, 2 Apr 2026 19:31:48 +0000 Subject: [PATCH] fix(v3.7.0): resolve issue #1426 --- features/steps/action_model_steps.py | 4 ++-- features/steps/automation_profile_steps.py | 4 ++-- features/steps/context_service_coverage_gaps_steps.py | 4 ++-- features/steps/context_service_new_coverage_steps.py | 2 +- features/steps/project_commands_coverage_steps.py | 2 +- features/steps/sandbox_strategy_registry_coverage_steps.py | 2 +- features/steps/skill_resolution_steps.py | 4 ++-- features/steps/tool_model_steps.py | 4 ++-- 8 files changed, 13 insertions(+), 13 deletions(-) diff --git a/features/steps/action_model_steps.py b/features/steps/action_model_steps.py index 6a444cf75..6d234c497 100644 --- a/features/steps/action_model_steps.py +++ b/features/steps/action_model_steps.py @@ -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}'" ) diff --git a/features/steps/automation_profile_steps.py b/features/steps/automation_profile_steps.py index 020271a9e..afd26b212 100644 --- a/features/steps/automation_profile_steps.py +++ b/features/steps/automation_profile_steps.py @@ -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}'" ) diff --git a/features/steps/context_service_coverage_gaps_steps.py b/features/steps/context_service_coverage_gaps_steps.py index 5d84ee090..722825b0c 100644 --- a/features/steps/context_service_coverage_gaps_steps.py +++ b/features/steps/context_service_coverage_gaps_steps.py @@ -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" ) diff --git a/features/steps/context_service_new_coverage_steps.py b/features/steps/context_service_new_coverage_steps.py index a33c318c7..c424b6e09 100644 --- a/features/steps/context_service_new_coverage_steps.py +++ b/features/steps/context_service_new_coverage_steps.py @@ -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) diff --git a/features/steps/project_commands_coverage_steps.py b/features/steps/project_commands_coverage_steps.py index ee6e7d52d..0c5f44c91 100644 --- a/features/steps/project_commands_coverage_steps.py +++ b/features/steps/project_commands_coverage_steps.py @@ -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 diff --git a/features/steps/sandbox_strategy_registry_coverage_steps.py b/features/steps/sandbox_strategy_registry_coverage_steps.py index bfc5fe25f..b22f62062 100644 --- a/features/steps/sandbox_strategy_registry_coverage_steps.py +++ b/features/steps/sandbox_strategy_registry_coverage_steps.py @@ -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 diff --git a/features/steps/skill_resolution_steps.py b/features/steps/skill_resolution_steps.py index 8dbb1bb27..4d559ac7f 100644 --- a/features/steps/skill_resolution_steps.py +++ b/features/steps/skill_resolution_steps.py @@ -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}'" ) diff --git a/features/steps/tool_model_steps.py b/features/steps/tool_model_steps.py index 8328a17fc..a73fdf28c 100644 --- a/features/steps/tool_model_steps.py +++ b/features/steps/tool_model_steps.py @@ -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}'" ) -- 2.52.0