Merge pull request #486 from saloonphp/revert-485-test-multiple-requests

Revert "Make sure multiple assertions can be made on the same request class"

Author: Sammyjo20

src/Http/Faking/MockClient.php

@@ -463,21 +463,16 @@ private function checkClosureAgainstResponses(callable $closure, ?int $index = n
         // and call the callable accordingly. We will only fail if it returns `false`.
 
         if ($fqcn = $this->getRequestClass($closure)) {
-            /** @var Response $response */
+            /** @var Response */
             foreach ($this->getRecordedResponses() as $response) {
                 if (get_class($request = $response->getPendingRequest()->getRequest()) !== $fqcn) {
                     continue;
                 }
 
-                $passed = $closure($request, $response) !== false;
-
-                if ($passed === true) {
-                    return true;
-                }
+                return $closure($request, $response) !== false;
             }
-
-            return false;
         }
+
         // Let's then check if the latest response resolves the callable
         // with a successful result.
 

tests/Unit/MockClientTest.php

@@ -267,28 +267,17 @@
 test('`assertSent` accepts the request class as a type-hint', function () {
     $mockClient = new MockClient([
         MockResponse::make(['name' => 'Sam']),
-        MockResponse::make(['name' => 'Sam']),
     ]);
 
-    $requestA = new UserRequest;
-    $requestA->headers()->add('X-Foo', 'bar');
-    $requestB = new UserRequest;
-    $requestB->headers()->add('X-Foo', 'two');
-
-    connector()->send($requestA, $mockClient);
-    connector()->send($requestB, $mockClient);
+    $request = new UserRequest;
+    $request->headers()->add('X-Foo', 'bar');
 
-    $mockClient->assertSent(function (UserRequest $request) {
-        $header = $request->headers()->get('X-Foo');
-        expect($header)->toBeIn(['bar', 'two']);
+    connector()->send($request, $mockClient);
 
-        return $header === 'bar';
-    });
     $mockClient->assertSent(function (UserRequest $request) {
-        $header = $request->headers()->get('X-Foo');
-        expect($header)->toBeIn(['bar', 'two']);
-
-        return $header === 'two';
+        expect($request->headers()->all())->toMatchArray([
+            'X-Foo' => 'bar',
+        ]);
     });
 });