fix(test): correct project list scenario that expected abort on empty db #90

Merged
brent.edwards merged 3 commits from fix/project-list-test into master 2026-02-17 16:04:05 +00:00

3 Commits

Author SHA1 Message Date
khyari hamza 46f697bd95 fix(test): add temp directory and DB bootstrap to project list scenario
CI / lint (pull_request) Successful in 15s
CI / quality (pull_request) Successful in 22s
CI / security (pull_request) Successful in 26s
CI / typecheck (pull_request) Successful in 29s
CI / build (pull_request) Successful in 14s
CI / coverage (pull_request) Failing after 8m40s
CI / integration_tests (pull_request) Successful in 10m0s
CI / unit_tests (pull_request) Successful in 22m57s
CI / docker (pull_request) Successful in 40s
The 'Test project list command with no projects' scenario was missing a
Given step to set up a writable working directory. Without it, the
container fell back to CWD/.cleveragents/db.sqlite which could not be
opened (directory did not exist), causing sqlite3.OperationalError.

- Add 'Given I have a temporary working directory' to the scenario
- Bootstrap the SQLite schema in the When step so the ns_projects table
  exists for the empty-list query
2026-02-17 14:40:16 +00:00
khyari hamza 4ba714c389 style: shorten docstring markdown tables to satisfy E501 (88 cols)
CI / lint (pull_request) Successful in 21s
CI / typecheck (pull_request) Successful in 42s
CI / security (pull_request) Successful in 25s
CI / quality (pull_request) Successful in 24s
CI / build (pull_request) Successful in 15s
CI / coverage (pull_request) Failing after 8m19s
CI / integration_tests (pull_request) Successful in 9m21s
CI / unit_tests (pull_request) Failing after 21m4s
CI / docker (pull_request) Has been skipped
2026-02-17 14:04:24 +00:00
khyari hamza 4d63b5af7e fix(test): correct project list scenario that expected abort on empty db
CI / lint (pull_request) Failing after 14s
CI / typecheck (pull_request) Successful in 30s
CI / coverage (pull_request) Has been skipped
CI / security (pull_request) Successful in 28s
CI / quality (pull_request) Successful in 17s
CI / integration_tests (pull_request) Successful in 5m14s
CI / build (pull_request) Successful in 16s
CI / unit_tests (pull_request) Failing after 17m36s
CI / docker (pull_request) Has been skipped
The 'Test project list command with no database' scenario asserted the
command should abort, but the DI container creates a default SQLite DB
so list_projects() returns an empty list and exits 0 with 'No projects
found'.  Update the scenario to match the actual (correct) behavior.
2026-02-17 13:19:06 +00:00