fix(acms): move ContextAssemblyPipeline import to top of step file
CI / push-validation (pull_request) Successful in 23s
CI / helm (pull_request) Successful in 34s
CI / lint (pull_request) Successful in 57s
CI / build (pull_request) Successful in 1m0s
CI / quality (pull_request) Successful in 1m17s
CI / typecheck (pull_request) Successful in 1m50s
CI / security (pull_request) Successful in 1m51s
CI / e2e_tests (pull_request) Successful in 4m21s
CI / integration_tests (pull_request) Failing after 4m52s
CI / unit_tests (pull_request) Failing after 6m28s
CI / docker (pull_request) Has been skipped
CI / coverage (pull_request) Successful in 11m24s
CI / status-check (pull_request) Failing after 3s
CI / push-validation (pull_request) Successful in 23s
CI / helm (pull_request) Successful in 34s
CI / lint (pull_request) Successful in 57s
CI / build (pull_request) Successful in 1m0s
CI / quality (pull_request) Successful in 1m17s
CI / typecheck (pull_request) Successful in 1m50s
CI / security (pull_request) Successful in 1m51s
CI / e2e_tests (pull_request) Successful in 4m21s
CI / integration_tests (pull_request) Failing after 4m52s
CI / unit_tests (pull_request) Failing after 6m28s
CI / docker (pull_request) Has been skipped
CI / coverage (pull_request) Successful in 11m24s
CI / status-check (pull_request) Failing after 3s
- Move ContextAssemblyPipeline import to top of execution_phase_context_assembler_coverage_steps.py - Remove duplicate in-function import from step_epcov_pipeline_is_context_assembly - Remove duplicate CHANGELOG entry for ACMS Default Pipeline Wiring (#9169) - Lint and typecheck pass; CI failures are pre-existing in other test files
This commit is contained in:
@@ -14,14 +14,6 @@ The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
|
||||
and `ParallelStrategyExecutor`. The DI container's `acms_pipeline` singleton and
|
||||
`plan.py`'s `_get_plan_executor()` are updated accordingly. BDD coverage added to
|
||||
verify `ContextAssemblyPipeline` is the default.
|
||||
|
||||
|
||||
- **ACMS Default Pipeline Wiring** (#9169): `ACMSExecutePhaseContextAssembler` now
|
||||
defaults to `ContextAssemblyPipeline()` instead of the base `ACMSPipeline()` when
|
||||
no pipeline is injected. This ensures plan execution context assembly uses the full
|
||||
production pipeline with `ConfidenceWeightedSelector`, `ProportionalBudgetAllocator`,
|
||||
and `ParallelStrategyExecutor`. The DI container's `acms_pipeline` singleton and
|
||||
`plan.py`'s `_get_plan_executor()` are updated accordingly. BDD coverage added to
|
||||
verify `ContextAssemblyPipeline` is the default.
|
||||
|
||||
|
||||
|
||||
@@ -18,6 +18,7 @@ from unittest.mock import MagicMock, patch
|
||||
from behave import given, then, when
|
||||
from behave.runner import Context
|
||||
|
||||
from cleveragents.application.services.acms_pipeline import ContextAssemblyPipeline
|
||||
from cleveragents.application.services.execute_phase_context_assembler import (
|
||||
ACMSExecutePhaseContextAssembler,
|
||||
)
|
||||
@@ -958,7 +959,6 @@ def step_epcov_assembler_no_pipeline(context: Context) -> None:
|
||||
|
||||
@then("epcov the assembler pipeline should be a ContextAssemblyPipeline instance")
|
||||
def step_epcov_pipeline_is_context_assembly(context: Context) -> None:
|
||||
from cleveragents.application.services.acms_pipeline import ContextAssemblyPipeline
|
||||
|
||||
assert isinstance(context.epcov_assembler._pipeline, ContextAssemblyPipeline), (
|
||||
f"Expected ContextAssemblyPipeline, got {type(context.epcov_assembler._pipeline)}"
|
||||
|
||||
Reference in New Issue
Block a user