test: more tests and cleanup

[fmorency]

This PR adds

• Additional keeper unit tests
• Check error content in tests
• Code cleanup (value vs pointer receiver mix, comments, lint)
• bump go to 1.22.4 (security fixes)
• bump alpine to 3.20 otherwise heighliner fails

[fmorency]

I'll rerun the failed job tomorrow as we reached Docker request limits.

toomanyrequests: You have reached your pull rate limit.

[codecov]

Codecov Report

Attention: Patch coverage is 56.52174% with 10 lines in your changes missing coverage. Please review.

Project coverage is 74.32%. Comparing base (5785a35) to head (32d5026).

Files	Patch %	Lines
x/manifest/types/msgs.go	0.00%	6 Missing ⚠️
x/manifest/module.go	63.63%	3 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #69      +/-   ##
==========================================
+ Coverage   73.41%   74.32%   +0.91%     
==========================================
  Files          31       31              
  Lines        1685     1675      -10     
==========================================
+ Hits         1237     1245       +8     
+ Misses        395      377      -18     
  Partials       53       53              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[fmorency]

Disregard the codecov/patch complain.

[Reecepbcups]

I remember removing this in the past for a network & it causing issues. They did come up from SDK v47 -> v50 though, so it's possible it was specific to that vs being on SDK v50 from the start like manifest. Just want to call it out now

[fmorency]

Do you remember the issue?