Added test

Author: Sammyjo20

src/Traits/Connector/SendsRequests.php

@@ -54,9 +54,10 @@ public function send(Request $request, MockClient $mockClient = null, callable $
             // the interval (if it has been provided)
 
             if ($attempts > 1) {
-                $sleepTime = $useExponentialBackoff 
-                             ? $retryInterval * (2 ** ($attempts - 1)) * 1000
-                             : $retryInterval * 1000;
+                $sleepTime = $useExponentialBackoff
+                    ? $retryInterval * (2 ** ($attempts - 2)) * 1000
+                    : $retryInterval * 1000;
+
                 usleep($sleepTime);
             }
 

src/Traits/RequestProperties/HasTries.php

@@ -26,6 +26,13 @@ trait HasTries
      */
     public ?int $retryInterval = null;
 
+    /**
+     * Should Saloon use exponential backoff during retries?
+     *
+     * When true, Saloon will double the retry interval after each attempt.
+     */
+    public bool $useExponentialBackoff = false;
+
     /**
      * Should Saloon throw an exception after exhausting the maximum number of retries?
      *
@@ -35,13 +42,6 @@ trait HasTries
      */
     public ?bool $throwOnMaxTries = null;
 
-    /**
-     * Should Saloon use exponential backoff during retries?
-     *
-     * When true, Saloon will double the retry interval after each attempt.
-     */
-    public bool $useExponentialBackoff = false;
-
     /**
      * Define whether the request should be retried.
      *

tests/Feature/SendAndRetryTest.php

@@ -108,6 +108,27 @@
     expect(round(microtime(true) - $start))->toBeGreaterThanOrEqual(2);
 });
 
+test('a failed request can have an interval with exponential backoff between each attempt', function () {
+    $mockClient = new MockClient([
+        MockResponse::make(['name' => 'Sam'], 500), // 1,000
+        MockResponse::make(['name' => 'Gareth'], 500), // 2,000
+        MockResponse::make(['name' => 'Michael'], 500), // 4,000
+        MockResponse::make(['name' => 'Teodor'], 200),
+    ]);
+
+    $connector = new TestConnector;
+    $connector->withMockClient($mockClient);
+
+    $start = microtime(true);
+
+    $connector->sendAndRetry(new UserRequest, 4, 1000, useExponentialBackoff: true);
+
+    // It should be a duration of > 7000ms (7 seconds) because the there are four requests
+    // after the first.
+
+    expect(round(microtime(true) - $start))->toBeGreaterThanOrEqual(7);
+});
+
 test('an exception other than a request exception will not be retried', function () {
     $mockClient = new MockClient([
         MockResponse::make(['name' => 'Sam'], 500),