[BUGFIX] Show correct type in exception message

.github/workflows/ci.yml

@@ -84,6 +84,6 @@ jobs:
           vendor/bin/psalm --no-progress
       - name: Run infection
         run: |
-          vendor/bin/infection --no-progress --min-msi=93 --threads=4
+          vendor/bin/infection --no-progress --min-msi=94 --threads=4
         env:
           STRYKER_DASHBOARD_API_KEY: ${{ secrets.STRYKER_DASHBOARD_API_KEY }}

composer.json

@@ -18,7 +18,8 @@
         "ext-curl": "*",
         "ext-json": "*",
         "kriswallsmith/buzz": "^1.1",
-        "nyholm/psr7": "^1.2"
+        "nyholm/psr7": "^1.2",
+        "symfony/polyfill-php80": "^1.15"
     },
     "require-dev": {
         "captainhook/captainhook": "^5.1",

src/Client/RestClient.php

@@ -139,7 +139,7 @@ public function request(string $method, string $resource, $data = []): ResponseI
             throw new RestClientException(
                 \sprintf(
                     'data must be an array, "%s" given',
-                    gettype($data)
+                    \get_debug_type($data)
                 ),
                 1578233543
             );

src/Model/Incident.php

@@ -259,7 +259,7 @@ public function setProcessTableField(string $name, $value): self
                 \sprintf(
                     'value has to be either a string, an integer, a boolean or an instance of %s, "%s" given',
                     FileInterface::class,
-                    gettype($value)
+                    \get_debug_type($value)
                 ),
                 1578225863
             );

tests/Unit/Client/RestClientTest.php

@@ -363,17 +363,33 @@ public function formDataIsCorrectlySend(): void
     /**
      * @test
      */
-    public function requestWhenDataIsNotAnArrayAnExceptionIsThrown(): void
+    public function requestWhenDataIsABooleanAnExceptionIsThrown(): void
     {
         $this->expectException(RestClientException::class);
         $this->expectExceptionCode(1578233543);
+        $this->expectExceptionMessage('data must be an array, "bool" given');
 
         $this->setResponseOfTokensPath();
         $restClient = new RestClient(self::$configuration);
 
         $restClient->request('POST', 'some/route', false);
     }
 
+    /**
+     * @test
+     */
+    public function requestWhenDataIsAClassAnExceptionIsThrown(): void
+    {
+        $this->expectException(RestClientException::class);
+        $this->expectExceptionCode(1578233543);
+        $this->expectExceptionMessage('data must be an array, "stdClass" given');
+
+        $this->setResponseOfTokensPath();
+        $restClient = new RestClient(self::$configuration);
+
+        $restClient->request('POST', 'some/route', new \stdClass());
+    }
+
     /**
      * @test
      */

tests/Unit/Model/IncidentTest.php

@@ -373,14 +373,28 @@ public function setAndGetProcessTableFieldAreImplementedCorrectlyForAFile(): voi
     /**
      * @test
      */
-    public function setProcessTableFieldThrowsExceptionWhenValueTypeIsNotAllowed(): void
+    public function setProcessTableFieldThrowsExceptionWhenValueTypeIsAClass(): void
     {
         $this->expectException(\InvalidArgumentException::class);
         $this->expectExceptionCode(1578225863);
+        $this->expectExceptionMessage('value has to be either a string, an integer, a boolean or an instance of Brotkrueml\JobRouterClient\Resource\FileInterface, "stdClass" given');
 
         $this->subject->setProcessTableField('some name', new \stdClass());
     }
 
+    /**
+     * @test
+     */
+    public function setProcessTableFieldThrowsExceptionWhenValueTypeIsAnArray(): void
+    {
+        $this->expectException(\InvalidArgumentException::class);
+        $this->expectExceptionCode(1578225863);
+        $this->expectExceptionMessage('value has to be either a string, an integer, a boolean or an instance of Brotkrueml\JobRouterClient\Resource\FileInterface, "array" given');
+
+        /** @noinspection PhpParamsInspection */
+        $this->subject->setProcessTableField('some name', ['invalid']);
+    }
+
     /**
      * @test
      */