Remove IPAssetRenderer (#14)

* feat: remove IPAssetRenderer
storyprotocol · Mar 24, 2024 · c7ac92f · c7ac92f
1 parent 08c70ba
commit c7ac92f
Show file tree

Hide file tree

Showing 10 changed files with 7 additions and 352 deletions.
diff --git a/contracts/registries/metadata/IPAssetRenderer.sol b/contracts/registries/metadata/IPAssetRenderer.sol
diff --git a/script/foundry/deployment/Main.s.sol b/script/foundry/deployment/Main.s.sol
@@ -26,7 +26,6 @@ import { IP_RESOLVER_MODULE_KEY, DISPUTE_MODULE_KEY, ROYALTY_MODULE_KEY, LICENSI
 import { IPMetadataProvider } from "contracts/registries/metadata/IPMetadataProvider.sol";
 import { IPAccountRegistry } from "contracts/registries/IPAccountRegistry.sol";
 import { IPAssetRegistry } from "contracts/registries/IPAssetRegistry.sol";
-import { IPAssetRenderer } from "contracts/registries/metadata/IPAssetRenderer.sol";
 import { ModuleRegistry } from "contracts/registries/ModuleRegistry.sol";
 import { LicenseRegistry } from "contracts/registries/LicenseRegistry.sol";
 import { LicensingModule } from "contracts/modules/licensing/LicensingModule.sol";
@@ -84,7 +83,6 @@ contract Main is Script, BroadcastManager, JsonDeploymentHandler {
     // Misc.
     Governance internal governance;
     AccessController internal accessController;
-    IPAssetRenderer internal ipAssetRenderer;
     IPResolver internal ipResolver;
 
     // Mocks
@@ -186,15 +184,6 @@ contract Main is Script, BroadcastManager, JsonDeploymentHandler {
         );
         _postdeploy(contractKey, address(ipAssetRegistry));
 
-        contractKey = "IPAssetRenderer";
-        _predeploy(contractKey);
-        ipAssetRenderer = new IPAssetRenderer(
-            address(ipAssetRegistry),
-            address(licenseRegistry),
-            address(royaltyModule)
-        );
-        _postdeploy(contractKey, address(ipAssetRenderer));
-
         contractKey = "RoyaltyModule";
         _predeploy(contractKey);
         royaltyModule = new RoyaltyModule(address(governance));

diff --git a/test/foundry/integration/BaseIntegration.t.sol b/test/foundry/integration/BaseIntegration.t.sol
@@ -27,9 +27,7 @@ contract BaseIntegration is BaseTest {
             DeployModuleCondition({ disputeModule: true, royaltyModule: true, licensingModule: true })
         );
         buildDeployPolicyCondition(DeployPolicyCondition({ arbitrationPolicySP: true, royaltyPolicyLAP: true }));
-        buildDeployMiscCondition(
-            DeployMiscCondition({ ipAssetRenderer: true, ipMetadataProvider: true, ipResolver: true })
-        );
+        buildDeployMiscCondition(DeployMiscCondition({ ipMetadataProvider: true, ipResolver: true }));
         deployConditionally();
         postDeploymentSetup();
 

diff --git a/test/foundry/modules/ModuleBase.t.sol b/test/foundry/modules/ModuleBase.t.sol
@@ -14,9 +14,7 @@ abstract contract ModuleBaseTest is BaseTest {
     /// @notice Initializes the base module for testing.
     function setUp() public virtual override(BaseTest) {
         super.setUp();
-        buildDeployMiscCondition(
-            DeployMiscCondition({ ipAssetRenderer: false, ipMetadataProvider: false, ipResolver: true })
-        );
+        buildDeployMiscCondition(DeployMiscCondition({ ipMetadataProvider: false, ipResolver: true }));
         deployConditionally();
         postDeploymentSetup();
 

diff --git a/test/foundry/modules/dispute/ArbitrationPolicySP.t.sol b/test/foundry/modules/dispute/ArbitrationPolicySP.t.sol
@@ -26,9 +26,7 @@ contract TestArbitrationPolicySP is BaseTest {
             DeployModuleCondition({ disputeModule: true, royaltyModule: false, licensingModule: false })
         );
         buildDeployPolicyCondition(DeployPolicyCondition({ arbitrationPolicySP: true, royaltyPolicyLAP: true }));
-        buildDeployMiscCondition(
-            DeployMiscCondition({ ipAssetRenderer: false, ipMetadataProvider: false, ipResolver: true })
-        );
+        buildDeployMiscCondition(DeployMiscCondition({ ipMetadataProvider: false, ipResolver: true }));
         deployConditionally();
         postDeploymentSetup();
 

diff --git a/test/foundry/modules/dispute/DisputeModule.t.sol b/test/foundry/modules/dispute/DisputeModule.t.sol
@@ -46,9 +46,7 @@ contract DisputeModuleTest is BaseTest {
             DeployModuleCondition({ disputeModule: true, royaltyModule: false, licensingModule: false })
         );
         buildDeployPolicyCondition(DeployPolicyCondition({ arbitrationPolicySP: true, royaltyPolicyLAP: true }));
-        buildDeployMiscCondition(
-            DeployMiscCondition({ ipAssetRenderer: false, ipMetadataProvider: false, ipResolver: true })
-        );
+        buildDeployMiscCondition(DeployMiscCondition({ ipMetadataProvider: false, ipResolver: true }));
         deployConditionally();
         postDeploymentSetup();
 

diff --git a/test/foundry/modules/royalty/AncestorsVaultLAP.t.sol b/test/foundry/modules/royalty/AncestorsVaultLAP.t.sol
@@ -50,9 +50,7 @@ contract TestAncestorsVaultLAP is BaseTest {
             DeployModuleCondition({ disputeModule: false, royaltyModule: true, licensingModule: false })
         );
         buildDeployPolicyCondition(DeployPolicyCondition({ arbitrationPolicySP: false, royaltyPolicyLAP: true }));
-        buildDeployMiscCondition(
-            DeployMiscCondition({ ipAssetRenderer: false, ipMetadataProvider: false, ipResolver: true })
-        );
+        buildDeployMiscCondition(DeployMiscCondition({ ipMetadataProvider: false, ipResolver: true }));
         deployConditionally();
         postDeploymentSetup();