added some tests to ensure EOA registries fail

Transient-Labs · Jan 4, 2024 · fb7b8bb · fb7b8bb
1 parent b86ece6
commit fb7b8bb
Show file tree

Hide file tree

Showing 2 changed files with 94 additions and 0 deletions.
diff --git a/test/erc-721/ERC721TL.t.sol b/test/erc-721/ERC721TL.t.sol
@@ -2538,6 +2538,23 @@ contract ERC721TLTest is Test {
         assertEq(address(tokenContract.blocklistRegistry()), blocklistRegistry);
     }
 
+    function test_blocklist_eoa() public {
+        // update blocklist registry to EOA
+        tokenContract.setBlockListRegistry(blocklistRegistry);
+
+        // mint
+        tokenContract.mint(address(this), "uri");
+
+        // expect revert
+        vm.expectRevert();
+        tokenContract.approve(address(10), 1);
+        vm.expectRevert();
+        tokenContract.setApprovalForAll(address(10), true);
+
+        // expect can set approval for all to false regardless
+        tokenContract.setApprovalForAll(address(10), false);
+    }
+
     function test_blocklist_mint(address collector, address operator) public {
         // limit fuzz
         vm.assume(collector != address(0));
@@ -2781,4 +2798,35 @@ contract ERC721TLTest is Test {
         tokenContract.setNftDelegationRegistry(nftDelegationRegistry);
         assertEq(address(tokenContract.tlNftDelegationRegistry()), nftDelegationRegistry);
     }
+
+    function test_delegation_eoa() public {
+        // set delegation registry to eoa
+        tokenContract.setNftDelegationRegistry(nftDelegationRegistry);
+
+        // mint
+        tokenContract.mint(address(1), "uri");
+
+        // add story reverts for delegate but not collector
+        vm.expectRevert();
+        vm.prank(address(2));
+        tokenContract.addStory(1, "", "story");
+
+        vm.prank(address(1));
+        tokenContract.addStory(1, "", "story");
+
+        // synergy reverts open for delegate but deterministic for collector
+        vm.expectRevert();
+        vm.prank(address(2));
+        tokenContract.acceptTokenUriUpdate(1);
+        vm.expectRevert();
+        vm.prank(address(2));
+        tokenContract.rejectTokenUriUpdate(1);
+
+        vm.expectRevert(ERC721TL.NoTokenUriUpdateAvailable.selector);
+        vm.prank(address(1));
+        tokenContract.acceptTokenUriUpdate(1);
+        vm.expectRevert(ERC721TL.NoTokenUriUpdateAvailable.selector);
+        vm.prank(address(1));
+        tokenContract.rejectTokenUriUpdate(1);
+    }
 }
diff --git a/test/erc-721/multi-metadata/ERC7160TL.t.sol b/test/erc-721/multi-metadata/ERC7160TL.t.sol
@@ -2660,6 +2660,23 @@ contract ERC7160TLTest is Test {
         assertEq(address(tokenContract.blocklistRegistry()), blocklistRegistry);
     }
 
+    function test_blocklist_eoa() public {
+        // update blocklist registry to EOA
+        tokenContract.setBlockListRegistry(blocklistRegistry);
+
+        // mint
+        tokenContract.mint(address(this), "uri");
+
+        // expect revert
+        vm.expectRevert();
+        tokenContract.approve(address(10), 1);
+        vm.expectRevert();
+        tokenContract.setApprovalForAll(address(10), true);
+
+        // expect can set approval for all to false regardless
+        tokenContract.setApprovalForAll(address(10), false);
+    }
+
     function test_blocklist_mint(address collector, address operator) public {
         // limit fuzz
         vm.assume(collector != address(0));
@@ -2903,4 +2920,33 @@ contract ERC7160TLTest is Test {
         tokenContract.setNftDelegationRegistry(nftDelegationRegistry);
         assertEq(address(tokenContract.tlNftDelegationRegistry()), nftDelegationRegistry);
     }
+
+    function test_delegation_eoa() public {
+        // set delegation registry to eoa
+        tokenContract.setNftDelegationRegistry(nftDelegationRegistry);
+
+        // mint
+        tokenContract.mint(address(1), "uri");
+
+        // add story reverts for delegate but not collector
+        vm.expectRevert();
+        vm.prank(address(2));
+        tokenContract.addStory(1, "", "story");
+
+        vm.prank(address(1));
+        tokenContract.addStory(1, "", "story");
+
+        // pin and unpin metadata reverts for delegate but not collector
+        vm.expectRevert();
+        vm.prank(address(2));
+        tokenContract.pinTokenURI(1, 0);
+        vm.expectRevert();
+        vm.prank(address(2));
+        tokenContract.unpinTokenURI(1);
+
+        vm.prank(address(1));
+        tokenContract.pinTokenURI(1, 0);
+        vm.prank(address(1));
+        tokenContract.unpinTokenURI(1);
+    }
 }