test(persistence): add plan/action persistence suites #96

Closed
brent.edwards wants to merge 2 commits from feature/m1-persistence-tests into master
Member
No description provided.
brent.edwards added 1 commit 2026-02-17 21:41:16 +00:00
test(persistence): add plan/action persistence suites
CI / lint (pull_request) Successful in 25s
CI / typecheck (pull_request) Successful in 59s
CI / security (pull_request) Successful in 49s
CI / quality (pull_request) Successful in 33s
CI / unit_tests (pull_request) Failing after 8m26s
CI / docker (pull_request) Has been skipped
CI / build (pull_request) Successful in 15s
CI / integration_tests (pull_request) Successful in 10m5s
CI / coverage (pull_request) Failing after 53s
f912a136b7
brent.edwards added 1 commit 2026-02-18 00:10:28 +00:00
fix(test): resolve AmbiguousStep collision, invalid ULIDs, and edge commit
CI / lint (pull_request) Waiting to run
CI / typecheck (pull_request) Waiting to run
CI / security (pull_request) Waiting to run
CI / quality (pull_request) Waiting to run
CI / unit_tests (pull_request) Waiting to run
CI / integration_tests (pull_request) Waiting to run
CI / coverage (pull_request) Blocked by required conditions
CI / build (pull_request) Waiting to run
CI / docker (pull_request) Blocked by required conditions
ea7fcc0ba0
Three issues prevented the behave suite from running:

1. plan_persistence_steps.py and plan_repository_steps.py both defined
   @then('the retrieved plan description should be "..."'), causing an
   AmbiguousStep error that broke all ~50 feature files. Renamed the
   persistence step to "the persisted plan description should be".

2. The three-level hierarchy scenario used plan IDs containing O, I,
   and L which are excluded from the ULID charset. Replaced with valid
   characters (O->0, I->0, L->K).

3. resource_repository_steps.py flushed but never committed the edge
   row, so the delete method's separate session could not see it.
brent.edwards added 1 commit 2026-02-18 00:43:53 +00:00
feat: merge branch 'master' into feature/m1-persistence-tests
CI / typecheck (pull_request) Waiting to run
CI / lint (pull_request) Waiting to run
CI / security (pull_request) Waiting to run
CI / quality (pull_request) Waiting to run
CI / unit_tests (pull_request) Waiting to run
CI / integration_tests (pull_request) Waiting to run
CI / coverage (pull_request) Blocked by required conditions
CI / build (pull_request) Waiting to run
CI / docker (pull_request) Blocked by required conditions
b528cde8e7
brent.edwards added 1 commit 2026-02-18 00:58:29 +00:00
feat: merge branch 'master' into feature/m1-persistence-tests
CI / lint (pull_request) Successful in 14s
CI / typecheck (pull_request) Successful in 30s
CI / security (pull_request) Successful in 25s
CI / quality (pull_request) Successful in 16s
CI / integration_tests (pull_request) Successful in 5m15s
CI / build (pull_request) Successful in 16s
CI / unit_tests (pull_request) Successful in 16m42s
CI / docker (pull_request) Successful in 9s
CI / coverage (pull_request) Successful in 13m40s
eb65bbd680
Author
Member

Closing; this is part of #101

Closing; this is part of https://git.cleverthis.com/cleveragents/cleveragents-core/pulls/101
brent.edwards closed this pull request 2026-02-18 02:50:21 +00:00
freemo added this to the v3.0.0 milestone 2026-02-23 17:25:46 +00:00
Owner

Closes #248

Closes #248
freemo added the
State
Wont Do
label 2026-03-04 01:00:16 +00:00
All checks were successful
CI / lint (pull_request) Successful in 14s
Required
Details
CI / typecheck (pull_request) Successful in 30s
Required
Details
CI / security (pull_request) Successful in 25s
Required
Details
CI / quality (pull_request) Successful in 16s
Required
Details
CI / integration_tests (pull_request) Successful in 5m15s
Required
Details
CI / build (pull_request) Successful in 16s
Required
Details
CI / unit_tests (pull_request) Successful in 16m42s
Required
Details
CI / docker (pull_request) Successful in 9s
Required
Details
CI / coverage (pull_request) Successful in 13m40s
Required
Details

Pull request closed

Sign in to join this conversation.
No Reviewers
No Label
State
Wont Do
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: cleveragents/cleveragents-core#96