fix(acms): move ContextAssemblyPipeline import to module level in step defs
CI / build (pull_request) Successful in 38s
CI / lint (pull_request) Successful in 41s
CI / helm (pull_request) Successful in 41s
CI / push-validation (pull_request) Successful in 28s
CI / quality (pull_request) Successful in 58s
CI / typecheck (pull_request) Successful in 1m21s
CI / security (pull_request) Successful in 1m22s
CI / unit_tests (pull_request) Successful in 4m59s
CI / docker (pull_request) Successful in 1m30s
CI / integration_tests (pull_request) Successful in 8m34s
CI / coverage (pull_request) Successful in 9m57s
CI / status-check (pull_request) Successful in 3s
CI / build (pull_request) Successful in 38s
CI / lint (pull_request) Successful in 41s
CI / helm (pull_request) Successful in 41s
CI / push-validation (pull_request) Successful in 28s
CI / quality (pull_request) Successful in 58s
CI / typecheck (pull_request) Successful in 1m21s
CI / security (pull_request) Successful in 1m22s
CI / unit_tests (pull_request) Successful in 4m59s
CI / docker (pull_request) Successful in 1m30s
CI / integration_tests (pull_request) Successful in 8m34s
CI / coverage (pull_request) Successful in 9m57s
CI / status-check (pull_request) Successful in 3s
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
This commit is contained in:
@@ -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,
|
||||
|
||||
Reference in New Issue
Block a user