fix: resolve ambiguous step definition conflict in execution_environment_steps.py
- Renamed 'it should contain' steps to 'the container types should contain' for specificity - Updated execution_environment.feature to use the new step names - This fixes the AmbiguousStep error that was preventing unit tests from running
This commit is contained in:
@@ -129,10 +129,10 @@ Feature: Execution environment routing
|
|||||||
|
|
||||||
Scenario: CONTAINER_RESOURCE_TYPES includes expected types
|
Scenario: CONTAINER_RESOURCE_TYPES includes expected types
|
||||||
Given I import CONTAINER_RESOURCE_TYPES
|
Given I import CONTAINER_RESOURCE_TYPES
|
||||||
Then it should contain "container-instance"
|
Then the container types should contain "container-instance"
|
||||||
And it should contain "devcontainer-instance"
|
And the container types should contain "devcontainer-instance"
|
||||||
And it should not contain "devcontainer-file"
|
And the container types should not contain "devcontainer-file"
|
||||||
And it should not contain "git-checkout"
|
And the container types should not contain "git-checkout"
|
||||||
|
|
||||||
# ── ContainerUnavailableError ───────────────────────────────────────
|
# ── ContainerUnavailableError ───────────────────────────────────────
|
||||||
|
|
||||||
@@ -145,4 +145,3 @@ Feature: Execution environment routing
|
|||||||
When I create a ContainerUnavailableError without project name
|
When I create a ContainerUnavailableError without project name
|
||||||
Then exec-env the error message should contain "Container resource unavailable"
|
Then exec-env the error message should contain "Container resource unavailable"
|
||||||
And exec-env the error message should not contain "for project"
|
And exec-env the error message should not contain "for project"
|
||||||
|
|
||||||
|
|||||||
@@ -342,12 +342,12 @@ def step_import_types(context: Context) -> None:
|
|||||||
context.container_types = CONTAINER_RESOURCE_TYPES
|
context.container_types = CONTAINER_RESOURCE_TYPES
|
||||||
|
|
||||||
|
|
||||||
@then('it should contain "{value}"')
|
@then('the container types should contain "{value}"')
|
||||||
def step_types_contain(context: Context, value: str) -> None:
|
def step_types_contain(context: Context, value: str) -> None:
|
||||||
assert value in context.container_types
|
assert value in context.container_types
|
||||||
|
|
||||||
|
|
||||||
@then('it should not contain "{value}"')
|
@then('the container types should not contain "{value}"')
|
||||||
def step_types_not_contain(context: Context, value: str) -> None:
|
def step_types_not_contain(context: Context, value: str) -> None:
|
||||||
assert value not in context.container_types
|
assert value not in context.container_types
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user