brent.edwards
b59100cc6c
CI / build (push) Successful in 16s
CI / helm (push) Successful in 28s
CI / lint (push) Successful in 3m20s
CI / quality (push) Successful in 3m53s
CI / security (push) Successful in 4m9s
CI / typecheck (push) Successful in 4m15s
CI / benchmark-regression (push) Has been skipped
CI / integration_tests (push) Successful in 6m57s
CI / unit_tests (push) Successful in 7m3s
CI / docker (push) Successful in 1m30s
CI / e2e_tests (push) Successful in 11m37s
CI / coverage (push) Successful in 11m27s
CI / status-check (push) Successful in 1s
CI / benchmark-publish (push) Successful in 27m7s
## Summary Adds TDD bug-capture coverage for bug #1141 (`session create` -> `session list` lifecycle) on metadata branch `tdd/m3-session-create-persist`. ### What changed - Added Behave bug-capture feature: `features/tdd_session_create_persist.feature` - Required tags: `@tdd_bug @tdd_bug_1141 @tdd_expected_fail` - Scenario path: `init --force --yes` -> `session list --format json` -> `session create` -> `session list --format json` - Added Behave step definitions: `features/steps/tdd_session_create_persist_steps.py` - Uses root CLI app (`cleveragents.cli.main:app`) for realistic command routing - Asserts expected list totals; underlying assertion intentionally fails while bug is present - Added Robot E2E bug-capture test: `robot/e2e/e2e_session_create_persist.robot` - Required tags: `E2E`, `tdd_expected_fail`, `tdd_bug`, `tdd_bug_1141` - Includes explicit in-file note to remove `tdd_expected_fail` when #1141 is fixed - Updated changelog (`CHANGELOG.md`, Unreleased) ### Notes - The issue subtask referenced `robot/e2e/e2e_session_lifecycle.robot`, which is not present on current `master`; equivalent E2E coverage is implemented in `robot/e2e/e2e_session_create_persist.robot`. - The branch was force-updated to remove stale merge-based history and keep an atomic, rebase-clean commit for this issue. ## Quality gates | Gate | Result | |---|---| | `nox -s lint` | ✅ pass | | `nox -s typecheck` | ✅ pass | | `nox -s unit_tests -- features/tdd_session_create_persist.feature` | ✅ pass (TDD inversion active; underlying assert fails) | | `nox -s integration_tests -- robot/e2e/e2e_session_create_persist.robot` | ✅ pass | | `nox -s e2e_tests` | ✅ pass | | `nox -s coverage_report` | ✅ pass (97.66173849218832%) | | `nox` (full default suite) | ✅ pass | ## Related issue Closes #1142 Reviewed-on: #1144 Reviewed-by: Jeffrey Phillips Freeman <jeffrey.freeman@cleverthis.com> Co-authored-by: Brent Edwards <brent.edwards@cleverthis.com> Co-committed-by: Brent Edwards <brent.edwards@cleverthis.com>
25 lines
1.3 KiB
Gherkin
25 lines
1.3 KiB
Gherkin
@tdd_expected_fail @tdd_issue @tdd_issue_1141
|
|
Feature: TDD Bug #1141 — session create does not persist for session list
|
|
As a developer
|
|
I want to verify that a session created via `agents session create`
|
|
appears in a subsequent `agents session list`
|
|
So that the persistence bug is captured and will be caught by a regression test
|
|
|
|
The bug: `agents session create` succeeds (exit code 0) but the created
|
|
session does not appear when `agents session list` is invoked immediately
|
|
after. This test captures bug #1141 and intentionally keeps
|
|
`@tdd_expected_fail` so CI passes while the bug remains unfixed.
|
|
|
|
Scenario: Init then create should make list total increase from 0 to 1
|
|
Given a CLI runner using the real session DI path
|
|
When I tdd1141 invoke init with force yes
|
|
Then the tdd1141 init should exit successfully
|
|
When I tdd1141 invoke session list with format json
|
|
Then the tdd1141 session list should exit successfully
|
|
And the tdd1141 session list output should report total 0
|
|
When I tdd1141 invoke session create
|
|
Then the tdd1141 session create should exit successfully
|
|
When I tdd1141 invoke session list with format json
|
|
Then the tdd1141 session list should exit successfully
|
|
And the tdd1141 session list output should report total 1
|