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

Encapsulate handling hello or isMaster in mongodb-mock-server

    • Type: Icon: Improvement Improvement
    • Resolution: Unresolved
    • Priority: Icon: Unknown Unknown
    • None
    • Affects Version/s: None
    • Component/s: Testing
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      Use Case

      As a node driver engineer I want to write less repetitive checks so that we have cleaner testing code.

      Dependencies

      • mongodb-mock-server

      Unknowns

      • questions that need to be answered to determine implementation

      Acceptance Criteria

      Implementation Requirements
      • ingest mongodb-mock-server into the testing directory of 4.0
      • remap commands with either hello or ismaster to set both to true so either can be used to trigger on the response
      • add an underscored property (or something to that effect) that wouldn't clash with anything mongodb would send that indicates the original commandName
      Testing Requirements
      • refactor all tests in 4.0 to use doc.hello where applicable
      Documentation Requirements
      • None
      Follow Up Requirements
      • None

            Assignee:
            Unassigned Unassigned
            Reporter:
            neal.beeken@mongodb.com Neal Beeken
            None
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated: