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

Remove unused error check in src/operations/connect.ts

    • Type: Icon: Task Task
    • Resolution: Unresolved
    • Priority: Icon: Minor - P4 Minor - P4
    • None
    • Affects Version/s: None
    • Component/s: Error
    • None

      Use Case

      As a driver developer
      I want to remove the error check at src/operations/connect.ts:38
      So that we can clean the codebase of unused legacy code
      Along with a skipped test in replicaset_mocks that uses the travis filter.

      User Impact

      None

      Dependencies

      • None

      Unknowns

      • None

      Acceptance Criteria

      Remove if block starting with

      if (err && err.message === 'no mongos proxies found in seed list')

      at src/operations/connect.ts:38

      Remove test at test/functional/replicaset_mock.test.js: 88

      Documentation Requirements

      • DOCSP ticket, API docs, etc

      Follow Up Requirements

      • additional tickets to file, required releases, etc

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

              Created:
              Updated: