@bannynet/core-v6 0.0.24 → 0.0.25

package/test/regression/ArrayLengthValidation.t.sol

@@ -1,57 +0,0 @@
-// SPDX-License-Identifier: MIT
-pragma solidity 0.8.28;
-
-import {Test} from "forge-std/Test.sol";
-
-import {Banny721TokenUriResolver} from "../../src/Banny721TokenUriResolver.sol";
-
-/// @notice Mismatched array lengths should revert.
-contract ArrayLengthValidationTest is Test {
-    Banny721TokenUriResolver resolver;
-    address deployer = makeAddr("deployer");
-
-    function setUp() public {
-        vm.prank(deployer);
-        resolver = new Banny721TokenUriResolver(
-            "<path/>", "<necklace/>", "<mouth/>", "<eyes/>", "<alieneyes/>", deployer, address(0)
-        );
-    }
-
-    function test_setProductNames_revertsOnMismatchedLengths() public {
-        uint256[] memory upcs = new uint256[](2);
-        upcs[0] = 1;
-        upcs[1] = 2;
-
-        string[] memory names = new string[](1);
-        names[0] = "Only One";
-
-        vm.prank(deployer);
-        vm.expectRevert(Banny721TokenUriResolver.Banny721TokenUriResolver_ArrayLengthMismatch.selector);
-        resolver.setProductNames(upcs, names);
-    }
-
-    function test_setSvgContentsOf_revertsOnMismatchedLengths() public {
-        uint256[] memory upcs = new uint256[](2);
-        upcs[0] = 1;
-        upcs[1] = 2;
-
-        string[] memory contents = new string[](1);
-        contents[0] = "only one";
-
-        vm.expectRevert(Banny721TokenUriResolver.Banny721TokenUriResolver_ArrayLengthMismatch.selector);
-        resolver.setSvgContentsOf(upcs, contents);
-    }
-
-    function test_setSvgHashesOf_revertsOnMismatchedLengths() public {
-        uint256[] memory upcs = new uint256[](2);
-        upcs[0] = 1;
-        upcs[1] = 2;
-
-        bytes32[] memory hashes = new bytes32[](1);
-        hashes[0] = keccak256("test");
-
-        vm.prank(deployer);
-        vm.expectRevert(Banny721TokenUriResolver.Banny721TokenUriResolver_ArrayLengthMismatch.selector);
-        resolver.setSvgHashesOf(upcs, hashes);
-    }
-}

package/test/regression/BodyCategoryValidation.t.sol

