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.
test(persistence): add plan/action persistence suites
Some checks failed
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
fix(test): resolve AmbiguousStep collision, invalid ULIDs, and edge commit
Some checks are pending
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.
feat: merge branch 'master' into feature/m1-persistence-tests
Some checks are pending
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
feat: merge branch 'master' into feature/m1-persistence-tests
All checks were successful
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
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 milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
cleveragents/cleveragents-core!96
No description provided.