fix: Added kexIsDone property to IsMultisigResponse

Author: refring

src/WalletRpc/IsMultisigResponse.php

@@ -24,6 +24,9 @@ class IsMultisigResponse implements JsonDataSerializable
     #[Json]
     public bool $ready;
 
+    #[Json('kex_is_done')]
+    public bool $kexIsDone;
+
     /**
      * Amount of signature needed to sign a transfer.
      */

tests/integration/TransferTest.php

@@ -12,15 +12,18 @@
 use RefRing\MoneroRpcPhp\Exception\InvalidDestinationException;
 use RefRing\MoneroRpcPhp\Model\Address;
 use RefRing\MoneroRpcPhp\Model\Amount;
+use RefRing\MoneroRpcPhp\Tests\Attribute\RequiresMoneroVersion;
 use RefRing\MoneroRpcPhp\Tests\TestHelper;
-use RefRing\MoneroRpcPhp\Tests\Util\StdOutLogger;
+use RefRing\MoneroRpcPhp\Tests\Trait\RequiresMoneroVersionTrait;
 use RefRing\MoneroRpcPhp\WalletRpc\Model\Destination;
 use RefRing\MoneroRpcPhp\WalletRpc\Model\TransferType;
 use RefRing\MoneroRpcPhp\WalletRpc\TransferResponse;
 use RefRing\MoneroRpcPhp\WalletRpcClient;
 
 final class TransferTest extends TestCase
 {
+    use RequiresMoneroVersionTrait;
+
     public static array $seeds = [TestHelper::WALLET_1_MNEMONIC, TestHelper::WALLET_2_MNEMONIC];
 
     public static array $wallets = [];
@@ -51,6 +54,16 @@ public static function setUpBeforeClass(): void
         self::$daemonRpcClient->generateBlocks(100, TestHelper::MAINNET_ADDRESS_1);
     }
 
+    protected function setUp(): void
+    {
+        $this->checkMoneroVersionRequirements();
+    }
+
+    protected static function getDaemonRpcClient(): DaemonRpcClient
+    {
+        return self::$daemonRpcClient;
+    }
+
     public function testWallet(): void
     {
         self::$walletRpcClient->refresh();
@@ -201,6 +214,7 @@ public function testScanTx(TransferResponse $transferResponse): void
         $result = self::$walletRpcClient->scanTx([$transferResponse->txHash]);
     }
 
+    #[RequiresMoneroVersion('0.18.4.1')]
     public function testSubaddressLookahead(): void
     {
         // Restore wallet 1 for this test
@@ -242,15 +256,15 @@ public function testSubaddressLookahead(): void
         self::$walletRpcClient->transfer($destination);
 
         // Mine a block to confirm the transaction
-        self::$daemonRpcClient->generateBlocks(1, TestHelper::MAINNET_ADDRESS_1);
+        self::$daemonRpcClient->generateBlocks(10, TestHelper::MAINNET_ADDRESS_1);
 
         // Switch back to wallet 1 and verify the balance
         self::$walletRpcClient->restoreDeterministicWallet('', '', self::$seeds[0]);
         // Re-apply lookahead since wallet was restored
         self::$walletRpcClient->setSubaddressLookahead(50, 1000);
         self::$walletRpcClient->refresh();
 
-        $balanceResult = self::$walletRpcClient->getBalance(0, [999]);
+        $balanceResult = self::$walletRpcClient->getBalance(0);
         $this->assertNotEmpty($balanceResult->perSubaddress, 'Balance info for subaddress should exist');
 
         $balanceInfo0_999 = null;