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

fix(evm): proper tx gas refund outside of CallContract #2132

Merged

Unique-Divine merged 18 commits into main from fix/gas-refund

Jan 14, 2025

Merged

fix(evm): proper tx gas refund outside of CallContract #2132

Apply suggestions from code review

Codecov / codecov/patch succeeded Jan 7, 2025 in 0s

41.66% of diff hit (target 0.00%)

View this Pull Request on Codecov

41.66% of diff hit (target 0.00%)

Annotations

Check warning on line 344 in eth/rpc/backend/call_tx.go

codecov / codecov/patch

eth/rpc/backend/call_tx.go#L343-L344

Added lines #L343 - L344 were not covered by tests

Check warning on line 85 in x/evm/keeper/funtoken_from_coin.go

codecov / codecov/patch

x/evm/keeper/funtoken_from_coin.go#L85

Added line #L85 was not covered by tests

Check warning on line 91 in x/evm/keeper/funtoken_from_coin.go

codecov / codecov/patch

x/evm/keeper/funtoken_from_coin.go#L90-L91

Added lines #L90 - L91 were not covered by tests

Check warning on line 150 in x/evm/keeper/keeper.go

codecov / codecov/patch

x/evm/keeper/keeper.go#L150

Added line #L150 was not covered by tests

Check warning on line 89 in x/evm/keeper/msg_server.go

codecov / codecov/patch

x/evm/keeper/msg_server.go#L88-L89

Added lines #L88 - L89 were not covered by tests

Check warning on line 565 in x/evm/keeper/msg_server.go

codecov / codecov/patch

x/evm/keeper/msg_server.go#L565

Added line #L565 was not covered by tests

Check warning on line 571 in x/evm/keeper/msg_server.go

codecov / codecov/patch

x/evm/keeper/msg_server.go#L570-L571

Added lines #L570 - L571 were not covered by tests

View more details on Codecov