8000 test: executor operations gas test by ly0va · Pull Request #1477 · matter-labs/era-contracts · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

test: executor operations gas test #1477

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 24 commits into from
May 20, 2025
Merged

test: executor operations gas test #1477

merged 24 commits into from
May 20, 2025

Conversation

ly0va
Copy link
Member
@ly0va ly0va commented May 15, 2025

What ❔

Gas snapshot tests for executor operations

Why ❔

To know how much things cost and/or to know if we accidentally increase the cost of some operation

Checklist

  • PR title corresponds to the body of PR (we generate changelog entries from PRs).
  • Tests for the changes have been added / updated.
  • Documentation comments have been added / updated.

StanislavBreadless
StanislavBreadless reviewed May 19, 2025
View reviewed changes
@ly0va ly0va requested a review from Deniallugo as a code owner May 19, 2025 13:02
@github-actions GitHub Actions
Copy link

Coverage after merging lyova-precommit-gas-tests into sb-precommit-support-proposal will be

81.18%

Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
contracts/bridge
   BridgeHelper.sol85.71%100%100%85%31, 36, 41
   BridgedStandardERC20.sol82.14%100%92.31%80.28%120–121, 126–127, 139–140, 164, 205, 212, 219, 229–230, 63, 91
   L1ERC20Bridge.sol94%100%100%93.02%189, 208, 270
   L1Nullifier.sol83.26%100%86.21%82.80%116, 130–132, 162, 223, 226, 236, 239, 249, 252, 420–422, 426–427, 454, 520, 604, 616, 654, 656, 700, 712, 715, 717, 730, 743–744, 748–749, 769
contracts/bridge/asset-router
   AssetRouterBase.sol95.45%100%100%94.44%59, 87
   L1AssetRouter.sol88.33%100%88.89%88.24%208, 246, 262, 349, 360, 391, 430–431, 445, 476, 548, 60, 637, 648, 662, 667, 76, 84
contracts/bridge/interfaces
   AssetHandlerModifiers.sol75%100%100%66.67%13
contracts/bridge/ntv
   L1NativeTokenVault.sol84.82%100%88.24%84.21%144–150, 152, 220–221, 223, 234, 236, 240, 66
   NativeTokenVault.sol89.50%100%92%89.10%102, 234, 239, 256, 274, 279, 295, 314, 322, 488, 503, 531, 564–565, 569–570, 74
contracts/bridgehub
   Bridgehub.sol69.82%100%79.55%68.31%116, 124, 131, 137–138, 166–167, 183, 225–228, 230–231, 238–240, 243, 245–246, 248–250, 252–254, 263, 278, 302, 306–307, 309–310, 331, 334, 400, 416, 422–427, 432–433, 447, 527, 609, 718, 723, 726, 730–731, 736, 739, 779, 782, 797, 813, 819, 821, 823, 830, 841–843, 845–846, 848–849, 852–855, 857–858, 861, 863, 865–866, 868–869, 871, 884, 888, 891, 925–926, 930–931
   CTMDeploymentTracker.sol78.26%100%90%75%123, 127–128, 32, 40, 63, 90, 94, 98
   MessageRoot.sol95.08%100%100%93.88%122, 68, 89
contracts/chain-registrar
   ChainRegistrar.sol0%100%0%0%102, 104, 112–115, 130, 141, 155–156, 159, 162–163, 166, 169–170, 172–173, 177, 182–184, 190–193, 196–201, 204, 211
contracts/common
   ReentrancyGuard.sol95%100%100%94.12%79
contracts/common/libraries
   DataEncoding.sol85.11%100%100%81.58%109–110, 117–118, 163, 177, 45
   DynamicIncrementalMerkle.sol73.91%100%80%73.17%66–70, 72–74, 76–78
   FullMerkle.sol100%100%100%100%
   L2ContractHelper.sol84.91%100%100%82.22%102, 108, 112, 129, 134, 73, 79, 83
   Merkle.sol98.15%100%100%97.96%83
   MessageHashing.sol100%100%100%100%
   SemVer.sol100%100%100%100%
   SystemContractsCaller.sol0%100%0%0%102, 114, 122–125, 128, 135–139, 141–142, 32–34, 37, 44–45, 47, 49, 51, 53, 66, 69, 72, 75, 78, 83, 89, 91, 93, 96, 98
   UncheckedMath.sol100%100%100%100%
   UnsafeBytes.sol100%100%100%100%
contracts/governance
   AccessControlRestriction.sol100%100%100%100%
   ChainAdmin.sol97.87%100%100%97.30%39
   ChainAdminOwnable.sol44.83%100%40%45.83%28, 38–40, 46–48, 57, 66, 77–79, 81
   Governance.sol100%100%100%100%
   L2ProxyAdminDeployer.sol0%100%0%0%16–18, 20
   PermanentRestriction.sol88.80%100%100%87.39%104, 111–112, 200–201, 204–205, 208, 210–211, 242, 290, 312, 343
   ServerNotifier.sol93.10%100%100%90.91%55, 65
   TransitionaryOwner.sol0%100%0%0%16–17, 21–23
contracts/governance/restriction
   Restriction.sol100%100%100%100%
   RestrictionValidator.sol100%100%100%100%
contracts/state-transition
   AccessControlEnumerablePerChainUpgradeable.sol24%100%9.09%28.21%104, 110, 125, 130, 136–137, 144, 149–150, 153–155, 161, 166–168, 171, 173–174, 59–60, 68, 77, 79, 90–91, 96–97
   ChainTypeManager.sol92.55%100%94.44%92.11%116, 240–243, 258, 387, 449, 477, 484, 511, 559
   ValidatorTimelock.sol61.96%100%58.82%62.67%106–108, 116, 122–124, 128–129, 137–139, 141–142, 144–145, 147–148, 150–151, 158–159, 176, 243–244, 279, 294, 309
contracts/state-transition/chain-deps
   DiamondInit.sol92.50%100%100%92.31%50, 54, 58
   DiamondProxy.sol100%100%100%100%
   GatewayCTMDeployer.sol0%100%0%0%166–167, 170, 172–174, 176, 178, 180, 188, 190, 192, 194, 196–198, 204, 216, 224, 227–228, 230, 236, 240–241, 249, 254–256, 263–266, 281–284, 298, 303–308, 312, 325, 331, 333, 335–336, 340, 342–344, 352, 357, 361–362, 368, 374, 380, 387, 397, 403, 405, 415, 423,

@StanislavBreadless StanislavBreadless merged commit 04808e0 into sb-precommit-support-proposal May 20, 2025
22 of 23 checks passed
@StanislavBreadless StanislavBreadless deleted the lyova-precommit-gas-tests branch May 20, 2025 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@StanislavBreadless StanislavBreadless StanislavBreadless left review comments

@vladbochok vladbochok Awaiting requested review from vladbochok vladbochok is a code owner

@Deniallugo Deniallugo Awaiting requested review from Deniallugo Deniallugo is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ly0va @StanislavBreadless
0