Limit the creation of uncessary EncodingConfigs

[robert-zaremba]

Summary

In some tests we are creating and using encoding config in addition to the codec object in app:

suite.encodingConfig = simapp.MakeEncodingConfig()

Moreover we have RegisterInterfaces module functions which use a module global codec variable.

Related to:

• Refactore simapp.MakeEncodingConfig #7642 Refactore simapp.MakeEncodingConfig
• Remove global amino codec instances #7310 Remove global amino codec instances

Problem Definition

Managing codec objects is cumbersome and error prone. Errors usually panic and it's hard to debug.

Proposal

Make a canonical way how we will use codec in tests and in modules.

---

For Admin Use

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

[aaronc]

Ideally the tests for each module would not depend on all of simapp. Instead each module would be able to quickly bundle up the modules it needs for testing into a lightweight "sim app". Obviously this would be a pretty big lift given the current state of how modules are initialized. But if we are aiming towards a 1.0 with more decoupling it's something good to aim for.

[julienrbrt]

Fixed with #12747.