Remove onlyOwner modifier from migrateToMCD method

patitonar · patitonar · commit 251f9495578c · 2019-12-06T14:31:41.000-03:00
diff --git a/contracts/upgradeable_contracts/erc20_to_native/ForeignBridgeErcToNative.sol b/contracts/upgradeable_contracts/erc20_to_native/ForeignBridgeErcToNative.sol
@@ -97,7 +97,7 @@ contract ForeignBridgeErcToNative is BasicForeignBridge, ERC20Bridge, OtherSideB
         super._relayTokens(_sender, _receiver, _amount);
     }
 
-    function migrateToMCD() external onlyOwner {
+    function migrateToMCD() external {
         bytes32 storageAddress = 0x3378953eb16363e06fd9ea9701d36ed7285d206d9de7df55b778462d74596a89; // keccak256(abi.encodePacked("migrationToMcdCompleted"))
         require(!boolStorage[storageAddress]);
 
diff --git a/test/erc_to_native/foreign_bridge.test.js b/test/erc_to_native/foreign_bridge.test.js
@@ -887,9 +887,6 @@ contract('ForeignBridge_ERC20_to_Native', async accounts => {
 
       // When
 
-      // should be called by owner
-      await foreignBridge.migrateToMCD({ from: accounts[5] }).should.be.rejectedWith(ERROR_MSG)
-
       const { logs } = await foreignBridge.migrateToMCD({ from: owner }).should.be.fulfilled
 
       // can't migrate token again

Original file line number	Diff line number	Diff line change
`@@ -97,7 +97,7 @@ contract ForeignBridgeErcToNative is BasicForeignBridge, ERC20Bridge, OtherSideB`
`97`	`97`	`super._relayTokens(_sender, _receiver, _amount);`
`98`	`98`	`}`
`99`	`99`
`100`		`- function migrateToMCD() external onlyOwner {`
	`100`	`+ function migrateToMCD() external {`
`101`	`101`	`bytes32 storageAddress = 0x3378953eb16363e06fd9ea9701d36ed7285d206d9de7df55b778462d74596a89; // keccak256(abi.encodePacked("migrationToMcdCompleted"))`
`102`	`102`	`require(!boolStorage[storageAddress]);`
`103`	`103`