From d049f6f8db6fbb393fed5f147ace71078f7a8b3b Mon Sep 17 00:00:00 2001 From: willclarktech Date: Tue, 20 Apr 2021 15:52:48 +0200 Subject: [PATCH] cosmwasm-stargate: Update tests for new Secp256k1HdWallet options --- .../cosmwasm-stargate/src/testutils.spec.ts | 42 +++---------------- 1 file changed, 6 insertions(+), 36 deletions(-) diff --git a/packages/cosmwasm-stargate/src/testutils.spec.ts b/packages/cosmwasm-stargate/src/testutils.spec.ts index b2b86e3a..a6de5881 100644 --- a/packages/cosmwasm-stargate/src/testutils.spec.ts +++ b/packages/cosmwasm-stargate/src/testutils.spec.ts @@ -1,12 +1,6 @@ /* eslint-disable @typescript-eslint/naming-convention */ -import { - AminoSignResponse, - makeCosmoshubPath, - Secp256k1HdWallet, - Secp256k1HdWalletOptions, - StdSignDoc, -} from "@cosmjs/amino"; -import { Bip39, EnglishMnemonic, Random, Secp256k1, Slip10, Slip10Curve } from "@cosmjs/crypto"; +import { AminoSignResponse, Secp256k1HdWallet, Secp256k1HdWalletOptions, StdSignDoc } from "@cosmjs/amino"; +import { Bip39, EnglishMnemonic, Random } from "@cosmjs/crypto"; import { Bech32, fromBase64 } from "@cosmjs/encoding"; import { DirectSecp256k1HdWallet, @@ -203,12 +197,6 @@ export async function makeWasmClient( return QueryClient.withExtensions(tmClient, setupAuthExtension, setupBankExtension, setupWasmExtension); } -const defaultHdWalletOptions = { - bip39Password: "", - hdPath: makeCosmoshubPath(0), - prefix: "cosmos", -}; - /** * A class for testing clients using an Amino signer which modifies the transaction it receives before signing */ @@ -217,18 +205,9 @@ export class ModifyingSecp256k1HdWallet extends Secp256k1HdWallet { mnemonic: string, options: Partial = {}, ): Promise { - const { bip39Password, hdPath, prefix } = { ...defaultHdWalletOptions, ...options }; const mnemonicChecked = new EnglishMnemonic(mnemonic); - const seed = await Bip39.mnemonicToSeed(mnemonicChecked, bip39Password); - const { privkey } = Slip10.derivePath(Slip10Curve.Secp256k1, seed, hdPath); - const uncompressed = (await Secp256k1.makeKeypair(privkey)).pubkey; - return new ModifyingSecp256k1HdWallet( - mnemonicChecked, - hdPath, - privkey, - Secp256k1.compressPubkey(uncompressed), - prefix, - ); + const seed = await Bip39.mnemonicToSeed(mnemonicChecked, options.bip39Password); + return new ModifyingSecp256k1HdWallet(mnemonicChecked, { ...options, seed: seed }); } public async signAmino(signerAddress: string, signDoc: StdSignDoc): Promise { @@ -252,18 +231,9 @@ export class ModifyingDirectSecp256k1HdWallet extends DirectSecp256k1HdWallet { mnemonic: string, options: Partial = {}, ): Promise { - const { bip39Password, hdPath, prefix } = { ...defaultHdWalletOptions, ...options }; const mnemonicChecked = new EnglishMnemonic(mnemonic); - const seed = await Bip39.mnemonicToSeed(mnemonicChecked, bip39Password); - const { privkey } = Slip10.derivePath(Slip10Curve.Secp256k1, seed, hdPath); - const uncompressed = (await Secp256k1.makeKeypair(privkey)).pubkey; - return new ModifyingDirectSecp256k1HdWallet( - mnemonicChecked, - hdPath, - privkey, - Secp256k1.compressPubkey(uncompressed), - prefix, - ); + const seed = await Bip39.mnemonicToSeed(mnemonicChecked, options.bip39Password); + return new ModifyingDirectSecp256k1HdWallet(mnemonicChecked, { ...options, seed: seed }); } public async signDirect(address: string, signDoc: SignDoc): Promise {