Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(op-deployer): add test for block time override #12276

Merged
merged 1 commit into from
Oct 3, 2024

Conversation

mds1
Copy link
Contributor

@mds1 mds1 commented Oct 2, 2024

Add test that block time can be overridden in the intent using the globalDeployOverrides section. For example:

[globalDeployOverrides]
  l2BlockTime = 1

@mslipper Let me know if there's a better way to add this test. I duplicated a lot of code from TestEndToEndApply, but can dedupe some setup code if this is the right place for this test

@mds1 mds1 requested a review from mslipper October 2, 2024 23:01
@mds1 mds1 requested review from a team as code owners October 2, 2024 23:01
@mds1 mds1 requested a review from protolambda October 2, 2024 23:01
@mds1 mds1 changed the title feat(op-deployer): allow block time override feat(op-deployer): add test block time override Oct 3, 2024
@mds1 mds1 changed the title feat(op-deployer): add test block time override feat(op-deployer): add test for block time override Oct 3, 2024
Copy link
Contributor

@blmalone blmalone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, optional comment.

@mds1 mds1 added this pull request to the merge queue Oct 3, 2024
Merged via the queue into develop with commit c4119f1 Oct 3, 2024
56 checks passed
@mds1 mds1 deleted the feat/block-time-override branch October 3, 2024 17:41
mininny pushed a commit to testinprod-io/optimism that referenced this pull request Oct 3, 2024
protolambda pushed a commit that referenced this pull request Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants