SP-X Updating dependencies and tests, adding generated config files to gitignore

Author: m-kwiatkowski

.gitignore

@@ -12,4 +12,8 @@ composer.phar
 .phpunit.result.cache
 test/unit/_html
 
-PrivateKey.key
+PrivateKey.key
+BitPay.config.json
+**/BitPay.config.json
+BitPay.config.yml
+**/BitPay.config.yml

composer.json

@@ -12,14 +12,14 @@
   ],
   "homepage": "https://github.com/bitpay/php-bitpay-client-v2",
   "require": {
-    "php": "^8.1 || ^8.2 || ^8.3 || ^8.4",
+    "php": "*",
     "ext-json": "*",
     "ext-reflection": "*",
     "bitpay/key-utils": "^2.1",
-    "guzzlehttp/guzzle": "^7.0",
-    "symfony/yaml": "^5.0 || ^6.0 || ^7.0",
+    "guzzlehttp/guzzle": "^7.9",
+    "symfony/yaml": "^6.4",
     "netresearch/jsonmapper": "^5.0",
-    "symfony/console": "^4.4 || ^5.4 || ^6.0"
+    "symfony/console": "^6.4"
   },
   "authors": [
     {
@@ -28,7 +28,7 @@
     }
   ],
   "require-dev": {
-    "phpunit/phpunit": "^10.5 || ^11.5 || ^12.0"
+    "phpunit/phpunit": "^10.5"
   },
   "scripts": {
     "setup": [

composer.lock

@@ -12,13 +12,36 @@ abstract class AbstractClientTestCase extends TestCase
 {
     protected Client $client;
 
+    // Default delay in seconds between API calls
+    private const API_CALL_DELAY = 0.5;
+    private static $lastApiCallTime = 0;
+
     /**
      * @throws BitPayGenericException
      */
     public function setUp(): void
     {
+        // Add delay to respect rate limits
+        $this->respectRateLimit();
+
         $this->client = Client::createWithFile(
             Config::FUNCTIONAL_TEST_PATH . DIRECTORY_SEPARATOR . Config::BITPAY_CONFIG_FILE
         );
     }
+
+    /**
+     * Delays execution if needed to respect rate limits
+     */
+    protected function respectRateLimit(): void
+    {
+        $currentTime = microtime(true);
+        $timeSinceLastCall = $currentTime - self::$lastApiCallTime;
+
+        if (self::$lastApiCallTime > 0 && $timeSinceLastCall < self::API_CALL_DELAY) {
+            $sleepTime = (self::API_CALL_DELAY - $timeSinceLastCall);
+            usleep((int)($sleepTime * 1000000)); // Convert to microseconds
+        }
+
+        self::$lastApiCallTime = microtime(true);
+    }
 }

test/functional/BitPaySDK/AbstractClientTestCase.php

@@ -34,6 +34,15 @@ public function testGetSettlement(): void
 
         $settlements = $this->client->getSettlements($currency, $dateStart, $dateEnd, $status);
 
+        // Skip test if no settlements exist
+        if (empty($settlements)) {
+            $this->markTestSkipped(
+                'No settlements found in test account. ' .
+                    'To test this functionality, ensure your test account has processed transactions.'
+            );
+            return;
+        }
+
         $settlement = $this->client->getSettlement($settlements[0]->getId());
 
         self::assertNotNull($settlement);
@@ -50,12 +59,22 @@ public function testGetReconciliationReport(): void
         $currency = 'USD';
 
         $settlements = $this->client->getSettlements($currency, $dateStart, $dateEnd, $status);
+
+        if (empty($settlements)) {
+            $this->markTestSkipped(
+                'No settlements found in test account. ' .
+                    'To test this functionality, ensure your test account has processed transactions.'
+            );
+            return;
+        }
+
         $settlement = $this->client->getSettlement($settlements[0]->getId());
-        $settlement = $this->client->getSettlementReconciliationReport($settlement);
+        $settlementId = $settlement->getId();
+        $token = $settlement->getToken();
+        $reconciliationReport = $this->client->getSettlementReconciliationReport($settlementId, $token);
 
-        self::assertEquals('processing', $settlement->getStatus());
-        self::assertNotNull($settlement);
-        self::assertEquals('USD', $settlement->getCurrency());
-        self::assertEquals($status, $settlement->getStatus());
+        self::assertNotNull($reconciliationReport);
+        self::assertEquals($currency, $reconciliationReport->getCurrency());
+        self::assertEquals($status, $reconciliationReport->getStatus());
     }
 }