fix(tests): resolve ambiguous step definition in persona state coverage tests
CI / push-validation (pull_request) Successful in 36s
CI / helm (pull_request) Successful in 40s
CI / lint (pull_request) Failing after 1m8s
CI / build (pull_request) Successful in 3m52s
CI / quality (pull_request) Successful in 4m28s
CI / typecheck (pull_request) Successful in 4m40s
CI / security (pull_request) Successful in 4m55s
CI / coverage (pull_request) Has been skipped
CI / unit_tests (pull_request) Failing after 5m12s
CI / docker (pull_request) Has been skipped
CI / e2e_tests (pull_request) Successful in 7m52s
CI / integration_tests (pull_request) Successful in 7m57s
CI / status-check (pull_request) Failing after 4s
CI / push-validation (pull_request) Successful in 36s
CI / helm (pull_request) Successful in 40s
CI / lint (pull_request) Failing after 1m8s
CI / build (pull_request) Successful in 3m52s
CI / quality (pull_request) Successful in 4m28s
CI / typecheck (pull_request) Successful in 4m40s
CI / security (pull_request) Successful in 4m55s
CI / coverage (pull_request) Has been skipped
CI / unit_tests (pull_request) Failing after 5m12s
CI / docker (pull_request) Has been skipped
CI / e2e_tests (pull_request) Successful in 7m52s
CI / integration_tests (pull_request) Successful in 7m57s
CI / status-check (pull_request) Failing after 4s
- Rename duplicate step 'the registry last persona should be set to' to 'the mock registry last persona should be set to' in tui_persona_state_coverage_steps.py - Update corresponding feature file to use the new step name - Fixes AmbiguousStep error that was preventing unit tests from running
This commit is contained in:
@@ -236,7 +236,7 @@ def step_verify_session_active_persona(context, session_id, expected):
|
||||
assert context.state.active_by_session[session_id] == expected
|
||||
|
||||
|
||||
@then('the registry last persona should be set to "{expected}"')
|
||||
@then('the mock registry last persona should be set to "{expected}"')
|
||||
def step_verify_last_persona_set(context, expected):
|
||||
context.mock_registry.set_last_persona.assert_called_with(expected)
|
||||
|
||||
|
||||
@@ -33,7 +33,7 @@ Feature: TUI Persona State Coverage
|
||||
When I set persona "coder" for session "sess-6"
|
||||
Then the returned persona name should be "coder"
|
||||
And session "sess-6" should have active persona "coder"
|
||||
And the registry last persona should be set to "coder"
|
||||
And the mock registry last persona should be set to "coder"
|
||||
|
||||
Scenario: set_active_persona skips preset init when session already has one
|
||||
Given the preset for session "sess-6b" is already set to "turbo"
|
||||
|
||||
Reference in New Issue
Block a user