mirror of
https://github.com/luxfi/erc-3643.git
synced 2026-10-30 16:58:28 +00:00
* ✨ integrate foundry alongside hardhat for dual testing framework - Add foundry.toml configuration - Reorganize test structure: test/forge/ and test/hardhat/ - Add Foundry scripts to package.json (build:forge, test:forge, coverage:forge) - Update hardhat.config.ts with evmVersion for compatibility - Add GitHub workflows and coverage scripts from template - Add Husky hooks (pre-commit, post-checkout, post-merge) - Add .vscode/settings.json for better IDE support - Disable coverage checks until tests are migrated (commented with TODO) This migration is based on TokenySolutions/contracts-template and enables gradual migration of tests from Hardhat (TypeScript) to Foundry (Solidity). Related to: BT-1418 * ✅ trex factory tests * ⚰️ remove trex suit * feat(forge): reorganize Foundry test structure and sync GitHub workflows - Move TREXFactorySetup to helpers folder - Rename TREXSuite.t.sol to TREXFactory.t.sol - Update fixture to inherit from Test for proper Foundry pattern - Transfer ownership to deployer address to match Hardhat behavior - Sync GitHub workflows with package.json scripts - Update test.yml to use npm run build and npm run test - Update push_checking.yml to use unified npm run lint - Update forge_coverage.sh to use npm run coverage * fix(ci): install Foundry and skip husky in push_checking workflow - Add Foundry installation step before npm ci - Set HUSKY=0 to skip husky setup in CI environment - Fixes 'forge: not found' error during npm prepare script * 🔧 keep hardhat settings * ✅ create addressess directly * ✅ removed test_StandardAddressesSet * ✅ fix lint * 💡 removed title natspecs * ✅ migrated trex factort tests * ✅ added test module multicall * ✅ migrate agent role test * ✅ migrate claim topics registry * ✅ migrated trusted issuer registry * ✅ migrate identity registry storage * ✅ migrated identity registry tests * ✅ migrated implementation authority test * ✅ migrated token agen restrictions * ✅ migrated token information test * ✅ migrated token recover * ✅ migrated token permit * ✅ migrated token transfer * ✅ migrated utilities upgrade * ✅ migrated freeze check * ✅ migrated compliance check in utilities test * ✅ migrated eligibility check * ✅ migrated transfer check in utilities * ✅ migrated compliance test * ✅ migrated gateway tests * 🔥 removed already defined events * 🗑️ imported Initializable error from openzeppelin * ➕ added already defined events and errors * ✅ added base test token * ✅ used idFactory * ♻️ refactored compliance test * ⚰️ remoced useless vm.prank * 🔧 deleted hardhat configurations * 💚 fixed ci * refactor: remove 100% coverage requirement from forge_coverage.sh - Remove strict 100% coverage checks that were causing workflow failures - Script now only generates and displays coverage report without failing - Aligns with previous behavior where coverage was informational only * ✅ resolve getVerifiedDetails * ✅ applied _getEligibility and remove via-ir from foundry.toml * ✅ cover tokena and token permit * ✅ cover compliance * ✅ cover identity registry * ✅ increased trex gateway coverage * ✅ increase coverage for trex factory * ✅ increased AbstractModuleUpgradeable.sol coverage * ⚰️ removed abstract module * 🔧 updted generate coverage * ✅ coverage 100% identity registry * ✅ coverage 100% trex gate way * ✅ coverage trex factory * ✅ coverage 100% AbstractModuleUpgradeable by test module * ✅ coverage IAFactory.sol * ✅ coverage 100% for implementation authority * ✅ coverage 1005 abstract proxy * ✅ coverage 100% claim topic registry * ✅ coverage 100% identity registry storage * ✅ coverage 100% modular compliance proxy * ✅ coverage trusted issuer registry proxy * ✅ coverage 100% identity registry * ✅ coverage 100% token proxy * 🚨 resolve linter in token information * ✅ coverage 100% roles * 🚨 resolve linter * 🔧 added no coverage for test, _testContracts and utils * ✅ coverage 100% for token.sol * ✅ coverage 100% for modular compliance * ✅ coverage 100% trusted issuer registry * 🔧 restore package.json * ✅ coverage 100% for utlity checker * 🚨 fix linter
383 lines
17 KiB
Solidity
383 lines
17 KiB
Solidity
// SPDX-License-Identifier: UNLICENSED
|
|
pragma solidity 0.8.30;
|
|
|
|
import { UUPSUpgradeable } from "@openzeppelin/contracts-upgradeable/proxy/utils/UUPSUpgradeable.sol";
|
|
import { ERC1967Proxy } from "@openzeppelin/contracts/proxy/ERC1967/ERC1967Proxy.sol";
|
|
import { IERC165 } from "@openzeppelin/contracts/utils/introspection/IERC165.sol";
|
|
import { TestModule } from "contracts/_testContracts/TestModule.sol";
|
|
import { ModularCompliance } from "contracts/compliance/modular/ModularCompliance.sol";
|
|
import { IModule } from "contracts/compliance/modular/modules/IModule.sol";
|
|
import { OnlyBoundComplianceCanCall } from "contracts/errors/ComplianceErrors.sol";
|
|
import {
|
|
ComplianceAlreadyBound,
|
|
ComplianceNotBound,
|
|
OnlyComplianceContractCanCall
|
|
} from "contracts/errors/ComplianceErrors.sol";
|
|
import { ZeroAddress } from "contracts/errors/InvalidArgumentErrors.sol";
|
|
import { IERC173 } from "contracts/roles/IERC173.sol";
|
|
import { Test } from "forge-std/Test.sol";
|
|
|
|
/// @notice tests for TestModule multicall functionality
|
|
contract TestModuleTest is Test {
|
|
|
|
// Contracts
|
|
ModularCompliance public compliance;
|
|
TestModule public testModule;
|
|
|
|
// Standard test addresses
|
|
address public deployer = makeAddr("deployer");
|
|
address public alice = makeAddr("alice");
|
|
address public bob = makeAddr("bob");
|
|
address public another = makeAddr("another");
|
|
|
|
bytes32 constant IMPLEMENTATION_SLOT = 0x360894a13ba1a3210667c828492db98dca3e2076cc3735a920a3ca505d382bbc;
|
|
|
|
/// @notice Sets up ModularCompliance and TestModule
|
|
function setUp() public {
|
|
// Deploy ModularCompliance implementation
|
|
ModularCompliance complianceImplementation = new ModularCompliance();
|
|
|
|
// Deploy ModularCompliance proxy with init using ERC1967Proxy
|
|
bytes memory initData = abi.encodeWithSelector(ModularCompliance.init.selector);
|
|
ERC1967Proxy complianceProxy = new ERC1967Proxy(address(complianceImplementation), initData);
|
|
compliance = ModularCompliance(address(complianceProxy));
|
|
|
|
// Transfer ownership to deployer (owner is initially the test contract (address(this)) since it deploys the proxy)
|
|
compliance.transferOwnership(deployer);
|
|
|
|
// Deploy TestModule implementation
|
|
TestModule testModuleImplementation = new TestModule();
|
|
|
|
// Deploy TestModule proxy with initialize using ERC1967Proxy
|
|
bytes memory moduleInitData = abi.encodeWithSelector(TestModule.initialize.selector);
|
|
ERC1967Proxy testModuleProxy = new ERC1967Proxy(address(testModuleImplementation), moduleInitData);
|
|
testModule = TestModule(address(testModuleProxy));
|
|
|
|
// Add module to compliance
|
|
vm.prank(deployer);
|
|
compliance.addModule(address(testModule));
|
|
}
|
|
|
|
/// @notice Should execute multiple doSomething calls in a single transaction
|
|
function test_MultipleDoSomethingCalls_Success() public {
|
|
// Encode function calls
|
|
bytes memory doSomething42Data = abi.encodeWithSignature("doSomething(uint256)", 42);
|
|
bytes memory doSomething43Data = abi.encodeWithSignature("doSomething(uint256)", 43);
|
|
|
|
// Create array of call data for multicall
|
|
bytes[] memory multicallData = new bytes[](2);
|
|
multicallData[0] = doSomething42Data;
|
|
multicallData[1] = doSomething43Data;
|
|
|
|
// Encode multicall
|
|
bytes memory multicallEncoded = abi.encodeWithSignature("multicall(bytes[])", multicallData);
|
|
|
|
// Call multicall via compliance
|
|
vm.prank(deployer);
|
|
compliance.callModuleFunction(multicallEncoded, address(testModule));
|
|
|
|
// Verify state changes, the last call should overwrite the first one
|
|
assertEq(testModule.getComplianceData(address(compliance)), 43);
|
|
}
|
|
|
|
/// @notice Should execute multiple blockModule calls in a single transaction
|
|
function test_MultipleBlockModuleCalls_Success() public {
|
|
// Encode function calls
|
|
bytes memory blockModuleTrueData = abi.encodeWithSignature("blockModule(bool)", true);
|
|
bytes memory blockModuleFalseData = abi.encodeWithSignature("blockModule(bool)", false);
|
|
|
|
// Create array of call data for multicall
|
|
bytes[] memory multicallData = new bytes[](2);
|
|
multicallData[0] = blockModuleTrueData;
|
|
multicallData[1] = blockModuleFalseData;
|
|
|
|
// Encode multicall
|
|
bytes memory multicallEncoded = abi.encodeWithSignature("multicall(bytes[])", multicallData);
|
|
|
|
// Call multicall via compliance, the compliance will call the multicall in the testModule which already has multicall functionality
|
|
vm.prank(deployer);
|
|
compliance.callModuleFunction(multicallEncoded, address(testModule));
|
|
|
|
// Verify state changes, the last call should overwrite the first one
|
|
assertEq(testModule.getBlockedTransfers(address(compliance)), false);
|
|
}
|
|
|
|
/// @notice Should execute mixed doSomething and blockModule calls in a single transaction
|
|
function test_MixedDoSomethingAndBlockModuleCalls_Success() public {
|
|
// Encode function calls
|
|
bytes memory doSomething100Data = abi.encodeWithSignature("doSomething(uint256)", 100);
|
|
bytes memory blockModuleTrueData = abi.encodeWithSignature("blockModule(bool)", true);
|
|
bytes memory doSomething200Data = abi.encodeWithSignature("doSomething(uint256)", 200);
|
|
|
|
// Create array of call data for multicall
|
|
bytes[] memory multicallData = new bytes[](3);
|
|
multicallData[0] = doSomething100Data;
|
|
multicallData[1] = blockModuleTrueData;
|
|
multicallData[2] = doSomething200Data;
|
|
|
|
// Encode multicall
|
|
bytes memory multicallEncoded = abi.encodeWithSignature("multicall(bytes[])", multicallData);
|
|
|
|
// Call multicall via compliance
|
|
vm.prank(deployer);
|
|
compliance.callModuleFunction(multicallEncoded, address(testModule));
|
|
|
|
// Verify state changes, both should be set to the last values
|
|
assertEq(testModule.getComplianceData(address(compliance)), 200);
|
|
assertEq(testModule.getBlockedTransfers(address(compliance)), true);
|
|
}
|
|
|
|
/// @notice Should revert when calling multicall directly on module, it should not be from compliance!
|
|
function test_MulticallDirectlyOnModule_RevertWhen_NotFromCompliance() public {
|
|
// Encode function call
|
|
bytes memory doSomething42Data = abi.encodeWithSignature("doSomething(uint256)", 42);
|
|
|
|
// Create array of call data for multicall
|
|
bytes[] memory multicallData = new bytes[](1);
|
|
multicallData[0] = doSomething42Data;
|
|
|
|
// Expect revert when calling multicall directly on module
|
|
vm.prank(deployer);
|
|
vm.expectRevert(OnlyBoundComplianceCanCall.selector);
|
|
testModule.multicall(multicallData);
|
|
}
|
|
|
|
// ============================================
|
|
// isComplianceBound Tests
|
|
// ============================================
|
|
|
|
/// @notice Should return true when compliance is bound
|
|
function test_isComplianceBound_ReturnsTrue_WhenBound() public view {
|
|
assertTrue(testModule.isComplianceBound(address(compliance)));
|
|
}
|
|
|
|
/// @notice Should return false when compliance is not bound
|
|
function test_isComplianceBound_ReturnsFalse_WhenNotBound() public {
|
|
address unboundCompliance = makeAddr("unboundCompliance");
|
|
assertFalse(testModule.isComplianceBound(unboundCompliance));
|
|
}
|
|
|
|
// ============================================
|
|
// getNonce Tests
|
|
// ============================================
|
|
|
|
/// @notice Should return zero nonce initially
|
|
function test_getNonce_ReturnsZero_Initially() public view {
|
|
assertEq(testModule.getNonce(address(compliance)), 0);
|
|
}
|
|
|
|
/// @notice Should increment nonce when unbinding compliance
|
|
function test_getNonce_Increments_WhenUnbinding() public {
|
|
assertEq(testModule.getNonce(address(compliance)), 0);
|
|
|
|
vm.prank(address(compliance));
|
|
testModule.unbindCompliance(address(compliance));
|
|
|
|
assertEq(testModule.getNonce(address(compliance)), 1);
|
|
|
|
vm.prank(address(compliance));
|
|
testModule.bindCompliance(address(compliance));
|
|
|
|
vm.prank(address(compliance));
|
|
testModule.unbindCompliance(address(compliance));
|
|
|
|
assertEq(testModule.getNonce(address(compliance)), 2);
|
|
}
|
|
|
|
// ============================================
|
|
// supportsInterface Tests
|
|
// ============================================
|
|
|
|
/// @notice Should return false for unsupported interface
|
|
function test_supportsInterface_ReturnsFalse_ForUnsupported() public view {
|
|
bytes4 unsupportedInterfaceId = bytes4(0x12345678);
|
|
assertFalse(testModule.supportsInterface(unsupportedInterfaceId));
|
|
}
|
|
|
|
/// @notice Should return true for IModule interface
|
|
function test_supportsInterface_ReturnsTrue_ForIModule() public view {
|
|
assertTrue(testModule.supportsInterface(type(IModule).interfaceId));
|
|
}
|
|
|
|
/// @notice Should return true for IERC173 interface
|
|
function test_supportsInterface_ReturnsTrue_ForIERC173() public view {
|
|
assertTrue(testModule.supportsInterface(type(IERC173).interfaceId));
|
|
}
|
|
|
|
/// @notice Should return true for IERC165 interface
|
|
function test_supportsInterface_ReturnsTrue_ForIERC165() public view {
|
|
assertTrue(testModule.supportsInterface(type(IERC165).interfaceId));
|
|
}
|
|
|
|
// ============================================
|
|
// _authorizeUpgrade Tests (via upgradeToAndCall)
|
|
// ============================================
|
|
|
|
/// @notice Should upgrade module when called by owner (covers _authorizeUpgrade)
|
|
function test_upgradeToAndCall_Success_CoversAuthorizeUpgrade() public {
|
|
testModule.transferOwnership(deployer);
|
|
|
|
TestModule newImplementation = new TestModule();
|
|
|
|
bytes32 slotValueBefore = vm.load(address(testModule), IMPLEMENTATION_SLOT);
|
|
address oldImplementation = address(uint160(uint256(slotValueBefore)));
|
|
|
|
vm.prank(deployer);
|
|
UUPSUpgradeable(address(testModule)).upgradeToAndCall(address(newImplementation), "");
|
|
|
|
bytes32 slotValueAfter = vm.load(address(testModule), IMPLEMENTATION_SLOT);
|
|
address actualImplementation = address(uint160(uint256(slotValueAfter)));
|
|
|
|
assertEq(actualImplementation, address(newImplementation));
|
|
assertNotEq(actualImplementation, oldImplementation);
|
|
}
|
|
|
|
/// @notice Should revert upgrade when not called by owner
|
|
function test_upgradeToAndCall_RevertWhen_NotOwner() public {
|
|
testModule.transferOwnership(deployer);
|
|
|
|
TestModule newImplementation = new TestModule();
|
|
|
|
vm.prank(alice);
|
|
vm.expectRevert();
|
|
UUPSUpgradeable(address(testModule)).upgradeToAndCall(address(newImplementation), "");
|
|
}
|
|
|
|
// ============================================
|
|
// onlyBoundCompliance Modifier Tests
|
|
// ============================================
|
|
|
|
/// @notice Should succeed when compliance is bound
|
|
function test_onlyBoundCompliance_Success_WhenBound() public {
|
|
testModule.invokeOnlyBoundCompliance(address(compliance));
|
|
}
|
|
|
|
/// @notice Should revert when compliance is not bound
|
|
function test_onlyBoundCompliance_RevertWhen_NotBound() public {
|
|
address unboundCompliance = makeAddr("unboundCompliance");
|
|
vm.expectRevert(ComplianceNotBound.selector);
|
|
testModule.invokeOnlyBoundCompliance(unboundCompliance);
|
|
}
|
|
|
|
// ============================================
|
|
// onlyComplianceCall Modifier Tests
|
|
// ============================================
|
|
|
|
/// @notice Should revert when called from non-bound compliance address
|
|
function test_onlyComplianceCall_RevertWhen_NotBoundCompliance() public {
|
|
address nonCompliance = makeAddr("nonCompliance");
|
|
vm.prank(nonCompliance);
|
|
vm.expectRevert(OnlyBoundComplianceCanCall.selector);
|
|
testModule.doSomething(42);
|
|
}
|
|
|
|
/// @notice Should revert when called from non-bound compliance address
|
|
function test_onlyComplianceCall_RevertWhen_NotBoundCompliance_BlockModule() public {
|
|
address nonCompliance = makeAddr("nonCompliance2");
|
|
vm.prank(nonCompliance);
|
|
vm.expectRevert(OnlyBoundComplianceCanCall.selector);
|
|
testModule.blockModule(true);
|
|
}
|
|
|
|
/// @notice Should revert when moduleTransferAction called from non-bound compliance
|
|
function test_onlyComplianceCall_RevertWhen_NotBoundCompliance_ModuleTransferAction() public {
|
|
address nonCompliance = makeAddr("nonCompliance3");
|
|
vm.prank(nonCompliance);
|
|
vm.expectRevert(OnlyBoundComplianceCanCall.selector);
|
|
testModule.moduleTransferAction(alice, bob, 100);
|
|
}
|
|
|
|
/// @notice Should revert when moduleMintAction called from non-bound compliance
|
|
function test_onlyComplianceCall_RevertWhen_NotBoundCompliance_ModuleMintAction() public {
|
|
address nonCompliance = makeAddr("nonCompliance4");
|
|
vm.prank(nonCompliance);
|
|
vm.expectRevert(OnlyBoundComplianceCanCall.selector);
|
|
testModule.moduleMintAction(alice, 100);
|
|
}
|
|
|
|
/// @notice Should revert when moduleBurnAction called from non-bound compliance
|
|
function test_onlyComplianceCall_RevertWhen_NotBoundCompliance_ModuleBurnAction() public {
|
|
address nonCompliance = makeAddr("nonCompliance5");
|
|
vm.prank(nonCompliance);
|
|
vm.expectRevert(OnlyBoundComplianceCanCall.selector);
|
|
testModule.moduleBurnAction(alice, 100);
|
|
}
|
|
|
|
// ============================================
|
|
// bindCompliance() Tests
|
|
// ============================================
|
|
|
|
/// @notice Should revert when compliance address is zero
|
|
function test_bindCompliance_RevertWhen_ZeroAddress() public {
|
|
vm.prank(address(compliance));
|
|
vm.expectRevert(ZeroAddress.selector);
|
|
testModule.bindCompliance(address(0));
|
|
}
|
|
|
|
/// @notice Should revert when compliance is already bound
|
|
function test_bindCompliance_RevertWhen_AlreadyBound() public {
|
|
// First unbind the compliance that was bound in setUp
|
|
vm.prank(address(compliance));
|
|
testModule.unbindCompliance(address(compliance));
|
|
|
|
// Bind it again
|
|
vm.prank(address(compliance));
|
|
testModule.bindCompliance(address(compliance));
|
|
|
|
// Try to bind again (should revert)
|
|
vm.prank(address(compliance));
|
|
vm.expectRevert(ComplianceAlreadyBound.selector);
|
|
testModule.bindCompliance(address(compliance));
|
|
}
|
|
|
|
/// @notice Should revert when called from non-compliance address
|
|
function test_bindCompliance_RevertWhen_NotFromCompliance() public {
|
|
// Use a new unbound compliance address to avoid hitting ComplianceAlreadyBound
|
|
address newCompliance = makeAddr("newCompliance");
|
|
address nonCompliance = makeAddr("nonCompliance");
|
|
vm.prank(nonCompliance);
|
|
vm.expectRevert(OnlyComplianceContractCanCall.selector);
|
|
testModule.bindCompliance(newCompliance);
|
|
}
|
|
|
|
// ============================================
|
|
// unbindCompliance() Tests
|
|
// ============================================
|
|
|
|
/// @notice Should revert when compliance address is zero
|
|
function test_unbindCompliance_RevertWhen_ZeroAddress() public {
|
|
vm.prank(address(compliance));
|
|
vm.expectRevert(ZeroAddress.selector);
|
|
testModule.unbindCompliance(address(0));
|
|
}
|
|
|
|
/// @notice Should revert when called from non-compliance address
|
|
/// @dev This test covers the onlyComplianceCall modifier which checks first
|
|
function test_unbindCompliance_RevertWhen_NotFromCompliance() public {
|
|
address nonCompliance = makeAddr("nonCompliance");
|
|
vm.prank(nonCompliance);
|
|
// The onlyComplianceCall modifier checks first, so it reverts with OnlyBoundComplianceCanCall
|
|
vm.expectRevert(OnlyBoundComplianceCanCall.selector);
|
|
testModule.unbindCompliance(address(compliance));
|
|
}
|
|
|
|
/// @notice Should revert when msg.sender is bound compliance but _compliance parameter is different
|
|
function test_unbindCompliance_RevertWhen_ComplianceMismatch() public {
|
|
// Deploy a second compliance and bind it
|
|
ModularCompliance compliance2Implementation = new ModularCompliance();
|
|
bytes memory initData2 = abi.encodeWithSelector(ModularCompliance.init.selector);
|
|
ERC1967Proxy compliance2Proxy = new ERC1967Proxy(address(compliance2Implementation), initData2);
|
|
ModularCompliance compliance2 = ModularCompliance(address(compliance2Proxy));
|
|
|
|
// Bind compliance2 to the module
|
|
vm.prank(address(compliance2));
|
|
testModule.bindCompliance(address(compliance2));
|
|
|
|
// Try to unbind compliance from compliance2 (should revert because msg.sender != _compliance)
|
|
vm.prank(address(compliance2));
|
|
vm.expectRevert(OnlyComplianceContractCanCall.selector);
|
|
testModule.unbindCompliance(address(compliance));
|
|
}
|
|
|
|
}
|