fix(model): execute valid write operations if calling bulkWrite() with ordered: false

[vkarpov15]

Fix #13176

Summary

Make the bulkWrite() API more consistent with insertMany():

If ordered: false, validate all write operations, and strip out any invalid ones. Execute the valid operations.

1. If remaining operations succeed, add mongoose.validationErrors to result to indicate which operations failed validation.
2. If remaining operations failed, add mongoose.validationErrors to the error to indicate which operations failed validation.

I'm not thrilled with the fact that bulkWrite() doesn't throw an error if some operations failed validation, but that is consistent with how insertMany() works. And I'd prefer to maintain consistency, and consider changing the API for the next major release. What do you think @hasezoey ?

Examples

[github-advanced-security]

You have successfully added a new CodeQL configuration .github/workflows/codeql.yml:CodeQL-Build. As part of the setup process, we have scanned this repository and found 5 existing alerts. Please check the repository Security tab to see all alerts.

[hasezoey]

i am not very familiar with this part of the code, so i cannot say much about it

I'm not thrilled with the fact that bulkWrite() doesn't throw an error if some operations failed validation, but that is consistent with how insertMany() works

i also think if there are errors it should throw by default, but maybe also allow a option like applyValidOperations or something (which is by default false in a next major release)

[vkarpov15]

@hasezoey it is a reasonable idea to throw an error if errors occurred. However, I'd like to add that in a separate PR because I would like to add that option to insertMany() as well as bulkWrite().

Probably call the option throwOnValidationError or something like that, because we would apply valid operations either way. The difference would be that, if throwOnValidationError is true and ordered is false, then bulkWrite() and insertMany() would throw an error if there were operations that failed validation, but all operations that passed validation succeeded.

[IslandRhythms]

LGTM

[doomhz]

This check should be moved somewhere above the if (ordered) { block. Please see #14117