Skip to content

Add comprehensive isolation pooling tests - #4636

Draft
mdaigle wants to merge 1 commit into
dotnet:mainfrom
mdaigle:dev/automation/isolation-contract-tests-only
Draft

Add comprehensive isolation pooling tests#4636
mdaigle wants to merge 1 commit into
dotnet:mainfrom
mdaigle:dev/automation/isolation-contract-tests-only

Conversation

@mdaigle

@mdaigle mdaigle commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Description

Adds a self-contained ManualTests suite for transaction isolation and connection pooling. It covers both pool implementations, sync and async paths, MARS, local and ambient completion, Snapshot transitions, enlistment boundaries, direct SQL observations, compatibility controls, Synapse, and deterministic reset failures.

Each test name and XML summary describes its complete setup and expected behavior. This PR intentionally contains no product changes.

Issues

Related to #96, #146, #4330, and #4335.

Testing

Guidelines

  • Tests added or updated
  • Public API changes documented (N/A)
  • Verified against customer repro
  • No breaking changes introduced

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings September 3, 2026 01:46
@github-project-automation github-project-automation Bot moved this to To triage in SqlClient Board Sep 3, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

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

Labels

None yet

Projects

Status: To triage

Development

Successfully merging this pull request may close these issues.

2 participants