Skip to content

ctb: remove unnecessary vm.assumes#13264

Merged
AmadiMichael merged 9 commits intodevelopfrom
sc/rm-unnecessary-vm-assume
Dec 6, 2024
Merged

ctb: remove unnecessary vm.assumes#13264
AmadiMichael merged 9 commits intodevelopfrom
sc/rm-unnecessary-vm-assume

Conversation

@AmadiMichael
Copy link
Copy Markdown
Contributor

@AmadiMichael AmadiMichael commented Dec 5, 2024

remove unnecessary vm.assume()s

@codecov
Copy link
Copy Markdown

codecov bot commented Dec 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 43.89%. Comparing base (f21f95e) to head (fbf6875).
Report is 1 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #13264      +/-   ##
===========================================
- Coverage    46.92%   43.89%   -3.03%     
===========================================
  Files          926      813     -113     
  Lines        77805    72710    -5095     
  Branches       849        0     -849     
===========================================
- Hits         36511    31918    -4593     
+ Misses       38641    38242     -399     
+ Partials      2653     2550     -103     
Flag Coverage Δ
cannon-go-tests-32 61.85% <ø> (ø)
cannon-go-tests-64 56.72% <ø> (ø)
contracts-bedrock-tests ?

Flags with carried forward coverage won't be shown. Click here to find out more.

see 123 files with indirect coverage changes

@AmadiMichael AmadiMichael marked this pull request as ready for review December 6, 2024 07:12
@AmadiMichael AmadiMichael requested a review from a team as a code owner December 6, 2024 07:12
@sebastianst sebastianst changed the title test utils ctb: test utils Dec 6, 2024
@AmadiMichael AmadiMichael changed the title ctb: test utils ctb: remove unnecessary vm.assumes Dec 6, 2024
@AmadiMichael AmadiMichael added this pull request to the merge queue Dec 6, 2024
Merged via the queue into develop with commit f0461a1 Dec 6, 2024
@AmadiMichael AmadiMichael deleted the sc/rm-unnecessary-vm-assume branch December 6, 2024 19:00
mslipper added a commit that referenced this pull request Dec 6, 2024
sigma pushed a commit that referenced this pull request Dec 19, 2024
* test utils

* test utils

* rm unncesessary vm.assume

* rm unncesessary vm.assume

* rm unncesessary vm.assume

* fixes...

* rm testutils
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