Add allow stray requests method to config (#522)

* allow stray requests

* Code style fixer

---------

Co-authored-by: Sammyjo20 <29132017+Sammyjo20@users.noreply.github.com>

Author: ziadoz

src/Config.php

@@ -47,6 +47,11 @@ final class Config
      */
     private static ?MiddlewarePipeline $globalMiddlewarePipeline = null;
 
+    /**
+     * Whether stray requests should be prevented
+     */
+    private static bool $preventStrayRequests = false;
+
     /**
      * Write a custom sender resolver
      */
@@ -86,10 +91,20 @@ public static function clearGlobalMiddleware(): void
      */
     public static function preventStrayRequests(): void
     {
+        self::$preventStrayRequests = true;
+
         self::globalMiddleware()->onRequest(static function (PendingRequest $pendingRequest) {
-            if (! $pendingRequest->hasMockClient()) {
+            if (self::$preventStrayRequests && ! $pendingRequest->hasMockClient()) {
                 throw new StrayRequestException;
             }
         }, order: PipeOrder::LAST);
     }
+
+    /**
+     * Allow stray requests without a MockClient.
+     */
+    public static function allowStrayRequests(): void
+    {
+        self::$preventStrayRequests = false;
+    }
 }

src/Http/BaseResource.php

@@ -9,7 +9,7 @@ class BaseResource
     /**
      * Constructor
      */
-    public function __construct(readonly protected Connector $connector)
+    public function __construct(protected readonly Connector $connector)
     {
         //
     }

tests/Fixtures/Authenticators/CustomOAuthAuthenticator.php

@@ -13,10 +13,10 @@ class CustomOAuthAuthenticator extends AccessTokenAuthenticator
      * Constructor
      */
     public function __construct(
-        readonly public string             $accessToken,
-        readonly public string             $greeting,
-        readonly public ?string            $refreshToken = null,
-        readonly public ?DateTimeImmutable $expiresAt = null,
+        public readonly string             $accessToken,
+        public readonly string             $greeting,
+        public readonly ?string            $refreshToken = null,
+        public readonly ?DateTimeImmutable $expiresAt = null,
     ) {
         //
     }

tests/Fixtures/Requests/QueryParameterRequest.php

@@ -25,7 +25,7 @@ class QueryParameterRequest extends Request
     /**
      * Constructor
      */
-    public function __construct(readonly public string $endpoint = '/user')
+    public function __construct(public readonly string $endpoint = '/user')
     {
         //
     }

tests/Unit/ConfigTest.php

@@ -84,3 +84,19 @@
 
     Config::clearGlobalMiddleware();
 });
+
+test('you can prevent and then allow stray api requests', function () {
+    Config::preventStrayRequests();
+
+    try {
+        TestConnector::make()->send(new UserRequest);
+    } catch (StrayRequestException $e) {
+        expect($e)->toBeInstanceOf(StrayRequestException::class);
+    }
+
+    Config::allowStrayRequests();
+
+    TestConnector::make()->send(new UserRequest);
+
+    Config::clearGlobalMiddleware();
+});