@@ -1,147 +0,0 @@
-// SPDX-License-Identifier: MIT
-pragma solidity 0.8.28;
-
-import {Test} from "forge-std/Test.sol";
-import {JB721Tier} from "@bananapus/721-hook-v6/src/structs/JB721Tier.sol";
-import {JB721TierFlags} from "@bananapus/721-hook-v6/src/structs/JB721TierFlags.sol";
-import {IERC721Receiver} from "@openzeppelin/contracts/token/ERC721/IERC721Receiver.sol";
-
-import {Banny721TokenUriResolver} from "../../src/Banny721TokenUriResolver.sol";
-
-/// @notice Minimal mock hook.
-contract MockHook57 {
-    mapping(uint256 => address) public ownerOf;
-    address public immutable MOCK_STORE;
-    mapping(address => mapping(address => bool)) public isApprovedForAll;
-
-    constructor(address store) {
-        MOCK_STORE = store;
-    }
-
-    function STORE() external view returns (address) {
-        return MOCK_STORE;
-    }
-
-    function setOwner(uint256 tokenId, address owner) external {
-        ownerOf[tokenId] = owner;
-    }
-
-    function setApprovalForAll(address operator, bool approved) external {
-        isApprovedForAll[msg.sender][operator] = approved;
-    }
-
-    function safeTransferFrom(address from, address to, uint256 tokenId) external {
-        require(
-            msg.sender == ownerOf[tokenId] || msg.sender == from || isApprovedForAll[from][msg.sender],
-            "MockHook: not authorized"
-        );
-        ownerOf[tokenId] = to;
-        if (to.code.length > 0) {
-            bytes4 retval = IERC721Receiver(to).onERC721Received(msg.sender, from, tokenId, "");
-            require(retval == IERC721Receiver.onERC721Received.selector, "MockHook: receiver rejected");
-        }
-    }
-
-    function pricingContext() external pure returns (uint256, uint256, uint256) {
-        return (1, 18, 0);
-    }
-
-    function baseURI() external pure returns (string memory) {
-        return "ipfs://";
-    }
-}
-
-/// @notice Minimal mock store.
-contract MockStore57 {
-    mapping(address => mapping(uint256 => JB721Tier)) public tiers;
-
-    function setTier(address hook, uint256 tokenId, JB721Tier memory tier) external {
-        tiers[hook][tokenId] = tier;
-    }
-
-    function tierOfTokenId(address hook, uint256 tokenId, bool) external view returns (JB721Tier memory) {
-        return tiers[hook][tokenId];
-    }
-
-    // forge-lint: disable-next-line(mixed-case-function)
-    function encodedIPFSUriOf(address, uint256) external pure returns (bytes32) {
-        return bytes32(0);
-    }
-}
-
-/// @notice decorateBannyWith should reject non-body-category tokens as bannyBodyId.
-contract BodyCategoryValidationTest is Test {
-    Banny721TokenUriResolver resolver;
-    MockHook57 hook;
-    MockStore57 store;
-
-    address deployer = makeAddr("deployer");
-    address alice = makeAddr("alice");
-
-    uint256 constant BODY_TOKEN = 4_000_000_001; // category 0 (body)
-    uint256 constant NECKLACE_TOKEN = 10_000_000_001; // category 3 (necklace)
-
-    function setUp() public {
-        store = new MockStore57();
-        hook = new MockHook57(address(store));
-
-        vm.prank(deployer);
-        resolver = new Banny721TokenUriResolver(
-            "<path/>", "<necklace/>", "<mouth/>", "<eyes/>", "<alieneyes/>", deployer, address(0)
-        );
-
-        // Set up body token (category 0).
-        _setupTier(BODY_TOKEN, 4, 0);
-        hook.setOwner(BODY_TOKEN, alice);
-
-        // Set up necklace token (category 3) -- NOT a body.
-        _setupTier(NECKLACE_TOKEN, 10, 3);
-        hook.setOwner(NECKLACE_TOKEN, alice);
-
-        vm.prank(alice);
-        hook.setApprovalForAll(address(resolver), true);
-    }
-
-    /// @notice Passing a non-body token as bannyBodyId should revert.
-    function test_decorateBannyWith_revertsIfNotBodyCategory() public {
-        uint256[] memory outfitIds = new uint256[](0);
-
-        vm.prank(alice);
-        vm.expectRevert(Banny721TokenUriResolver.Banny721TokenUriResolver_BannyBodyNotBodyCategory.selector);
-        resolver.decorateBannyWith(address(hook), NECKLACE_TOKEN, 0, outfitIds);
-    }
-
-    /// @notice Passing a valid body token should succeed.
-    function test_decorateBannyWith_succeedsWithBodyCategory() public {
-        uint256[] memory outfitIds = new uint256[](0);
-
-        vm.prank(alice);
-        resolver.decorateBannyWith(address(hook), BODY_TOKEN, 0, outfitIds);
-        // Should not revert.
-    }
-
-    function _setupTier(uint256 tokenId, uint32 tierId, uint24 category) internal {
-        JB721Tier memory tier = JB721Tier({
-            id: tierId,
-            price: 0.01 ether,
-            remainingSupply: 100,
-            initialSupply: 100,
-            votingUnits: 0,
-            reserveFrequency: 0,
-            reserveBeneficiary: address(0),
-            encodedIPFSUri: bytes32(0),
-            category: category,
-            discountPercent: 0,
-            flags: JB721TierFlags({
-                allowOwnerMint: false,
-                transfersPausable: false,
-                cantBeRemoved: false,
-                cantIncreaseDiscountPercent: false,
-                cantBuyWithCredits: false
-            }),
-            splitPercent: 0,
-            resolvedUri: ""
-        });
-        store.setTier(address(hook), tokenId, tier);
-    }
-}

