Skip to content

Conversation

tadjik1
Copy link
Member

@tadjik1 tadjik1 commented Sep 22, 2025

Description

Summary of Changes

This PR migrates the integration tests for collection-management. The changes include:

  • Converting the spec test file from .js to .ts.
  • Replacing callbacks with async/await for integration tests.
  • Importing directly from src to avoid using the legacy wrapper.
Notes for Reviewers

To simplify the review process, this PR is split into several commits:

  • The first commit covers the .js to .ts conversion.
  • Other commits contains the remaining refactoring changes (split by the purpose).

What is the motivation for this change?

This work is part of a larger, ongoing initiative to convert all tests to use async/await, with the ultimate goal of removing the legacy driver wrapper.

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 September 23, 2025 12:29
@tadjik1 tadjik1 requested a review from a team as a code owner September 23, 2025 12:29
@baileympearson baileympearson merged commit 06b50af into main Sep 23, 2025
27 of 31 checks passed
@baileympearson baileympearson deleted the NODE-7177 branch September 23, 2025 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants