fix: add init logic of module accounts just in case

[jaeseung-bae]

Description

• (audit issue) add init logic of foundation module accounts to InitGenesis in order to eliminate potential panic
• ~NOTE: This is likely to change account number state in auth module.~ --> InitGenesis never runs more than once

Motivation and context

How has this been tested?

Screenshots (if appropriate):

Checklist:

• [x] I followed the contributing guidelines and code of conduct.
• [x] I have added a relevant changelog to CHANGELOG.md
• [ ] I have added tests to cover my changes.
• [ ] I have updated the documentation accordingly.
• [ ] I have updated API documentation client/docs/swagger-ui/swagger.yaml

[codecov[bot]]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 69.74%. Comparing base (47a5e9f) to head (feb9962).

Additional details and impacted files

[](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1277?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia) ```diff @@ Coverage Diff @@ ## main #1277 +/- ## ======================================= Coverage 69.74% 69.74% ======================================= Files 643 643 Lines 67261 67266 +5 ======================================= + Hits 46909 46914 +5 Misses 18179 18179 Partials 2173 2173 ``` | [Files](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1277?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia) | Coverage Δ | | |---|---|---| | [x/foundation/keeper/internal/genesis.go](https://app.codecov.io/gh/Finschia/finschia-sdk/pull/1277?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Finschia#diff-eC9mb3VuZGF0aW9uL2tlZXBlci9pbnRlcm5hbC9nZW5lc2lzLmdv) | `97.40% <100.00%> (+0.18%)` | :arrow_up: |

[ulbqb]

NOTE: This is likely to change account number state in auth module.

You mean, this may have a negative impact on mainnet?

[jaeseung-bae]

NOTE: This is likely to change account number state in auth module.

You mean, this may have a negative impact on mainnet?

A newly joined node that tries to sync with our operating node can never sync, because they may have different account number state. ~So i labeled state breaking change, we need to apply this to the next major version.~ -> (note, InitGenesis runs only once)

GetModuleAccount will create a new account if not exists.