Commit c54de600 authored by Mark Tyneway's avatar Mark Tyneway

develop merge fixes

parent 1d91ac6d
......@@ -56,7 +56,7 @@ describe('Native ETH Integration Tests', async () => {
it('Should estimate gas for ETH withdraw', async () => {
const amount = utils.parseEther('0.5')
const gas = await env.ovmEth.estimateGas.withdraw(amount, 0, '0xFFFF')
expect(gas).to.be.deep.eq(BigNumber.from(6140049))
expect(gas).to.be.deep.eq(BigNumber.from(6580050))
})
})
......
......@@ -19,7 +19,10 @@ import { RollupDeployConfig } from '../src/contract-deployment'
ovmGlobalContext: {
ovmCHAINID: parseInt(CHAIN_ID, 10),
},
gasPriceOracleOwner: GAS_PRICE_ORACLE_OWNER
gasPriceOracleConfig: {
owner: GAS_PRICE_ORACLE_OWNER,
initialGasPrice: 0,
}
}
const dump = await makeStateDump(config as RollupDeployConfig)
......
......@@ -35,7 +35,10 @@ export interface RollupDeployConfig {
owner: string | Signer
allowArbitraryContractDeployment: boolean
}
gasPriceOracleOwner: string
gasPriceOracleConfig: {
owner: string | Signer
initialGasPrice: number
}
addressManager?: string
dependencies?: string[]
deployOverrides: Overrides
......@@ -262,7 +265,15 @@ export const makeContractDeployConfig = async (
},
OVM_GasPriceOracle: {
factory: getContractFactory('OVM_GasPriceOracle'),
params: [config.gasPriceOracleOwner],
params: [
(() => {
if (typeof config.gasPriceOracleConfig.owner !== 'string') {
return config.gasPriceOracleConfig.owner.getAddress()
}
return config.gasPriceOracleConfig.owner
})(),
config.gasPriceOracleConfig.initialGasPrice,
],
},
}
}
......@@ -140,7 +140,10 @@ export const makeStateDump = async (cfg: RollupDeployConfig): Promise<any> => {
],
deployOverrides: {},
waitForReceipts: false,
gasPriceOracleOwner: cfg.gasPriceOracleOwner,
gasPriceOracleConfig: {
owner: signer,
initialGasPrice: 0,
},
}
config = { ...config, ...cfg }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment