-
Type: Improvement
-
Resolution: Fixed
-
Priority: Unknown
-
Affects Version/s: None
-
Component/s: None
In the retryable spec tests we need to test partial deep equality with the expected result, in addition to checking first for a "value" property that holds the real operation result.
We should investigate what changes are needed to make these CRUD operations exactly deep equal to the spec expectations.
Test runner in:
retryable-writes/retryable_writes.spec.test.ts
AC:
- Make the sure the following are accessible on the BulkWriteResult class:
-
- deletedCount
- insertedCount
- insertedIds
- matchedCount
- modifiedCount
- upsertedCount
- upsertedIds
- Make the result property private and non-enumerable. (
NODE-4393) - enable the deep equality checking in the test runner
- Mark BulkResult as internal in the API.
- Remove BulkWriteResult.toJSON()
- Document changes in the migration guide
- depends on
-
NODE-4733 Deprecate bulk result properties for V5 and investigate spec compliance
- Closed
- is depended on by
-
NODE-4913 insertedIds don't correspond to initial operation ordering in an unordered bulkwrite
- Backlog
- is related to
-
NODE-4791 Deprecate duplicate BulkWriteResult accessors
- Closed
-
NODE-4393 Make `result` property from BulkWriteResult class non-enumerable
- Closed
- related to
-
NODE-2936 Full CRUD spec result type conformance
- Closed