package/test/regression/BurnedTokenCheck.t.sol

@@ -1,186 +0,0 @@
-// SPDX-License-Identifier: MIT
-pragma solidity 0.8.28;
-
-import {Test} from "forge-std/Test.sol";
-import {JB721Tier} from "@bananapus/721-hook-v6/src/structs/JB721Tier.sol";
-import {JB721TierFlags} from "@bananapus/721-hook-v6/src/structs/JB721TierFlags.sol";
-import {IERC721Receiver} from "@openzeppelin/contracts/token/ERC721/IERC721Receiver.sol";
-
-import {Banny721TokenUriResolver} from "../../src/Banny721TokenUriResolver.sol";
-
-/// @notice Mock hook that supports burning (setting owner to address(0) so ownerOf reverts).
-contract MockHook62 {
-    mapping(uint256 => address) public owners;
-    address public immutable MOCK_STORE;
-    mapping(address => mapping(address => bool)) public isApprovedForAll;
-
-    constructor(address store) {
-        MOCK_STORE = store;
-    }
-
-    function STORE() external view returns (address) {
-        return MOCK_STORE;
-    }
-
-    function setOwner(uint256 tokenId, address owner) external {
-        owners[tokenId] = owner;
-    }
-
-    function ownerOf(uint256 tokenId) external view returns (address) {
-        address owner = owners[tokenId];
-        require(owner != address(0), "ERC721: token does not exist");
-        return owner;
-    }
-
-    function burn(uint256 tokenId) external {
-        owners[tokenId] = address(0);
-    }
-
-    function setApprovalForAll(address operator, bool approved) external {
-        isApprovedForAll[msg.sender][operator] = approved;
-    }
-
-    function safeTransferFrom(address from, address to, uint256 tokenId) external {
-        address owner = owners[tokenId];
-        require(owner != address(0), "ERC721: token does not exist");
-        require(
-            msg.sender == owner || msg.sender == from || isApprovedForAll[from][msg.sender], "MockHook: not authorized"
-        );
-        owners[tokenId] = to;
-        if (to.code.length > 0) {
-            bytes4 retval = IERC721Receiver(to).onERC721Received(msg.sender, from, tokenId, "");
-            require(retval == IERC721Receiver.onERC721Received.selector, "MockHook: receiver rejected");
-        }
-    }
-
-    function pricingContext() external pure returns (uint256, uint256, uint256) {
-        return (1, 18, 0);
-    }
-
-    function baseURI() external pure returns (string memory) {
-        return "ipfs://";
-    }
-}
-
-/// @notice Minimal mock store.
-contract MockStore62 {
-    mapping(address => mapping(uint256 => JB721Tier)) public tiers;
-
-    function setTier(address hook, uint256 tokenId, JB721Tier memory tier) external {
-        tiers[hook][tokenId] = tier;
-    }
-
-    function tierOfTokenId(address hook, uint256 tokenId, bool) external view returns (JB721Tier memory) {
-        return tiers[hook][tokenId];
-    }
-
-    // forge-lint: disable-next-line(mixed-case-function)
-    function encodedIPFSUriOf(address, uint256) external pure returns (bytes32) {
-        return bytes32(0);
-    }
-}
-
-/// @notice Burned equipped tokens should not lock the body.
-contract BurnedTokenCheckTest is Test {
-    Banny721TokenUriResolver resolver;
-    MockHook62 hook;
-    MockStore62 store;
-
-    address deployer = makeAddr("deployer");
-    address alice = makeAddr("alice");
-
-    uint256 constant BODY_TOKEN = 4_000_000_001;
-    uint256 constant NECKLACE_TOKEN = 10_000_000_001;
-    uint256 constant EYES_TOKEN = 30_000_000_001;
-
-    function setUp() public {
-        store = new MockStore62();
-        hook = new MockHook62(address(store));
-
-        vm.prank(deployer);
-        resolver = new Banny721TokenUriResolver(
-            "<path/>", "<necklace/>", "<mouth/>", "<eyes/>", "<alieneyes/>", deployer, address(0)
-        );
-
-        _setupTier(BODY_TOKEN, 4, 0);
-        _setupTier(NECKLACE_TOKEN, 10, 3);
-        _setupTier(EYES_TOKEN, 30, 5);
-
-        hook.setOwner(BODY_TOKEN, alice);
-        hook.setOwner(NECKLACE_TOKEN, alice);
-        hook.setOwner(EYES_TOKEN, alice);
-
-        vm.prank(alice);
-        hook.setApprovalForAll(address(resolver), true);
-    }
-
-    /// @notice If an equipped outfit is burned, the body should still be able to change outfits.
-    function test_decorateBannyWith_succeedsAfterEquippedOutfitBurned() public {
-        // Equip a necklace.
-        uint256[] memory outfitIds = new uint256[](1);
-        outfitIds[0] = NECKLACE_TOKEN;
-
-        vm.prank(alice);
-        resolver.decorateBannyWith(address(hook), BODY_TOKEN, 0, outfitIds);
-
-        // Verify necklace is equipped.
-        assertEq(resolver.wearerOf(address(hook), NECKLACE_TOKEN), BODY_TOKEN);
-
-        // Burn the necklace (simulate external burn while it's equipped).
-        hook.burn(NECKLACE_TOKEN);
-
-        // Now try to change outfits -- equip eyes instead. This should NOT revert.
-        uint256[] memory newOutfitIds = new uint256[](1);
-        newOutfitIds[0] = EYES_TOKEN;
-
-        vm.prank(alice);
-        resolver.decorateBannyWith(address(hook), BODY_TOKEN, 0, newOutfitIds);
-
-        // Verify new outfit is equipped.
-        assertEq(resolver.wearerOf(address(hook), EYES_TOKEN), BODY_TOKEN);
-    }
-
-    /// @notice If an equipped outfit is burned, the body should be able to clear all outfits.
-    function test_decorateBannyWith_canClearOutfitsAfterBurn() public {
-        // Equip a necklace.
-        uint256[] memory outfitIds = new uint256[](1);
-        outfitIds[0] = NECKLACE_TOKEN;
-
-        vm.prank(alice);
-        resolver.decorateBannyWith(address(hook), BODY_TOKEN, 0, outfitIds);
-
-        // Burn the necklace.
-        hook.burn(NECKLACE_TOKEN);
-
-        // Clear all outfits. This should NOT revert.
-        uint256[] memory emptyOutfits = new uint256[](0);
-
-        vm.prank(alice);
-        resolver.decorateBannyWith(address(hook), BODY_TOKEN, 0, emptyOutfits);
-    }
-
-    function _setupTier(uint256 tokenId, uint32 tierId, uint24 category) internal {
-        JB721Tier memory tier = JB721Tier({
-            id: tierId,
-            price: 0.01 ether,
-            remainingSupply: 100,
-            initialSupply: 100,
-            votingUnits: 0,
-            reserveFrequency: 0,
-            reserveBeneficiary: address(0),
-            encodedIPFSUri: bytes32(0),
-            category: category,
-            discountPercent: 0,
-            flags: JB721TierFlags({
-                allowOwnerMint: false,
-                transfersPausable: false,
-                cantBeRemoved: false,
-                cantIncreaseDiscountPercent: false,
-                cantBuyWithCredits: false
-            }),
-            splitPercent: 0,
-            resolvedUri: ""
-        });
-        store.setTier(address(hook), tokenId, tier);
-    }
-}

