Skip to content

test(NODE-7393): sync spec tests for transaction logging#4850

Merged
baileympearson merged 4 commits intomainfrom
NODE-7393
Jan 30, 2026
Merged

test(NODE-7393): sync spec tests for transaction logging#4850
baileympearson merged 4 commits intomainfrom
NODE-7393

Conversation

@tadjik1
Copy link
Copy Markdown
Member

@tadjik1 tadjik1 commented Jan 28, 2026

Description

Summary of Changes

Sync spec changes (tests) for command logging, in particular for commitTransaction and abortTransaction commands. See DRIVERS-3369 for more information.

Double check the following

  • Lint is passing (npm run check:lint)
  • Self-review completed using the steps outlined here
  • PR title follows the correct format: type(NODE-xxxx)[!]: description
    • Example: feat(NODE-1234)!: rewriting everything in coffeescript
  • Changes are covered by tests
  • New TODOs have a related JIRA ticket

@tadjik1 tadjik1 marked this pull request as ready for review January 28, 2026 19:31
@tadjik1 tadjik1 requested a review from a team as a code owner January 28, 2026 19:31
@tadjik1 tadjik1 changed the title test(NODE-7393): spec tests for transaction logging test(NODE-7393): sync spec tests for transaction logging Jan 29, 2026
@baileympearson baileympearson self-assigned this Jan 30, 2026
@baileympearson baileympearson added the Primary Review In Review with primary reviewer, not yet ready for team's eyes label Jan 30, 2026
@baileympearson baileympearson merged commit d46b8d0 into main Jan 30, 2026
14 of 17 checks passed
@baileympearson baileympearson deleted the NODE-7393 branch January 30, 2026 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Primary Review In Review with primary reviewer, not yet ready for team's eyes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants