use golangci-lint on tests

[faddat]

The way I found #5060 was to use golangc-ilint on tests.

The following PR's are relevant to this:

• ineffectual assignments #5117
• style: address forced type assertions in tests #5116
• eliminate unnecessary type conversions from tests #5115
• use os.ReadFile instead of ioutil.ReadFile #5113
• style: remove unused code in tests #5108
• style: eliminate double imports in tests #5107
• style: lint stableswap the same as the rest of osmosis #5105
• chore: check all errors in tests #5104
• chore: run make format #5103
• chore: lint tests and sync for app, osmomath, tests, and wasmbinding #5206

I'm trying to keep the PR's as small and easy to review as possible, please lmk how I'm doing with that!

The final boss of this issue is this PR:

• lint all tests #5127

In order to close this issue, we'll need to open an issue to update the querygen code...

goal

have greater consistency throughout the codebase and make dev tooling more immediately useful (eg: highlighting)

[faddat]

Final boss accompished.