Fix on catalog factory, transfer ownership to deployer.

rmrk-team · Apr 29, 2024 · 85f9e55 · 85f9e55
1 parent 67d5853
commit 85f9e55
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/contracts/implementations/catalog/RMRKCatalogFactory.sol b/contracts/implementations/catalog/RMRKCatalogFactory.sol
@@ -26,6 +26,7 @@ contract RMRKCatalogFactory {
     ) public returns (address) {
         RMRKCatalogImpl catalog = new RMRKCatalogImpl(metadataURI, type_);
         _deployerCatalogs[msg.sender].push(address(catalog));
+        catalog.transferOwnership(msg.sender);
         emit CatalogDeployed(msg.sender, address(catalog));
         return address(catalog);
     }

diff --git a/test/implementations/catalogFactory.ts b/test/implementations/catalogFactory.ts
@@ -32,11 +32,10 @@ describe('CatalogImpl', async () => {
       throw new Error('Catalog address not found');
     }
     const catalog = await ethers.getContractAt('RMRKCatalogImpl', catalogAddress);
-    const metadataURI = await catalog.getMetadataURI();
-    const mediaType = await catalog.getType();
 
-    expect(metadataURI).to.equal('ipfs://catalogMetadata');
-    expect(mediaType).to.equal('img/jpeg');
+    expect(await catalog.getMetadataURI()).to.equal('ipfs://catalogMetadata');
+    expect(await catalog.getType()).to.equal('img/jpeg');
+    expect(await catalog.owner()).to.equal(deployer1.address);
   });
 
   it('can get catalogs deployed by a deployer', async () => {