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

    XMLWordPrintable

    Details

    • Type: Task
    • Status: Closed
    • Priority: Unknown
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Documentation Changes:
      Not Needed

      Description

      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

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              neal.beeken Neal Beeken
              Reporter:
              daria.pardue Daria Pardue
              Implementer:
              Daria Pardue Daria Pardue
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: