empty from for gas estimate [WIP]

[thomas-nguy]

Closes: #201

Description

Allows empty from field when simulating transaction

---

For contributor use:

• Targeted PR against correct branch (see CONTRIBUTING.md)
• Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
• Code follows the module structure standards.
• Wrote unit and integration tests
• Updated relevant documentation (docs/) or specification (x/<module>/spec/)
• Added relevant godoc comments.
• Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
• Re-reviewed Files changed in the Github PR explorer

---

For admin use:

• Added appropriate labels to PR (ex. WIP, R4R, docs, etc)
• Reviewers assigned
• Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

[codecov]

Codecov Report

Merging #212 (3ea09bf) into main (735f00d) will decrease coverage by 0.12%.
The diff coverage is 8.33%.

@@            Coverage Diff             @@
##             main     #212      +/-   ##
==========================================
- Coverage   47.64%   47.51%   -0.13%     
==========================================
  Files          44       44              
  Lines        3031     3039       +8     
==========================================
  Hits         1444     1444              
- Misses       1508     1511       +3     
- Partials       79       84       +5     

Impacted Files	Coverage Δ
x/evm/keeper/state_transition.go	0.00% <0.00%> (ø)
app/ante/eth.go	83.53% <9.09%> (-3.50%)	⬇️

[leejw51crypto]

lgtm

[fedekunze]

is this PR still WIP?

[thomas-nguy]

Yes,

We need to be able to skip the "gas refund" in case of simulate in state_transition.go. I havent found a way to do it yet>
Maybe #236 might help

[yihuang]

I think #236 fixes this empty from field issue, according to my test.

[thomas-nguy]

Ok closing the PR as it will be fixed in #236