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

Remove OpQuery handling in command monitoring helpers

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

      Use Case

      As a... Driver engineer
      I want... to remove handling for opquery messages from extractCommand and extractReply helpers
      So that... we eliminate code that is only used during testing

      User Experience

      • N/A

      Dependencies

      • None

      Risks/Unknowns

      • Are the code paths that handle opquery payloads actually being used or are they just being exercised because of our MockServer/Unified test runner

      Acceptance Criteria

      Implementation Requirements

      • If these code path can be removed, then remove them, otherwise, document where/why they are used so we don't spend time investigating this in future

      Testing Requirements

      • Remove tests for these code paths if it turns out they are legitimately not used.

      Documentation Requirements

      • See implementation requirements

      Follow Up Requirements

      • None

       

      Note: The utility of these code paths came into question when working on NODE-6452.

            Assignee:
            Unassigned Unassigned
            Reporter:
            warren.james@mongodb.com Warren James
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: