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

Add integration tests that assert client.close() cleans up all resources

    • Type: Icon: Task Task
    • Resolution: Unresolved
    • Priority: Icon: Unknown Unknown
    • None
    • Affects Version/s: None
    • Component/s: None
    • Not Needed
    • Hide

      1. What would you like to communicate to the user about this feature?
      2. Would you like the user to see examples of the syntax and/or executable code and its output?
      3. Which versions of the driver/connector does this apply to?

      Show
      1. What would you like to communicate to the user about this feature? 2. Would you like the user to see examples of the syntax and/or executable code and its output? 3. Which versions of the driver/connector does this apply to?

      Use Case

      As a... node.js engineer
      I want... integration tests that assert all related esources have been cleaned up after client.close() is called
      So that... i know the new client.close has the correct behavior

      Acceptance Criteria

      Implementation Requirements

      • Determine which tests cases are needed for integration testing client.close() resource clean-up
      • Run each test casewith the following structure:
        • Node-js Resource Test Structure
          • Run subprocess
          • Create and assert resources creation through client
          • Close the client
          • Assert no new resources remain through process.report.getReport().libuv and node.js beforeExit event
        • Server-side Resource Test Structure
          • Run subprocess
          • Create and assert resources exist on the server
          • Close the client
          • Assert no new resources remain through commands to the server

      Documentation Requirements

      • N/A

      Follow Up Requirements

      • There may be additional integration tests cases as implementation uncovers more information
      • Unskip tests once updated client.close is implemented

            Assignee:
            aditi.khare@mongodb.com Aditi Khare
            Reporter:
            aditi.khare@mongodb.com Aditi Khare
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: