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

XMLWordPrintableJSON

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

      Create a copy of the Kickoff Template with the issue key (NODE-XXX) in the filename and share a link to the new doc via this field.

      Show
      Create a copy of the Kickoff Template with the issue key (NODE-XXX) in the filename and share a link to the new doc via this field.
    • Not Needed
    • None
    • None
    • None
    • None
    • None
    • None

      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:
            Neal Beeken
            Reporter:
            Aditi Khare (Inactive)
            None
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: