Updated composer added disable option

Author: Sammyjo20

composer.json

@@ -18,16 +18,15 @@
     "minimum-stability": "stable",
     "require": {
         "php": "^8.0",
-        "sammyjo20/saloon": "^1.0",
+        "sammyjo20/saloon": "^1.0.1",
         "nesbot/carbon": "^2.57"
     },
     "require-dev": {
-        "friendsofphp/php-cs-fixer": "^3.5",
-        "pestphp/pest": "^1.21",
-        "spatie/ray": "^1.33",
+        "friendsofphp/php-cs-fixer": "^3.8",
+        "pestphp/pest": "^1.21.2",
+        "spatie/ray": "^1.34.2",
         "league/flysystem": "^3.0",
-        "illuminate/cache": "^8.0|^9.0",
-        "orchestra/testbench": "^7.4"
+        "orchestra/testbench": "^7.0"
     },
     "scripts": {
         "test": [

src/Traits/AlwaysCacheResponses.php

@@ -15,11 +15,18 @@ trait AlwaysCacheResponses
      *
      * @var array
      */
-    private array $safeMethods = [
+    private array $safeCacheMethods = [
         Saloon::GET,
         Saloon::OPTIONS,
     ];
 
+    /**
+     * Is caching enabled?
+     *
+     * @var bool
+     */
+    protected bool $cachingEnabled = true;
+
     /**
      * Boot the Saloon plugin
      *
@@ -30,10 +37,14 @@ trait AlwaysCacheResponses
      */
     public function bootAlwaysCacheResponses(SaloonRequest $request): void
     {
+        if ($this->cachingEnabled === false) {
+            return;
+        }
+
         // We should only cache on "read-only" methods and not on methods
         // like POST, PUT.
 
-        if (! in_array($request->getMethod(), $this->safeMethods, true)) {
+        if (! in_array($request->getMethod(), $this->safeCacheMethods, true)) {
             return;
         }
 
@@ -86,6 +97,30 @@ public function generateCacheKey(SaloonRequest $request, array $headers): string
         return hash('sha256', $this->cacheKey($request, $headers));
     }
 
+    /**
+     * Enable caching for the request.
+     *
+     * @return $this
+     */
+    public function enableCaching(): self
+    {
+        $this->cachingEnabled = true;
+
+        return $this;
+    }
+
+    /**
+     * Disable caching for the request.
+     *
+     * @return $this
+     */
+    public function disableCaching(): self
+    {
+        $this->cachingEnabled = false;
+
+        return $this;
+    }
+
     /**
      * Return an instance of the cache driver that should be used.
      *

tests/Feature/CacheTest.php

@@ -172,3 +172,32 @@
     expect($filesystem->directoryExists('custom'))->toBeTrue();
     expect(count($filesystem->listContents('custom')->toArray()))->toEqual(1);
 });
+
+test('you can disable the cache', function () {
+    $mockClient = new MockClient([
+        MockResponse::make(['name' => 'Sam']),
+        MockResponse::make(['name' => 'Gareth']),
+        MockResponse::make(['name' => 'Michael']),
+    ]);
+
+    $requestA = new CachedUserRequest();
+    $responseA = $requestA->send($mockClient);
+
+    expect($responseA->isCached())->toBeFalse();
+    expect($responseA->json())->toEqual(['name' => 'Sam']);
+
+    $requestB = new CachedUserRequest();
+    $responseB = $requestB->send($mockClient);
+
+    expect($responseB->isCached())->toBeTrue();
+    expect($responseB->header('X-Saloon-Cache'))->toEqual('Cached');
+    expect($responseB->json())->toEqual(['name' => 'Sam']);
+
+    $requestC = new CachedUserRequest();
+    $requestC->disableCaching();
+
+    $responseC = $requestC->send($mockClient);
+
+    expect($responseC->isCached())->toBeFalse();
+    expect($responseC->json())->toEqual(['name' => 'Michael']);
+});