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

Separate tests that do not require a real server into a separate suite and group related tests by functionality

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

      Use Case

      As a Node driver developer
      I want a test suite that is logically organized
      So that it's easy to find existing tests and identify missing tests related to a given driver functionality

      User Impact

      N/A

      Dependencies

      N/A

      Unknowns

      • Whether NODE-2899 will be a blocker to fully separate the mock server tests from the real server ones

      Acceptance Criteria

      • The functional test directory should only contain tests that require a real server
      • The functional test directory should mimic the specs directory structure
      • Tests that do not require a real server should go into the unit tests directory
      • The unit test directory should mimic the src directory structure
      • Tests that require a mock server may be optionally separated into a contract test directory or gain a "contract_" prefix
      • File a follow up ticket for a timeboxed effort to review all tests for accuracy

      Implementation Requirements

      • Files can be renamed and test blocks can be separated as needed in order to obtain the logical structure described in the AC

      Testing Requirements

      • All tests that are running and passing should continue to run and pass even after being moved around

      Documentation Requirements

      N/A

      Follow Up Requirements

      • If NODE-2899 is not a blocker for this work, move it out of this epic

            Assignee:
            daria.pardue@mongodb.com Daria Pardue
            Reporter:
            daria.pardue@mongodb.com Daria Pardue
            Neal Beeken
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: