Commit c0ba7548 authored by ben-chain's avatar ben-chain Committed by GitHub

split out OVM_Proposer role (#252)

Co-authored-by: default avatarMaurelian <maurelian@protonmail.ch>
parent 093ec7eb
...@@ -318,7 +318,7 @@ contract OVM_StateCommitmentChain is iOVM_StateCommitmentChain, Lib_AddressResol ...@@ -318,7 +318,7 @@ contract OVM_StateCommitmentChain is iOVM_StateCommitmentChain, Lib_AddressResol
) )
internal internal
{ {
address sequencer = resolve("OVM_Sequencer"); address sequencer = resolve("OVM_Proposer");
(uint40 totalElements, uint40 lastSequencerTimestamp) = _getBatchExtraData(); (uint40 totalElements, uint40 lastSequencerTimestamp) = _getBatchExtraData();
if (msg.sender == sequencer) { if (msg.sender == sequencer) {
......
...@@ -69,7 +69,7 @@ contract mockOVM_BondManager is iOVM_BondManager, Lib_AddressResolver { ...@@ -69,7 +69,7 @@ contract mockOVM_BondManager is iOVM_BondManager, Lib_AddressResolver {
) )
{ {
// Only authenticate sequencer to submit state root batches. // Only authenticate sequencer to submit state root batches.
return _who == resolve("OVM_Sequencer"); return _who == resolve("OVM_Proposer");
} }
function getGasSpent( function getGasSpent(
......
...@@ -141,6 +141,9 @@ export const makeContractDeployConfig = async ( ...@@ -141,6 +141,9 @@ export const makeContractDeployConfig = async (
await _sendTx( await _sendTx(
AddressManager.setAddress('OVM_Sequencer', sequencerAddress) AddressManager.setAddress('OVM_Sequencer', sequencerAddress)
) )
await _sendTx(
AddressManager.setAddress('OVM_Proposer', sequencerAddress)
)
await _sendTx(AddressManager.setAddress('Sequencer', sequencerAddress)) await _sendTx(AddressManager.setAddress('Sequencer', sequencerAddress))
}, },
}, },
......
...@@ -54,7 +54,7 @@ describe('OVM_StateCommitmentChain', () => { ...@@ -54,7 +54,7 @@ describe('OVM_StateCommitmentChain', () => {
Mock__OVM_BondManager.smocked.isCollateralized.will.return.with(true) Mock__OVM_BondManager.smocked.isCollateralized.will.return.with(true)
await AddressManager.setAddress( await AddressManager.setAddress(
'OVM_Sequencer', 'OVM_Proposer',
await sequencer.getAddress() await sequencer.getAddress()
) )
}) })
......
...@@ -25,11 +25,11 @@ describe('mockOVM_BondManager', () => { ...@@ -25,11 +25,11 @@ describe('mockOVM_BondManager', () => {
await ethers.getContractFactory('mockOVM_BondManager') await ethers.getContractFactory('mockOVM_BondManager')
).deploy(AddressManager.address) ).deploy(AddressManager.address)
AddressManager.setAddress('OVM_Sequencer', await sequencer.getAddress()) AddressManager.setAddress('OVM_Proposer', await sequencer.getAddress())
}) })
describe('isCollateralized', () => { describe('isCollateralized', () => {
it('should return true for OVM_Sequencer', async () => { it('should return true for OVM_Proposer', async () => {
expect( expect(
await mockOVM_BondManager.isCollateralized(await sequencer.getAddress()) await mockOVM_BondManager.isCollateralized(await sequencer.getAddress())
).to.equal(true) ).to.equal(true)
......
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