Reverting change to assert is signer

iron-fish · Feb 5, 2024 · cf03a87 · cf03a87
1 parent 354244b
commit cf03a87
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/ironfish/src/wallet/account/encoder/multiSigKeys.ts b/ironfish/src/wallet/account/encoder/multiSigKeys.ts
@@ -64,8 +64,7 @@ export function isSignerMultiSig(multiSigKeys: MultiSigKeys): multiSigKeys is Mu
 }
 
 export function AssertIsSignerMultiSig(
-  multiSigKeys: MultiSigKeys | undefined,
+  multiSigKeys: MultiSigKeys,
 ): asserts multiSigKeys is MultiSigSigner {
-  Assert.isNotUndefined(multiSigKeys)
   Assert.isTrue(isSignerMultiSig(multiSigKeys))
 }
diff --git a/ironfish/src/wallet/wallet.test.slow.ts b/ironfish/src/wallet/wallet.test.slow.ts
@@ -1215,6 +1215,7 @@ describe('Wallet', () => {
 
       const signingCommitments: Commitment[] = []
       for (const participant of participants) {
+        Assert.isNotUndefined(participant.multiSigKeys)
         AssertIsSignerMultiSig(participant.multiSigKeys)
         signingCommitments.push(
           createSigningCommitment(participant.multiSigKeys.keyPackage, seed),
@@ -1287,6 +1288,7 @@ describe('Wallet', () => {
       const signatureShares: Record<string, string> = {}
 
       for (const participant of participants) {
+        Assert.isNotUndefined(participant.multiSigKeys)
         AssertIsSignerMultiSig(participant.multiSigKeys)
         signatureShares[participant.multiSigKeys.identifier] = createSigningShare(
           signingPackage,