Skip to content

Commit

Permalink
Add rollupManager initializer versioning
Browse files Browse the repository at this point in the history
  • Loading branch information
ignasirv committed Aug 22, 2024
1 parent 2de7a15 commit 1b0b851
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 75 deletions.
12 changes: 12 additions & 0 deletions contracts/v2/PolygonRollupManager.sol
Original file line number Diff line number Diff line change
Expand Up @@ -382,6 +382,11 @@ contract PolygonRollupManager is
*/
event SetBatchFee(uint256 newBatchFee);

/**
* @dev Emitted when rollup manager is upgraded
*/
event UpdateRollupManagerVersion(string rollupManagerVersion);

/**
* @param _globalExitRootManager Global exit root manager address
* @param _pol POL token address
Expand All @@ -400,6 +405,13 @@ contract PolygonRollupManager is
_disableInitializers();
}

/**
* Initializer function to set new rollup manager version
*/
function initialize() external virtual reinitializer(3) {
emit UpdateRollupManagerVersion("banana");
}

///////////////////////////////////////
// Rollups management functions
///////////////////////////////////////
Expand Down
75 changes: 0 additions & 75 deletions contracts/v2/mocks/PolygonRollupManagerMockInternalTest.sol

This file was deleted.

2 changes: 2 additions & 0 deletions test/contractsv2/PolygonRollupManager-Pessimistic.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -166,6 +166,8 @@ describe("Polygon Rollup Manager with Polygon Pessimistic Consensus", () => {
emergencyCouncil.address
);

await expect(rollupManagerContract.initialize()).to.emit(rollupManagerContract, "UpdateRollupManagerVersion")

// fund sequencer address with Matic tokens
await polTokenContract.transfer(trustedSequencer.address, ethers.parseEther("1000"));
});
Expand Down

0 comments on commit 1b0b851

Please sign in to comment.