Skip to content

Commit

Permalink
text: fix tests for using signature twice
Browse files Browse the repository at this point in the history
  • Loading branch information
b00ste committed Aug 9, 2023
1 parent 1e2c0d2 commit 413ca43
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions tests/LSP6KeyManager/Relay/ExecuteRelayCall.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1142,8 +1142,8 @@ export const shouldBehaveLikeExecuteRelayCall = (
[transferLyxPayload, transferLyxPayload],
),
)
.to.be.revertedWithCustomError(context.keyManager, 'InvalidRelayNonce')
.withArgs(incorrectRecoveredAddress, ownerNonce.add(1), transferLyxSignature);
.to.be.revertedWithCustomError(context.keyManager, 'CannotUseSignatureTwice')
.withArgs(transferLyxSignature);
});

it('should 1) give the permission to someone to mint, 2) let the controller mint, 3) remove the permission to the controller to mint', async () => {
Expand Down
4 changes: 2 additions & 2 deletions tests/Reentrancy/LSP6/LSP6Reentrancy.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -182,8 +182,8 @@ export const shouldBehaveLikeLSP6ReentrancyScenarios = (
.connect(relayer)
.executeRelayCall(signature, nonce, validityTimestamps, executeRelayCallPayload),
)
.to.be.revertedWithCustomError(context.keyManager, 'InvalidRelayNonce')
.withArgs(signer.address, nonce, signature);
.to.be.revertedWithCustomError(context.keyManager, 'CannotUseSignatureTwice')
.withArgs(signature);
});
});
});
Expand Down

0 comments on commit 413ca43

Please sign in to comment.