package/test/regression/CEIReorder.t.sol

@@ -1,209 +0,0 @@
-// SPDX-License-Identifier: MIT
-pragma solidity 0.8.28;
-
-import {Test} from "forge-std/Test.sol";
-import {JB721Tier} from "@bananapus/721-hook-v6/src/structs/JB721Tier.sol";
-import {JB721TierFlags} from "@bananapus/721-hook-v6/src/structs/JB721TierFlags.sol";
-import {IERC721Receiver} from "@openzeppelin/contracts/token/ERC721/IERC721Receiver.sol";
-
-import {Banny721TokenUriResolver} from "../../src/Banny721TokenUriResolver.sol";
-
-/// @notice Mock hook that records transfer ordering so we can verify CEI.
-contract MockHookI25 {
-    mapping(uint256 => address) public ownerOf;
-    address public immutable MOCK_STORE;
-    mapping(address => mapping(address => bool)) public isApprovedForAll;
-
-    /// @notice Tracks order of transfers for CEI verification.
-    uint256[] public transferLog;
-
-    constructor(address store) {
-        MOCK_STORE = store;
-    }
-
-    function STORE() external view returns (address) {
-        return MOCK_STORE;
-    }
-
-    function setOwner(uint256 tokenId, address owner) external {
-        ownerOf[tokenId] = owner;
-    }
-
-    function setApprovalForAll(address operator, bool approved) external {
-        isApprovedForAll[msg.sender][operator] = approved;
-    }
-
-    function safeTransferFrom(address from, address to, uint256 tokenId) external {
-        require(
-            msg.sender == ownerOf[tokenId] || msg.sender == from || isApprovedForAll[from][msg.sender],
-            "MockHook: not authorized"
-        );
-        ownerOf[tokenId] = to;
-        transferLog.push(tokenId);
-        if (to.code.length > 0) {
-            bytes4 retval = IERC721Receiver(to).onERC721Received(msg.sender, from, tokenId, "");
-            require(retval == IERC721Receiver.onERC721Received.selector, "MockHook: receiver rejected");
-        }
-    }
-
-    function transferLogLength() external view returns (uint256) {
-        return transferLog.length;
-    }
-
-    function pricingContext() external pure returns (uint256, uint256, uint256) {
-        return (1, 18, 0);
-    }
-
-    function baseURI() external pure returns (string memory) {
-        return "ipfs://";
-    }
-}
-
-/// @notice Minimal mock store.
-contract MockStoreI25 {
-    mapping(address => mapping(uint256 => JB721Tier)) public tiers;
-
-    function setTier(address hook, uint256 tokenId, JB721Tier memory tier) external {
-        tiers[hook][tokenId] = tier;
-    }
-
-    function tierOfTokenId(address hook, uint256 tokenId, bool) external view returns (JB721Tier memory) {
-        return tiers[hook][tokenId];
-    }
-
-    // forge-lint: disable-next-line(mixed-case-function)
-    function encodedIPFSUriOf(address, uint256) external pure returns (bytes32) {
-        return bytes32(0);
-    }
-}
-
-/// @notice _decorateBannyWithBackground follows CEI pattern.
-/// @dev The fix reordered state writes (effects) before external transfers (interactions)
-/// in _decorateBannyWithBackground. This test verifies that after a background replacement,
-/// state is consistent and both the old background return and new background custody work.
-contract CEIReorderTest is Test {
-    Banny721TokenUriResolver resolver;
-    MockHookI25 hook;
-    MockStoreI25 store;
-
-    address deployer = makeAddr("deployer");
-    address alice = makeAddr("alice");
-
-    uint256 constant BODY_TOKEN = 4_000_000_001;
-    uint256 constant BG_TOKEN_A = 5_000_000_001; // background A
-    uint256 constant BG_TOKEN_B = 5_000_000_002; // background B
-
-    function setUp() public {
-        store = new MockStoreI25();
-        hook = new MockHookI25(address(store));
-
-        vm.prank(deployer);
-        resolver = new Banny721TokenUriResolver(
-            "<path/>", "<necklace/>", "<mouth/>", "<eyes/>", "<alieneyes/>", deployer, address(0)
-        );
-
-        // Set up body (category 0).
-        _setupTier(BODY_TOKEN, 4, 0);
-        // Set up two backgrounds (category 1).
-        _setupTier(BG_TOKEN_A, 5, 1);
-        _setupTier(BG_TOKEN_B, 6, 1);
-
-        hook.setOwner(BODY_TOKEN, alice);
-        hook.setOwner(BG_TOKEN_A, alice);
-        hook.setOwner(BG_TOKEN_B, alice);
-
-        vm.prank(alice);
-        hook.setApprovalForAll(address(resolver), true);
-    }
-
-    /// @notice Replacing background A with B should update state correctly and return A to caller.
-    function test_replaceBackground_stateConsistentAfterCEIReorder() public {
-        uint256[] memory emptyOutfits = new uint256[](0);
-
-        // Attach background A.
-        vm.prank(alice);
-        resolver.decorateBannyWith(address(hook), BODY_TOKEN, BG_TOKEN_A, emptyOutfits);
-
-        // Verify background A is attached.
-        assertEq(resolver.userOf(address(hook), BG_TOKEN_A), BODY_TOKEN, "BG_A should be used by body");
-        assertEq(hook.ownerOf(BG_TOKEN_A), address(resolver), "BG_A should be held by resolver");
-
-        // Replace background A with B.
-        vm.prank(alice);
-        resolver.decorateBannyWith(address(hook), BODY_TOKEN, BG_TOKEN_B, emptyOutfits);
-
-        // Verify state is consistent after replacement.
-        assertEq(resolver.userOf(address(hook), BG_TOKEN_B), BODY_TOKEN, "BG_B should now be used by body");
-        assertEq(resolver.userOf(address(hook), BG_TOKEN_A), 0, "BG_A should no longer be used");
-        assertEq(hook.ownerOf(BG_TOKEN_B), address(resolver), "BG_B should be held by resolver");
-        assertEq(hook.ownerOf(BG_TOKEN_A), alice, "BG_A should be returned to alice");
-
-        // Verify assetIdsOf reflects the new background.
-        (uint256 backgroundId,) = resolver.assetIdsOf(address(hook), BODY_TOKEN);
-        assertEq(backgroundId, BG_TOKEN_B, "assetIdsOf should show BG_B");
-    }
-
-    /// @notice Clearing a background (setting to 0) should update state before transfer.
-    function test_clearBackground_stateConsistentAfterCEIReorder() public {
-        uint256[] memory emptyOutfits = new uint256[](0);
-
-        // Attach background A.
-        vm.prank(alice);
-        resolver.decorateBannyWith(address(hook), BODY_TOKEN, BG_TOKEN_A, emptyOutfits);
-
-        // Clear background.
-        vm.prank(alice);
-        resolver.decorateBannyWith(address(hook), BODY_TOKEN, 0, emptyOutfits);
-
-        // State should be cleared and token returned.
-        assertEq(resolver.userOf(address(hook), BG_TOKEN_A), 0, "BG_A should no longer be used");
-        assertEq(hook.ownerOf(BG_TOKEN_A), alice, "BG_A should be returned to alice");
-
-        (uint256 backgroundId,) = resolver.assetIdsOf(address(hook), BODY_TOKEN);
-        assertEq(backgroundId, 0, "assetIdsOf should show no background");
-    }
-
-    /// @notice Setting the same background again should be a no-op (no redundant transfers).
-    function test_sameBackground_noRedundantTransfer() public {
-        uint256[] memory emptyOutfits = new uint256[](0);
-
-        // Attach background A.
-        vm.prank(alice);
-        resolver.decorateBannyWith(address(hook), BODY_TOKEN, BG_TOKEN_A, emptyOutfits);
-
-        uint256 logBefore = hook.transferLogLength();
-
-        // Re-attach same background.
-        vm.prank(alice);
-        resolver.decorateBannyWith(address(hook), BODY_TOKEN, BG_TOKEN_A, emptyOutfits);
-
-        // No new transfers should have occurred for the background.
-        uint256 logAfter = hook.transferLogLength();
-        assertEq(logAfter, logBefore, "No transfers should occur when re-attaching same background");
-    }
-
-    function _setupTier(uint256 tokenId, uint32 tierId, uint24 category) internal {
-        JB721Tier memory tier = JB721Tier({
-            id: tierId,
-            price: 0.01 ether,
-            remainingSupply: 100,
-            initialSupply: 100,
-            votingUnits: 0,
-            reserveFrequency: 0,
-            reserveBeneficiary: address(0),
-            encodedIPFSUri: bytes32(0),
-            category: category,
-            discountPercent: 0,
-            flags: JB721TierFlags({
-                allowOwnerMint: false,
-                transfersPausable: false,
-                cantBeRemoved: false,
-                cantIncreaseDiscountPercent: false,
-                cantBuyWithCredits: false
-            }),
-            splitPercent: 0,
-            resolvedUri: ""
-        });
-        store.setTier(address(hook), tokenId, tier);
-    }
-}

