Allow extra fields in aggregate results

jmikola · jmikola · commit 42bce4ea0e4c · 2023-05-31T15:21:19.000+08:00
Most operations that use ASSERT_SAME_DOCUMENT and ASSERT_SAME_DOCUMENTS should technically use ASSERT_MATCHES_DOCUMENT and ASSERT_DOCUMENTS_MATCH, respectively; however, this is the minimum change needed to satisfy new CSFLE spec tests. Related to dc1bb17
diff --git a/tests/SpecTests/Operation.php b/tests/SpecTests/Operation.php
@@ -723,7 +723,7 @@ private function getResultAssertionTypeForCollection()
                     return ResultExpectation::ASSERT_NOTHING;
                 }
 
-                return ResultExpectation::ASSERT_SAME_DOCUMENTS;
+                return ResultExpectation::ASSERT_DOCUMENTS_MATCH;
 
             case 'bulkWrite':
                 return ResultExpectation::ASSERT_BULKWRITE;

Original file line number	Diff line number	Diff line change
`@@ -723,7 +723,7 @@ private function getResultAssertionTypeForCollection()`
`723`	`723`	`return ResultExpectation::ASSERT_NOTHING;`
`724`	`724`	`}`
`725`	`725`
`726`		`- return ResultExpectation::ASSERT_SAME_DOCUMENTS;`
	`726`	`+ return ResultExpectation::ASSERT_DOCUMENTS_MATCH;`
`727`	`727`
`728`	`728`	`case 'bulkWrite':`
`729`	`729`	`return ResultExpectation::ASSERT_BULKWRITE;`