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

[EVM] Add more ante evm unit tests #1561

Merged
merged 3 commits into from
Apr 18, 2024
Merged

Conversation

stevenlanders
Copy link
Contributor

Describe your changes and provide context

  • adds more unit tests to increase coverage
  • comments out blob validation (can be re-enabled when blob testing exists)

Testing performed to validate your change

  • unit tests

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 60.89%. Comparing base (4025070) to head (804e4ea).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##            seiv2    #1561      +/-   ##
==========================================
- Coverage   60.96%   60.89%   -0.07%     
==========================================
  Files         366      366              
  Lines       26084    26056      -28     
==========================================
- Hits        15901    15866      -35     
- Misses       9172     9184      +12     
+ Partials     1011     1006       -5     
Files Coverage Δ
x/evm/ante/basic.go 82.60% <ø> (+45.35%) ⬆️

... and 7 files with indirect coverage changes

@stevenlanders stevenlanders merged commit 9403876 into seiv2 Apr 18, 2024
45 of 47 checks passed
@stevenlanders stevenlanders deleted the more-ante-evm-unit-tests branch April 18, 2024 02:07
udpatil pushed a commit that referenced this pull request Apr 18, 2024
* add more ante tests

* ocd spacing of imports

* ocd spacing of imports
udpatil pushed a commit that referenced this pull request Apr 19, 2024
* add more ante tests

* ocd spacing of imports

* ocd spacing of imports
udpatil pushed a commit that referenced this pull request Apr 19, 2024
* add more ante tests

* ocd spacing of imports

* ocd spacing of imports
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants