Convert embedded metadata document to stdClass

mongodb · Sep 8, 2023 · a326ea4 · a326ea4
1 parent 3a665f1
commit a326ea4
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 7 deletions.
diff --git a/tests/Fixtures/Codec/TestFileCodec.php b/tests/Fixtures/Codec/TestFileCodec.php
@@ -11,6 +11,8 @@
 use MongoDB\Exception\UnsupportedValueException;
 use MongoDB\Tests\Fixtures\Document\TestFile;
 
+use function assert;
+
 final class TestFileCodec implements DocumentCodec
 {
     use DecodeIfSupported;
@@ -31,13 +33,17 @@ public function decode($value): TestFile
         $fileObject->id = $value->get('_id');
         $fileObject->length = (int) $value->get('length');
         $fileObject->chunkSize = (int) $value->get('chunkSize');
-        $fileObject->filename = $value->get('filename');
+        $fileObject->filename = (string) $value->get('filename');
 
         $uploadDate = $value->get('uploadDate');
-        $fileObject->uploadDate = $uploadDate ? DateTimeImmutable::createFromMutable($uploadDate->toDateTime()) : null;
+        if ($uploadDate instanceof UTCDateTime) {
+            $fileObject->uploadDate = DateTimeImmutable::createFromMutable($uploadDate->toDateTime());
+        }
 
         if ($value->has('metadata')) {
-            $fileObject->metadata = $value->get('metadata');
+            $metadata = $value->get('metadata');
+            assert($metadata instanceof Document);
+            $fileObject->metadata = $metadata->toPHP();
         }
 
         return $fileObject;

diff --git a/tests/Fixtures/Document/TestFile.php b/tests/Fixtures/Document/TestFile.php
@@ -18,6 +18,7 @@
 namespace MongoDB\Tests\Fixtures\Document;
 
 use DateTimeImmutable;
+use stdClass;
 
 final class TestFile
 {
@@ -26,5 +27,5 @@ final class TestFile
     public int $chunkSize;
     public ?DateTimeImmutable $uploadDate = null;
     public string $filename;
-    public $metadata;
+    public ?stdClass $metadata = null;
 }
diff --git a/tests/GridFS/BucketFunctionalTest.php b/tests/GridFS/BucketFunctionalTest.php
@@ -3,7 +3,6 @@
 namespace MongoDB\Tests\GridFS;
 
 use MongoDB\BSON\Binary;
-use MongoDB\BSON\Document;
 use MongoDB\Collection;
 use MongoDB\Driver\ReadConcern;
 use MongoDB\Driver\ReadPreference;
@@ -19,6 +18,7 @@
 use MongoDB\Tests\Fixtures\Codec\TestDocumentCodec;
 use MongoDB\Tests\Fixtures\Codec\TestFileCodec;
 use MongoDB\Tests\Fixtures\Document\TestFile;
+use stdClass;
 
 use function array_merge;
 use function call_user_func;
@@ -509,8 +509,8 @@ public function testGetFileDocumentForStreamUsesCodec(): void
         $this->assertInstanceOf(TestFile::class, $fileDocument);
 
         $this->assertSame('filename', $fileDocument->filename);
-        $this->assertInstanceOf(Document::class, $fileDocument->metadata);
-        $this->assertSame($metadata, $fileDocument->metadata->toPHP(['root' => 'array']));
+        $this->assertInstanceOf(stdClass::class, $fileDocument->metadata);
+        $this->assertSame($metadata, (array) $fileDocument->metadata);
     }
 
     public function testGetFileDocumentForStreamWithReadableStream(): void