seadrop icon indicating copy to clipboard operation
seadrop copied to clipboard

Add upgradeable ERC721 SeaDrop proxy tests

Open rofergon opened this issue 2 months ago • 0 comments

Title: Add upgradeable ERC721 SeaDrop proxy tests

Motivation

The upgradeable package relied on Hardhat/OZ upgrades but shipped without any tests, so the proxy-safe ERC721 implementation had zero coverage. Without parity checks we cannot detect regressions in owner-only forwarding, mint gating, or storage layout across upgrades.

Solution

Added a lightweight SeaDrop mock plus a V2 token stub, then wrote a dedicated Hardhat test suite under src-upgradeable/test/ that deploys the proxy, verifies privileged configuration/minting mirrors the non-upgradeable contract, and asserts state survives upgrades. Wired the upgradeable Hardhat config/tsconfig to compile these specs and documented how to run yarn test:upgradeable. (Tests currently fail here because Node/yarn aren’t installed in this environment.)

rofergon avatar Nov 17 '25 23:11 rofergon