Skip to content

Add $abortTransaction command tests#600

Draft
alinaliBQ wants to merge 14 commits into
documentdb:mainfrom
alinaliBQ:abortTransaction
Draft

Add $abortTransaction command tests#600
alinaliBQ wants to merge 14 commits into
documentdb:mainfrom
alinaliBQ:abortTransaction

Conversation

@alinaliBQ

Copy link
Copy Markdown
Contributor

This change adds tests for the $abortTransaction command operator.

Add command operator tests for $abortTransaction. Tests database $abortTransaction behavior, output collection, syntax, and expected errors.

@alinaliBQ

Copy link
Copy Markdown
Contributor Author

Once #560 is merged, I can rebase this PR and prepare it for review.

@documentdb-triage-tool documentdb-triage-tool Bot added compatibility test Compatibility test related enhancement New feature or request labels Jun 12, 2026
@documentdb-triage-tool

Copy link
Copy Markdown

🤖 Auto-triaged by documentdb-triage-tool.

Applied: compatibility test, enhancement
Project fields suggested: Component test-coverage · Priority P2 · Effort L · Status In Progress
Confidence: 0.85 (mixed)

Reasoning

component from path globs (test-coverage, test-framework, ci); effort from diff stats (1817+0 LOC, 17 files); LLM: Adds new test coverage for the $abortTransaction command operator, covering behavior, output collection, syntax, and error cases — a meaningful functional addition to the test suite.

If a label is wrong, remove it manually and ping @patty-chow so the rules can be tuned. The bot will not re-label items that already have component labels.

alinaliBQ added 14 commits June 12, 2026 14:44
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
rename NO_SUCH_TRANSACTION_ERROR, and Added readback verification to execute_abort_session_command

Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

compatibility test Compatibility test related enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant