Remove global amino codec instances

[robert-zaremba]

Summary

amino codec requires to register all structures for encoding and decoding.
This is done during the module initialization process using RegisterLegacyAminoCodec function.

Problem Definition

We need to ensure that the codec we are using for encoding and decoding data structures has registered that structures in advance. In some modules (x/bank, x/auth...) we are defining a global variable amino and register local structures with it.
At the same time we are also doing the same things in module initialization from the application amino instance. The reason we use a module global variable is because it's needed in some exported functions and it was probably hard to pass amino codec as a dependency (through part of a struct or an argument). Examples: GetPubKey code,

Moreover, in some places we are saying that we use this global variables for tests, but they are declared in non-tests files and used in non-test files. Example.

Proposal

1. Either add this issue to 0.41 release tracker -> as a part of amino removal
2. or plan for refactoring.

/cc @amaurymartiny , @aaronc

---

For Admin Use

• Not duplicate issue
• Appropriate labels applied
• Appropriate contributors tagged
• Contributor assigned/self-assigned

[tac0turtle]

done with eden, amino is inferred with proto files