R4R: GenTx command refactoring - take #2

[alessio]

Don't call gaiacli tx sign. Use utils.SignStdTx() instead.

This is to avoid command redirection and reduce the use of viper's global variables.

Closes: #2875

• Linked to github-issue with discussion and accepted design OR link to spec that describes this work.
• Wrote tests
• Updated relevant documentation (docs/)
• Added entries in PENDING.md with issue #
• rereviewed Files changed in the github PR explorer

---

For Admin Use:

• Added appropriate labels to PR (ex. wip, ready-for-review, docs)
• Reviewers Assigned
• Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

[cwgoes]

Sadly the integration_tests fail 😿

[codecov]

Codecov Report

Merging #2881 into develop will decrease coverage by 0.14%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           develop   #2881      +/-   ##
==========================================
- Coverage    56.84%   56.7%   -0.15%     
==========================================
  Files          120     120              
  Lines         8298    8315      +17     
==========================================
- Hits          4717    4715       -2     
- Misses        3263    3282      +19     
  Partials       318     318

[alessio]

@cwgoes fixed, please review

[alexanderbez]

Tested and LGTM. I left a few minor cosmetic suggestions. Also, I'm not 100% sure we do commission param validations (e.g. my given rate can be > the default max rate).

[alessio]

@bez comments accepted, thanks

[alexanderbez]

LGTM. Tested and changes looks clean and concise 👌

[cwgoes]

Tested ACK