From bedd1ec0c500c11e6b032cc129b9fb57ef55bd8a Mon Sep 17 00:00:00 2001 From: CleverThis Date: Sun, 14 Jun 2026 07:29:32 -0400 Subject: [PATCH] fix(acms): move ContextAssemblyPipeline import to module level in step defs Per project import rules, all imports must appear at the top of the file. The ContextAssemblyPipeline import was inside the @then step function body; moved it to the module-level imports section alongside other production-code imports. ISSUES CLOSED: #10027 --- .../execute_phase_context_assembler_coverage_steps.py | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/features/steps/execute_phase_context_assembler_coverage_steps.py b/features/steps/execute_phase_context_assembler_coverage_steps.py index ffb899842..68ac3e0d3 100644 --- a/features/steps/execute_phase_context_assembler_coverage_steps.py +++ b/features/steps/execute_phase_context_assembler_coverage_steps.py @@ -19,6 +19,9 @@ 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, ) @@ -162,10 +165,6 @@ def step_epcov_explicit_pipeline(context: Context) -> None: @then("epcov the internal pipeline should be a ContextAssemblyPipeline") def step_epcov_default_pipeline_is_context_assembly(context: Context) -> None: - from cleveragents.application.services.acms_pipeline import ( - ContextAssemblyPipeline, - ) - assert isinstance( context.epcov_assembler._pipeline, ContextAssemblyPipeline,