package/test/regression/ClearMetadata.t.sol

@@ -1,52 +0,0 @@
-// SPDX-License-Identifier: MIT
-pragma solidity 0.8.28;
-
-import {Test} from "forge-std/Test.sol";
-
-import {Banny721TokenUriResolver} from "../../src/Banny721TokenUriResolver.sol";
-
-/// @notice setMetadata should allow clearing fields to empty string.
-contract ClearMetadataTest is Test {
-    Banny721TokenUriResolver resolver;
-    address deployer = makeAddr("deployer");
-
-    function setUp() public {
-        vm.prank(deployer);
-        resolver = new Banny721TokenUriResolver(
-            "<path/>", "<necklace/>", "<mouth/>", "<eyes/>", "<alieneyes/>", deployer, address(0)
-        );
-    }
-
-    function test_setMetadata_canClearToEmpty() public {
-        // First, set some values.
-        vm.prank(deployer);
-        resolver.setMetadata("Initial desc", "https://initial.url", "https://initial.base/");
-
-        assertEq(resolver.svgDescription(), "Initial desc");
-        assertEq(resolver.svgExternalUrl(), "https://initial.url");
-        assertEq(resolver.svgBaseUri(), "https://initial.base/");
-
-        // Now clear all fields by passing empty strings.
-        vm.prank(deployer);
-        resolver.setMetadata("", "", "");
-
-        // Fields should now be empty (cleared), not still holding old values.
-        assertEq(resolver.svgDescription(), "", "description should be cleared");
-        assertEq(resolver.svgExternalUrl(), "", "url should be cleared");
-        assertEq(resolver.svgBaseUri(), "", "baseUri should be cleared");
-    }
-
-    function test_setMetadata_canClearIndividualField() public {
-        // Set all fields.
-        vm.prank(deployer);
-        resolver.setMetadata("desc", "https://url", "https://base/");
-
-        // Clear only description, keep others.
-        vm.prank(deployer);
-        resolver.setMetadata("", "https://url", "https://base/");
-
-        assertEq(resolver.svgDescription(), "", "description should be cleared");
-        assertEq(resolver.svgExternalUrl(), "https://url", "url should be unchanged");
-        assertEq(resolver.svgBaseUri(), "https://base/", "baseUri should be unchanged");
-    }
-}