Add explanatory comment for ID filling behaviour

mongodb · Aug 16, 2023 · aed2665 · aed2665
1 parent 0be8dd0
commit aed2665
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/tests/Collection/CodecCollectionFunctionalTest.php b/tests/Collection/CodecCollectionFunctionalTest.php
@@ -395,6 +395,8 @@ public function testInsertMany($expected, $options): void
         $result = $this->collection->insertMany($documents, $options);
         $this->assertSame(3, $result->getInsertedCount());
 
+        // Add missing identifiers. This is relevant for the "No codec" data set, as the encoded document will not have
+        // an "_id" field and the driver will automatically generate one.
         foreach ($expected as $index => &$expectedDocument) {
             if ($expectedDocument instanceof BSONDocument && $expectedDocument->_id === null) {
                 $expectedDocument->_id = $result->getInsertedIds()[$index];
@@ -434,6 +436,8 @@ public function testInsertOne($expected, $options): void
         $result = $this->collection->insertOne(TestObject::createForFixture(1), $options);
         $this->assertSame(1, $result->getInsertedCount());
 
+        // Add missing identifiers. This is relevant for the "No codec" data set, as the encoded document will not have
+        // an "_id" field and the driver will automatically generate one.
         if ($expected instanceof BSONDocument && $expected->_id === null) {
             $expected->_id = $result->getInsertedId();
         }