Note that write errorResponse may need to be skipped in some drivers

mongodb · Oct 14, 2022 · b775b7a · b775b7a
1 parent 082b2e8
commit b775b7a
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 0 deletions.
diff --git a/source/crud/tests/unified/bulkWrite-errorResponse.yml b/source/crud/tests/unified/bulkWrite-errorResponse.yml
@@ -15,6 +15,11 @@ createEntities:
       collectionName: &collection0Name test
 
 tests:
+  # This test intentionally executes only a single insert operation in the bulk
+  # write to make the error code and response assertions less ambiguous. That
+  # said, some drivers may still need to skip this test because the CRUD spec
+  # does not prescribe how drivers should formulate a BulkWriteException beyond
+  # collecting write and write concern errors.
   - description: "bulkWrite operations support errorResponse assertions"
     runOnRequirements:
       - minServerVersion: "4.0.0"

diff --git a/source/crud/tests/unified/deleteOne-errorResponse.yml b/source/crud/tests/unified/deleteOne-errorResponse.yml
@@ -15,6 +15,9 @@ createEntities:
       collectionName: &collection0Name test
 
 tests:
+  # Some drivers may still need to skip this test because the CRUD spec does not
+  # prescribe how drivers should formulate a WriteException beyond collecting a
+  # write or write concern error.
   - description: "delete operations support errorResponse assertions"
     runOnRequirements:
       - minServerVersion: "4.0.0"

diff --git a/source/crud/tests/unified/insertOne-errorResponse.yml b/source/crud/tests/unified/insertOne-errorResponse.yml
@@ -15,6 +15,9 @@ createEntities:
       collectionName: &collection0Name test
 
 tests:
+  # Some drivers may still need to skip this test because the CRUD spec does not
+  # prescribe how drivers should formulate a WriteException beyond collecting a
+  # write or write concern error.
   - description: "insert operations support errorResponse assertions"
     runOnRequirements:
       - minServerVersion: "4.0.0"

diff --git a/source/crud/tests/unified/updateOne-errorResponse.yml b/source/crud/tests/unified/updateOne-errorResponse.yml
@@ -15,6 +15,9 @@ createEntities:
       collectionName: &collection0Name test
 
 tests:
+  # Some drivers may still need to skip this test because the CRUD spec does not
+  # prescribe how drivers should formulate a WriteException beyond collecting a
+  # write or write concern error.
   - description: "update operations support errorResponse assertions"
     runOnRequirements:
       - minServerVersion: "4.0.0"