fix #3 add more test cases

Author: bernard-ng

src/Client.php

@@ -4,6 +4,7 @@
 
 namespace Devscast\Pexels;
 
+use Devscast\Pexels\Data\Collection;
 use Devscast\Pexels\Data\CollectionMedia;
 use Devscast\Pexels\Data\Collections;
 use Devscast\Pexels\Data\Photo;
@@ -305,7 +306,7 @@ private function getMappedData(string $type, array $data): Photos|Videos|Collect
                 $data['videos']
             ),
             $mapped instanceof Collections =>$mapped->collections = array_map(
-                fn ($m) => $this->serializer->denormalize($m, type: Collections::class),
+                fn ($m) => $this->serializer->denormalize($m, type: Collection::class),
                 $data['collections']
             ),
             $mapped instanceof CollectionMedia => $mapped->media = array_map(function ($m) {

src/Data/Resource.php

@@ -30,4 +30,9 @@ abstract class Resource
      * @var string The Pexels URL where the resource is located.
      */
     public string $url;
+
+    /**
+     * @var string|null The type of the resource. Possible values are photo and video.
+     */
+    public ?string $type = null;
 }

tests/ClientTest.php

@@ -11,10 +11,10 @@
 use Devscast\Pexels\Data\Photo;
 use Devscast\Pexels\Data\Photos;
 use Devscast\Pexels\Data\Videos;
+use Devscast\Pexels\Data\Collection;
 use Devscast\Pexels\Data\Collections;
 use Devscast\Pexels\Data\CollectionMedia;
 use Symfony\Component\HttpClient\MockHttpClient;
-use Symfony\Contracts\HttpClient\HttpClientInterface;
 use Symfony\Component\HttpClient\Response\MockResponse;
 
 /**
@@ -48,6 +48,7 @@ public function testSearchPhotos(): void
         $photos = $pexels->searchPhotos('westie dog');
 
         $this->assertInstanceOf(Photos::class, $photos);
+        $this->assertContainsOnlyInstancesOf(Photo::class, $photos->photos);
     }
 
     public function testSearchVideos(): void
@@ -57,6 +58,7 @@ public function testSearchVideos(): void
         $videos = $pexels->searchVideos('westie dog');
 
         $this->assertInstanceOf(Videos::class, $videos);
+        $this->assertContainsOnlyInstancesOf(Video::class, $videos->videos);
     }
 
     public function testPhoto(): void
@@ -84,6 +86,7 @@ public function testPopularVideos(): void
         $videos = $pexels->popularVideos();
 
         $this->assertInstanceOf(Videos::class, $videos);
+        $this->assertContainsOnlyInstancesOf(Video::class, $videos->videos);
     }
 
     public function curatedPhotos(): void
@@ -93,6 +96,7 @@ public function curatedPhotos(): void
         $photos = $pexels->curatedPhotos();
 
         $this->assertInstanceOf(Photos::class, $photos);
+        $this->assertContainsOnlyInstancesOf(Photo::class, $photos->photos);
     }
 
     public function testFeaturedCollections(): void
@@ -102,6 +106,7 @@ public function testFeaturedCollections(): void
         $collections = $pexels->featuredCollections();
 
         $this->assertInstanceOf(Collections::class, $collections);
+        $this->assertContainsOnlyInstancesOf(Collection::class, $collections->collections);
     }
 
     public function testCollections(): void
@@ -111,6 +116,7 @@ public function testCollections(): void
         $collections = $pexels->collections();
 
         $this->assertInstanceOf(Collections::class, $collections);
+        $this->assertContainsOnlyInstancesOf(Collection::class, $collections->collections);
     }
 
     public function testCollection(): void
@@ -120,5 +126,14 @@ public function testCollection(): void
         $collections = $pexels->collection("33");
 
         $this->assertInstanceOf(CollectionMedia::class, $collections);
+        foreach ($collections->media as $media) {
+            if ('photo' === $media->type) {
+                $this->assertInstanceOf(Photo::class, $media);
+            }
+
+            if ('video' === $media->type) {
+                $this->assertInstanceOf(Video::class, $media);
+            }
+        }
     }
 }

tests/MapperTest.php

@@ -0,0 +1,51 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Devscast\Pexels\Tests;
+
+use stdClass;
+use Devscast\Pexels\Mapper;
+use PHPUnit\Framework\TestCase;
+
+/**
+ * Class MapperTest.
+ *
+ * @author bernard-ng <bernard@devscast.tech>
+ */
+final class MapperTest extends TestCase
+{
+    public function testToObjectMethod(): void
+    {
+        // Test case: Test toObject method
+        $data = ['name' => 'John', 'age' => 30];
+        $object = new stdClass();
+
+        $resultObject = Mapper::toObject($object, $data);
+
+        // Assertions for toObject() method
+        $this->assertInstanceOf(stdClass::class, $resultObject);
+        $this->assertEquals('John', $resultObject->name);
+        $this->assertEquals(30, $resultObject->age);
+    }
+
+    public function testToArrayMethod(): void
+    {
+        // Test case: Test toArray method
+        $data = new class {
+            public string $name;
+            public int $age;
+        };
+        $data->name = 'Alice';
+        $data->age = 25;
+
+        $resultArray = Mapper::toArray([], $data);
+
+        // Assertions for toArray() method
+        $this->assertIsArray($resultArray);
+        $this->assertArrayHasKey('name', $resultArray);
+        $this->assertEquals('Alice', $resultArray['name']);
+        $this->assertArrayHasKey('age', $resultArray);
+        $this->assertEquals(25, $resultArray['age']);
+    }
+}