Skip to content

Prove board detail reads use split queries - #2618

Open
Chris0Jeky wants to merge 1 commit into
mainfrom
issue-1133/board-query-proof
Open

Prove board detail reads use split queries#2618
Chris0Jeky wants to merge 1 commit into
mainfrom
issue-1133/board-query-proof

Conversation

@Chris0Jeky

Copy link
Copy Markdown
Owner

Summary\n\n- Add a real-SQLite regression for BoardRepository.GetByIdWithDetailsAsync with two columns, two cards, two labels, and card-label relationships.\n- Clear tracking and captured setup commands before the read, then assert the complete graph and multiple executed SELECT commands.\n- Keep production code unchanged.\n\n## Verification\n\n- dotnet test backend/tests/Taskdeck.Api.Tests/Taskdeck.Api.Tests.csproj -c Release -m:1 --filter FullyQualifiedName~BoardRepositoryIntegrationTests — 19/19\n- Mutation control: temporarily removed only BoardRepository.AsSplitQuery(); the new test failed with one combined SELECT, then the line was restored and the test passed.\n- git diff --check origin/main...HEAD — passed\n\n## Docs\n\nNo canonical docs changed; this is a bounded regression-proof slice.\n\n## Scope\n\nRefs #1133

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Pending

Development

Successfully merging this pull request may close these issues.

1 participant