test: add tests for new custom error

ERC725Alliance · Sep 21, 2023 · a0c1c1a · a0c1c1a
1 parent 044788d
commit a0c1c1a
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 0 deletions.
diff --git a/implementations/test/ERC725X.behaviour.ts b/implementations/test/ERC725X.behaviour.ts
@@ -118,6 +118,14 @@ export const shouldBehaveLikeERC725X = (buildContext: () => Promise<ERC725XTestC
 
         expect(accountOwner).to.equal(context.accounts.anyone.address);
       });
+
+      it('should revert when transferring ownership to `address(0)`', async () => {
+        await expect(
+          context.erc725X
+            .connect(context.accounts.owner)
+            .transferOwnership(ethers.constants.AddressZero),
+        ).to.be.revertedWithCustomError(context.erc725X, 'OwnableCannotSetZeroAddressAsOwner');
+      });
     });
 
     describe('When non-owner is transferring ownership', () => {

diff --git a/implementations/test/ERC725Y.behaviour.ts b/implementations/test/ERC725Y.behaviour.ts
@@ -64,6 +64,14 @@ export const shouldBehaveLikeERC725Y = (buildContext: () => Promise<ERC725YTestC
 
         expect(accountOwner).to.equal(context.accounts.anyone.address);
       });
+
+      it('should revert when transferring ownership to `address(0)`', async () => {
+        await expect(
+          context.erc725Y
+            .connect(context.accounts.owner)
+            .transferOwnership(ethers.constants.AddressZero),
+        ).to.be.revertedWithCustomError(context.erc725Y, 'OwnableCannotSetZeroAddressAsOwner');
+      });
     });
 
     describe('When non-owner is transferring ownership', () => {