Backport findOneAndReplace fix to unified runner in 3.7

XMLWordPrintableJSON

    • Type: Task
    • Resolution: Fixed
    • Priority: Minor - P4
    • 3.7.0
    • Affects Version/s: None
    • Component/s: None
    • 1
    • None
    • Not Needed
    • None
    • None
    • None
    • None
    • None
    • None

      Use Case

      The unified runner has a mistake in the findOneAndReplace operation.
      Currently:
      return collection.findOneAndReplace(filter, replacement, translateOptions(opts));
      Should be:
      return (await collection.findOneAndReplace(filter, replacement, translateOptions(opts))).value;

      It's fixed in 4.0 as a part of NODE-3094

      Dependencies

      • Unified test runner

      Unknowns

      • none

      Acceptance Criteria

      Implementation Requirements
      • Make the stated change, it is discovered as part of the dots and dollars spec, which we aren't backporting.
      • Should we write a simple unified CRUD test to cover this path?
      Documentation Requirements
      • None
      Follow Up Requirements
      • None

            Assignee:
            Neal Beeken
            Reporter:
            Neal Beeken
            None
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: