Remove redundant check when owner is set to 0

gnosisguild · Sep 10, 2021 · ad2579a · ad2579a
1 parent 1bef9c1
commit ad2579a
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/contracts/ScopeGuard.sol b/contracts/ScopeGuard.sol
@@ -25,7 +25,6 @@ contract ScopeGuard is FactoryFriendly, BaseGuard {
     function setUp(bytes memory initializeParams) public override {
         __Ownable_init();
         address _owner = abi.decode(initializeParams, (address));
-        require(_owner != address(0), "Owner can not be zero address");
 
         transferOwnership(_owner);
 

diff --git a/test/ScopeGuard.spec.ts b/test/ScopeGuard.spec.ts
@@ -46,7 +46,7 @@ describe("ScopeGuard", async () => {
     it("throws if owner is zero address", async () => {
       const Guard = await hre.ethers.getContractFactory("ScopeGuard");
       await expect(Guard.deploy(AddressZero)).to.be.revertedWith(
-        "Owner can not be zero address"
+        "Ownable: new owner is the zero address"
       );
     });