Skip to content

test: reduce repeated app fixture setup - #173

Merged
bifrost0x merged 1 commit into
mainfrom
dev/optimize-pytest-fixtures
Aug 31, 2026
Merged

test: reduce repeated app fixture setup#173
bifrost0x merged 1 commit into
mainfrom
dev/optimize-pytest-fixtures

Conversation

@bifrost0x

Copy link
Copy Markdown
Owner

Summary

  • create one initialized empty SQLite template per pytest worker and clone it into each test data directory
  • keep function-level database isolation while avoiding hundreds of repeated schema builds
  • skip background runtime jobs for ordinary app fixtures and use an explicit runtime fixture for lifecycle contracts
  • generate immutable private-key fixtures once per pytest session

Performance

Measured on the same Python 3.11 environment and test set:

  • serial suite: 253.19s -> 137.62s (46% faster)
  • current CI command with 2 workers and loadscope: 160.62s -> 76.96s (52% faster)

The workflow worker count and test selection are unchanged.

Validation

  • 107 focused runtime, lifecycle, health, and key tests passed
  • 2,278 passed, 2 skipped serially
  • 2,278 passed, 2 skipped with the current CI pytest command
  • 2,278 passed, 2 skipped with an alternate work-stealing distribution to check order independence
  • git diff --check

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Aug 31, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-08-31T11:36:19.936233Z cbc47ac PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@bifrost0x bifrost0x self-assigned this Aug 31, 2026
@bifrost0x bifrost0x added dependencies Pull requests that update a dependency file python Pull requests that update python code github-actions docker Docker and container image updates labels Aug 31, 2026
@bifrost0x
bifrost0x merged commit b77398b into main Aug 31, 2026
21 checks passed
@bifrost0x
bifrost0x deleted the dev/optimize-pytest-fixtures branch August 31, 2026 11:36
@github-project-automation github-project-automation Bot moved this from Backlog to Done in WebSSH Roadmap Aug 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file docker Docker and container image updates github-actions python Pull requests that update python code

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant