-
Type: Task
-
Resolution: Fixed
-
Priority: Unknown
-
Affects Version/s: None
-
Component/s: None
The BulkWriteResult class has a public property called `result` which contains the raw bulkWrite response. The class also has getters which just reach into the `result` object to pass nested properties out.
This causes issues with the unified test suite runner for `rewrapManyDataKey` operations because our BulkWriteResult class has extra properties (the `result` property), causing tests to fail. We worked around this in the test runner by making the property non-enumerable before returning it from the operation handler, but this is not ideal.
Acceptance Criteria
- Refactor the BulkWriteResult class to make the result property private and non-enumerable.
- Document change in the migration guide
- depends on
-
NODE-4733 Deprecate bulk result properties for V5 and investigate spec compliance
- Closed
- is depended on by
-
COMPASS-6252 Investigate changes in NODE-4393: remove public `result` property from BulkWriteResult class
- Closed
- is related to
-
NODE-4973 Support Optional Inserted Ids in CSFLE Unified Tests
- Closed
- related to
-
NODE-4034 Spec Compliance: BulkWrite result are not the same as spec expectations
- Closed
- links to