From abbb3b8ac86581761fe8e073383ca13cd03a58ee Mon Sep 17 00:00:00 2001 From: Pranav Jain Date: Sun, 1 Dec 2024 15:18:55 -0500 Subject: [PATCH] chore(sdk-core): fix unknown types of sinon spy getCalls TICKET: WP-00000 --- .../test/v2/unit/internal/tssUtils/ecdsa.ts | 4 +-- modules/bitgo/test/v2/unit/wallet.ts | 30 ++++++++++--------- 2 files changed, 18 insertions(+), 16 deletions(-) diff --git a/modules/bitgo/test/v2/unit/internal/tssUtils/ecdsa.ts b/modules/bitgo/test/v2/unit/internal/tssUtils/ecdsa.ts index 7885ac1f20..3d06907e6e 100644 --- a/modules/bitgo/test/v2/unit/internal/tssUtils/ecdsa.ts +++ b/modules/bitgo/test/v2/unit/internal/tssUtils/ecdsa.ts @@ -910,7 +910,7 @@ describe('TSS Ecdsa Utils:', async function () { reqId, }); signedTxRequest.unsignedTxs.should.deepEqual(txRequest.unsignedTxs); - const userGpgActual = sendShareSpy.getCalls()[0].args[10]; + const userGpgActual = sendShareSpy.getCalls()[0].args[10] as string; userGpgActual.should.startWith('-----BEGIN PGP PUBLIC KEY BLOCK-----'); }); @@ -927,7 +927,7 @@ describe('TSS Ecdsa Utils:', async function () { reqId, }); signedTxRequest.unsignedTxs.should.deepEqual(txRequest.unsignedTxs); - const userGpgActual = sendShareSpy.getCalls()[0].args[10]; + const userGpgActual = sendShareSpy.getCalls()[0].args[10] as string; userGpgActual.should.startWith('-----BEGIN PGP PUBLIC KEY BLOCK-----'); }); diff --git a/modules/bitgo/test/v2/unit/wallet.ts b/modules/bitgo/test/v2/unit/wallet.ts index afdd210185..1d87d89190 100644 --- a/modules/bitgo/test/v2/unit/wallet.ts +++ b/modules/bitgo/test/v2/unit/wallet.ts @@ -32,6 +32,8 @@ import { SendManyOptions, PopulatedIntent, TxRequestVersion, + WalletSignMessageOptions, + WalletSignTypedDataOptions, } from '@bitgo/sdk-core'; import { TestBitGo } from '@bitgo/sdk-test'; @@ -3334,8 +3336,8 @@ describe('V2 Wallet:', function () { prv: 'secretKey', }); signMessage.should.deepEqual(expectedWithCoinField); - const actualArg = signMessageTssSpy.getCalls()[0].args[0]; - actualArg.message.messageEncoded.should.equal( + const actualArg = signMessageTssSpy.getCalls()[0].args[0] as WalletSignMessageOptions; + actualArg.message?.messageEncoded?.should.equal( `\u0019Ethereum Signed Message:\n${messageRaw.length}${messageRaw}` ); }); @@ -3351,8 +3353,8 @@ describe('V2 Wallet:', function () { prv: 'secretKey', }); signMessage.should.deepEqual(expectedWithCoinField); - const actualArg = signMessageTssSpy.getCalls()[0].args[0]; - actualArg.message.messageEncoded.should.equal( + const actualArg = signMessageTssSpy.getCalls()[0].args[0] as WalletSignMessageOptions; + actualArg.message?.messageEncoded?.should.equal( `\u0019Ethereum Signed Message:\n${messageRaw.length}${messageRaw}` ); }); @@ -3367,8 +3369,8 @@ describe('V2 Wallet:', function () { prv: 'secretKey', }); signMessage.should.deepEqual(expectedWithCoinField); - const actualArg = signMessageTssSpy.getCalls()[0].args[0]; - actualArg.message.messageEncoded.should.equal( + const actualArg = signMessageTssSpy.getCalls()[0].args[0] as WalletSignMessageOptions; + actualArg.message?.messageEncoded?.should.equal( `\u0019Ethereum Signed Message:\n${messageRaw.length}${messageRaw}` ); }); @@ -3530,8 +3532,8 @@ describe('V2 Wallet:', function () { prv: 'secretKey', }); signedTypedData.should.deepEqual(expected); - const actualArg = signTypedDataTssSpy.getCalls()[0].args[0]; - actualArg.typedData.typedDataEncoded.toString('hex').should.equal(txHash); + const actualArg = signTypedDataTssSpy.getCalls()[0].args[0] as WalletSignTypedDataOptions; + actualArg.typedData?.typedDataEncoded?.toString('hex').should.equal(txHash); }); it('should sign typed data V3 when custodianMessageID is provided', async function () { @@ -3552,8 +3554,8 @@ describe('V2 Wallet:', function () { prv: 'secretKey', }); signedTypedData.should.deepEqual(expected); - const actualArg = signTypedDataTssSpy.getCalls()[0].args[0]; - actualArg.typedData.typedDataEncoded.toString('hex').should.equal(txHash); + const actualArg = signTypedDataTssSpy.getCalls()[0].args[0] as WalletSignTypedDataOptions; + actualArg.typedData?.typedDataEncoded?.toString('hex').should.equal(txHash); }); it('should fail to sign typed data V3 with empty prv', async function () { @@ -3577,8 +3579,8 @@ describe('V2 Wallet:', function () { prv: 'secretKey', }); signedTypedData.should.deepEqual(expected); - const actualArg = signedTypedDataTssSpy.getCalls()[0].args[0]; - actualArg.typedData.typedDataEncoded.toString('hex').should.equal(txHash); + const actualArg = signedTypedDataTssSpy.getCalls()[0].args[0] as WalletSignTypedDataOptions; + actualArg.typedData?.typedDataEncoded?.toString('hex').should.equal(txHash); }); }); @@ -3643,8 +3645,8 @@ describe('V2 Wallet:', function () { prv: 'secretKey', }); signedTypedData.should.deepEqual(expected); - const actualArg = signedTypedDataTssSpy.getCalls()[0].args[0]; - actualArg.typedData.typedDataEncoded.toString('hex').should.equal(txHash); + const actualArg = signedTypedDataTssSpy.getCalls()[0].args[0] as WalletSignTypedDataOptions; + actualArg.typedData?.typedDataEncoded?.toString('hex').should.equal(txHash); }); }); });