Skip to content

test(NODE-5043): assert MongoClients are garbage collectable#3561

Merged
dariakp merged 9 commits into4.xfrom
NODE-5028-add-leak-repro-4.x
Feb 13, 2023
Merged

test(NODE-5043): assert MongoClients are garbage collectable#3561
dariakp merged 9 commits into4.xfrom
NODE-5028-add-leak-repro-4.x

Conversation

@nbbeeken
Copy link
Copy Markdown
Contributor

@nbbeeken nbbeeken commented Feb 8, 2023

Description

What is changing?

Backport of
#3553

Double check the following

  • Ran npm run check:lint script
  • 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

@durran durran added the Primary Review In Review with primary reviewer, not yet ready for team's eyes label Feb 8, 2023
Comment thread test/tools/runner/config.ts
@dariakp dariakp added the Blocked Blocked on other work label Feb 10, 2023
@dariakp
Copy link
Copy Markdown
Contributor

dariakp commented Feb 10, 2023

Marking blocked pending review of #3553

@dariakp dariakp added wip and removed Blocked Blocked on other work labels Feb 10, 2023
@nbbeeken nbbeeken removed the wip label Feb 10, 2023
@dariakp dariakp added Team Review Needs review from team and removed Primary Review In Review with primary reviewer, not yet ready for team's eyes labels Feb 10, 2023
Comment thread test/integration/node-specific/resource_tracking_script_builder.ts Outdated
@dariakp dariakp merged commit ed5182a into 4.x Feb 13, 2023
@dariakp dariakp deleted the NODE-5028-add-leak-repro-4.x branch February 13, 2023 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Team Review Needs review from team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants