Skip to content

Commit

Permalink
temporarily remove unused test code
Browse files Browse the repository at this point in the history
  • Loading branch information
kingster-will committed Nov 27, 2023
1 parent 2d28a5c commit e7c0c3f
Showing 1 changed file with 0 additions and 25 deletions.
25 changes: 0 additions & 25 deletions test/foundry/utils/BaseTest.sol
Original file line number Diff line number Diff line change
Expand Up @@ -38,9 +38,6 @@ contract BaseTest is BaseTestUtils, ProxyHelper, AccessControlHelper {
RegistrationModule public registrationModule;
TermsRepository public termsRepository;

address public defaultCollectNftImpl;
address public collectModuleImpl;

address constant upgrader = address(6969);
address constant ipOrgOwner = address(456);
address constant relManager = address(9999);
Expand Down Expand Up @@ -117,10 +114,6 @@ contract BaseTest is BaseTestUtils, ProxyHelper, AccessControlHelper {
);
moduleRegistry.registerProtocolModule(ModuleRegistryKeys.RELATIONSHIP_MODULE, relationshipModule);


defaultCollectNftImpl = _deployCollectNFTImpl();
collectModule = ICollectModule(_deployCollectModule(defaultCollectNftImpl));

IPOrgParams.RegisterIPOrgParams memory ipAssetOrgParams = IPOrgParams.RegisterIPOrgParams(
address(registry),
"IPOrgName",
Expand All @@ -146,24 +139,6 @@ contract BaseTest is BaseTestUtils, ProxyHelper, AccessControlHelper {

}

function _deployCollectNFTImpl() internal virtual returns (address) {
// TODO: temporarily commented out for alpha
// return address(new MockCollectNFT());
return address(0);
}

function _deployCollectModule(address collectNftImpl) internal virtual returns (address) {
// TODO: temporarily commented out for alpha
// collectModuleImpl = address(new MockCollectModule(address(registry), collectNftImpl));
// return _deployUUPSProxy(
// collectModuleImpl,
// abi.encodeWithSelector(
// bytes4(keccak256(bytes("initialize(address)"))), address(accessControl)
// )
// );
return address(0);
}

/// @dev Helper function for creating an IP asset for an owner and IP type.
/// TO-DO: Replace this with a simpler set of default owners that get
/// tested against. The reason this is currently added is that during
Expand Down

0 comments on commit e7c0c3f

Please sign in to comment.