Skip to content
New issue

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

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

Already on GitHub? # to your account

test: adapt to fresh safe deployment to avoid block height dependancy #59

Merged
merged 2 commits into from
Jun 18, 2024

Conversation

petrovska-petro
Copy link
Collaborator

closes #58
closes #46

Copy link

codecov bot commented Jun 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.05%. Comparing base (3d78a09) to head (2572cea).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #59      +/-   ##
==========================================
+ Coverage   88.77%   89.05%   +0.27%     
==========================================
  Files           3        3              
  Lines         196      201       +5     
  Branches       21       21              
==========================================
+ Hits          174      179       +5     
  Misses         21       21              
  Partials        1        1              
Files Coverage Δ
src/RoboSaverVirtualModule.sol 87.93% <100.00%> (+0.21%) ⬆️
test/BaseFixture.sol 100.00% <100.00%> (ø)

Copy link
Contributor

@gosuto-inzasheru gosuto-inzasheru left a comment

Choose a reason for hiding this comment

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

great, makes the suite more realistic by forking on latest block

@petrovska-petro petrovska-petro merged commit 59b5fdf into main Jun 18, 2024
2 checks passed
@petrovska-petro petrovska-petro deleted the feat/test-fresh-safe branch June 18, 2024 11:46
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
2 participants