Uploaded image for project: 'Node.js Driver'
  1. Node.js Driver
  2. NODE-3985

Simplify TestConfiguration newClient test helper

    • Type: Icon: Improvement Improvement
    • Resolution: Unresolved
    • Priority: Icon: Minor - P4 Minor - P4
    • None
    • Affects Version/s: None
    • Component/s: Testing
    • Labels:

      Use Case

      As a node engineer
      I want a simple API to create new clients and track unclosed resources
      So that I don't impose shared assumptions and rules on all tests.

      User Impact

      *What is the number of impacted customers? How severe is the impact? Is anyone blocked or broken?

      None

      Dependencies

      • upstream and/or downstream requirements and timelines to bear in mind

      Unknowns

      • questions that need to be answered to determine implementation

      Acceptance Criteria

      Implementation Requirements

      • functional reqs, potential snafus to avoid, performance targets, etc

      Testing Requirements

      • Remove logic that sets options in newClient, beyond obvious settings like authentication
      • Removed withClient and withClientv2
      • Move client leak logic to newClient
      • Consider adding a lint rule to prevent direct usage of the MongoClient constructor, "illegal syntax" should work

      Documentation Requirements

      • DOCSP ticket, API docs, etc

      Follow Up Requirements

      • additional tickets to file, required releases, etc

            Assignee:
            Unassigned Unassigned
            Reporter:
            neal.beeken@mongodb.com Neal Beeken
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: