Skip to content

Formal verification: native token balance updates#582

Merged
mmv08 merged 4 commits intomainfrom
certora/native-balance-update
Jun 15, 2023
Merged

Formal verification: native token balance updates#582
mmv08 merged 4 commits intomainfrom
certora/native-balance-update

Conversation

@mmv08
Copy link
Copy Markdown
Contributor

@mmv08 mmv08 commented Jun 2, 2023

This PR:

  • Adds a rule for which methods can update the native token balance
  • Adds a unit test rule per particular method
  • Adds getStorageAt to the list of unreachable methods because it doesn't work with storage hooks

@mmv08 mmv08 force-pushed the certora/native-balance-update branch from e48e17b to 23da7f7 Compare June 2, 2023 13:26
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 2, 2023

Pull Request Test Coverage Report for Build 5156037295

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 92.8%

Totals Coverage Status
Change from base Build 5134646122: 0.0%
Covered Lines: 308
Relevant Lines: 330

💛 - Coveralls

@mmv08 mmv08 force-pushed the certora/native-balance-update branch from 23da7f7 to b2447f4 Compare June 2, 2023 13:28
@mmv08 mmv08 force-pushed the certora/native-balance-update branch from b2447f4 to 38f5fd9 Compare June 2, 2023 13:29
@mmv08 mmv08 requested review from a team, Uxio0, akshay-ap and rmeissner and removed request for a team June 2, 2023 13:40
@mmv08 mmv08 merged commit 36a07fe into main Jun 15, 2023
@mmv08 mmv08 deleted the certora/native-balance-update branch June 15, 2023 14:29
@github-actions github-actions bot locked and limited conversation to collaborators Jun 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants