fix: traceTransaction returns zero gas used by yihuang · Pull Request #1179

fix: traceTransaction returns zero gas used by yihuang · Pull Request #1179 · evmos/ethermint

Solution:
- call CaptureTxStart and CaptureTxEnd

yihuang changed the title ~~Problem: traceTransaction returns zero gas used~~ fix: traceTransaction returns zero gas used

Jul 14, 2022

HuangYi added 3 commits

July 18, 2022 11:02

mergify bot pushed a commit that referenced this pull request

Jul 25, 2022

* Problem: traceTransaction returns zero gas used

Solution:
- call CaptureTxStart and CaptureTxEnd

* fix trace unit tests

* add overflow check

* add comment

* check gas used is positive

* Update x/evm/keeper/state_transition.go

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
(cherry picked from commit b74b37f)

# Conflicts:
#	x/evm/keeper/grpc_query_test.go

yihuang pushed a commit to yihuang/ethermint that referenced this pull request

Jul 26, 2022

* Problem: traceTransaction returns zero gas used

Solution:
- call CaptureTxStart and CaptureTxEnd

* fix trace unit tests

* add overflow check

* add comment

* check gas used is positive

* Update x/evm/keeper/state_transition.go

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

hoanguyenkh pushed a commit to AstraProtocol/ethermint that referenced this pull request

Jul 27, 2022

* Problem: traceTransaction returns zero gas used

Solution:
- call CaptureTxStart and CaptureTxEnd

* fix trace unit tests

* add overflow check

* add comment

* check gas used is positive

* Update x/evm/keeper/state_transition.go

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

facs95 pushed a commit that referenced this pull request

Aug 12, 2022

* Problem: traceTransaction returns zero gas used

Solution:
- call CaptureTxStart and CaptureTxEnd

* fix trace unit tests

* add overflow check

* add comment

* check gas used is positive

* Update x/evm/keeper/state_transition.go

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

hoanguyenkh pushed a commit to AstraProtocol/ethermint that referenced this pull request

Aug 17, 2022

* Problem: traceTransaction returns zero gas used

Solution:
- call CaptureTxStart and CaptureTxEnd

* fix trace unit tests

* add overflow check

* add comment

* check gas used is positive

* Update x/evm/keeper/state_transition.go

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>

devon-chain pushed a commit to PundiAI/ethermint-old that referenced this pull request

Nov 17, 2022

* Problem: traceTransaction returns zero gas used

Solution:
- call CaptureTxStart and CaptureTxEnd

* fix trace unit tests

* add overflow check

* add comment

* check gas used is positive

* Update x/evm/keeper/state_transition.go

Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com>
(cherry picked from commit 84afdf0)