fix/database-parallelism #410

Merged
brent.edwards merged 4 commits from fix/database-parallelism into master 2026-02-24 19:06:37 +00:00
Member

Description

This PR helps resolve the problem of parallel tests by ensuring that every disk file created for SQLite is different.

## Description This PR helps resolve the problem of parallel tests by ensuring that every disk file created for SQLite is different.
brent.edwards added 2 commits 2026-02-24 02:27:25 +00:00
Each behave-parallel worker previously resolved to the same file-based
SQLite database (cleveragents.db or .cleveragents/db.sqlite) because
before_scenario removed the CLEVERAGENTS_DATABASE_URL env var and the
fallback paths are shared across processes.  Under parallel execution
this caused intermittent 'database is locked' and duplicate-project
errors—most visibly in the coverage_report nox session.

Replace the env-var removal in before_scenario with unique temp database
paths via tempfile.mktemp(), giving every scenario its own isolated
database file.  Temp files are cleaned up in after_scenario.

Also fix cli_streaming_steps.py where a Background + duplicate Given
sequence triggered a duplicate project name collision, and update
coverage_boost_steps.py so Settings-default assertions explicitly clear
the env vars before testing pydantic defaults.
Merge branch 'master' into fix/database-parallelism
CI / benchmark-publish (pull_request) Has been skipped
CI / lint (pull_request) Successful in 15s
CI / build (pull_request) Successful in 16s
CI / quality (pull_request) Successful in 19s
CI / security (pull_request) Successful in 32s
CI / typecheck (pull_request) Successful in 33s
CI / integration_tests (pull_request) Successful in 3m53s
CI / unit_tests (pull_request) Successful in 7m31s
CI / docker (pull_request) Successful in 16s
CI / benchmark-regression (pull_request) Successful in 18m4s
CI / coverage (pull_request) Successful in 23m36s
dd963daeaa
freemo added the
MoSCoW
Must have
Points
5
Priority
High
State
In Progress
Type
Bug
labels 2026-02-24 03:36:45 +00:00
brent.edwards was assigned by freemo 2026-02-24 03:37:55 +00:00
CoreRasurae approved these changes 2026-02-24 15:35:00 +00:00
CoreRasurae left a comment
Member

Looks good, i've approved.

Looks good, i've approved.
brent.edwards added 1 commit 2026-02-24 16:02:54 +00:00
Merge branch 'master' into fix/database-parallelism
CI / quality (pull_request) Successful in 17s
CI / lint (pull_request) Successful in 20s
CI / benchmark-publish (pull_request) Has been skipped
CI / typecheck (pull_request) Successful in 30s
CI / build (pull_request) Successful in 18s
CI / security (pull_request) Successful in 50s
CI / integration_tests (pull_request) Successful in 3m34s
CI / unit_tests (pull_request) Successful in 6m43s
CI / docker (pull_request) Successful in 1m4s
CI / benchmark-regression (pull_request) Successful in 20m59s
CI / coverage (pull_request) Successful in 25m3s
c402e6402b
brent.edwards added 1 commit 2026-02-24 18:33:46 +00:00
Merge branch 'master' into fix/database-parallelism
CI / benchmark-publish (pull_request) Has been skipped
CI / lint (pull_request) Successful in 22s
CI / build (pull_request) Successful in 23s
CI / quality (pull_request) Successful in 31s
CI / security (pull_request) Successful in 50s
CI / typecheck (pull_request) Successful in 56s
CI / integration_tests (pull_request) Successful in 2m42s
CI / unit_tests (pull_request) Successful in 15m24s
CI / docker (pull_request) Successful in 1m0s
CI / benchmark-regression (pull_request) Successful in 20m58s
CI / coverage (pull_request) Successful in 31m36s
aa06b7aa37
brent.edwards merged commit df6e0f9e48 into master 2026-02-24 19:06:37 +00:00
brent.edwards deleted branch fix/database-parallelism 2026-02-24 19:06:37 +00:00
Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: cleveragents/cleveragents-core#410