diff --git a/TestsList.md b/TestsList.md index 95bbc24..59e38e6 100644 --- a/TestsList.md +++ b/TestsList.md @@ -201,7 +201,9 @@ Negatives [X] TestWallet should fail to set default profile with sig with signature deadline mismatch [X] TestWallet should fail to set default profile with sig with invalid deadline [X] TestWallet should fail to set default profile with sig with invalid nonce + + [-] TestWallet should sign attempt to set default profile with sig, cancel with empty permitForAll, then fail to set default profile with sig Scenarios [X] TestWallet should set the default profile with sig @@ -412,8 +414,8 @@ Module Globals Governance Misc NFT Transfer Emitters -[ ] User should not be able to call the follow NFT transfer event emitter function -[ ] User should not be able to call the collect NFT transfer event emitter function +[X] User should not be able to call the follow NFT transfer event emitter function +[X] User should not be able to call the collect NFT transfer event emitter function Lens Hub Misc [ ] UserTwo should fail to burn profile owned by user without being approved [ ] User should burn profile owned by user diff --git a/test/foundry/Misc.t.sol b/test/foundry/Misc.t.sol index 23baa7a..b354f81 100644 --- a/test/foundry/Misc.t.sol +++ b/test/foundry/Misc.t.sol @@ -4,6 +4,21 @@ pragma solidity ^0.8.13; import './base/BaseTest.t.sol'; import '../../contracts/mocks/MockFollowModule.sol'; +// Original Misc +contract NFTTransferEmittersTest is BaseTest { + // Negatives + function testCannotEmitFollowNFTTransferEvent() public { + vm.expectRevert(Errors.CallerNotFollowNFT.selector); + hub.emitFollowNFTTransferEvent(newProfileId, 1, profileOwner, otherSigner); + } + + function testCannotEmitCollectNFTTransferEvent() public { + vm.expectRevert(Errors.CallerNotCollectNFT.selector); + hub.emitCollectNFTTransferEvent(newProfileId, 1, 1, profileOwner, otherSigner); + } +} + +// New Misc contract MiscTest is BaseTest { // Negatives function testSetDefaultProfileNotExecutorFails() public {