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

    • Task
    • Status: Closed
    • Unknown
    • Resolution: Done
    • None
    • None
    • None
    • 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

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

              Dates

                Created:
                Updated:
                Resolved: