Merge pull request #231 from Sammyjo20/fix/support-null-mock-responses

(V1) Fix | Use of `null` in MockResponse data

src/Http/MockResponse.php

@@ -148,6 +148,10 @@ public function getFormattedData(): mixed
             return json_encode($data);
         }
 
+        if (empty($data)) {
+            return null;
+        }
+
         return $data;
     }
 

src/Traits/CollectsData.php

@@ -76,7 +76,7 @@ public function addData(string $data, $value): static
     }
 
     /**
-     *  Get all data or filter with a key.
+     * Get all data or filter with a key.
      *
      * @param string|null $key
      * @return mixed

src/Traits/CollectsHeaders.php

@@ -96,12 +96,12 @@ public function getHeaders(string $key = null): mixed
     /**
      * Get an individual header
      *
-     * @param string|null $key
-     * @return array
+     * @param string $key
+     * @return string
      */
     public function getHeader(string $key): string
     {
-        return $this->getHeaders($key);
+        return $this->getHeaders($key) ?? '';
     }
 
     /**

tests/Unit/MockResponseTest.php

@@ -68,3 +68,12 @@
     expect($response)->customCastMethod()->toBeInstanceOf(UserData::class);
     expect($response)->foo()->toBe('bar');
 });
+
+test('a mock response can have null for the data', function () {
+    $mockClient = new MockClient([MockResponse::make(null, 200)]);
+    $request = new UserRequest();
+
+    $response = $request->send($mockClient);
+
+    expect($response->body())->toEqual('');
+});