test: implement codec registration tests for ibc core and lightclients

[damiannolan]

Description

ref: #4747

---

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

• Targeted PR against correct branch (see CONTRIBUTING.md).
• Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
• Code follows the module structure standards and Go style guide.
• Wrote unit and integration tests.
• Updated relevant documentation (docs/) or specification (x/<module>/spec/).
• Added relevant godoc comments.
• Provide a commit message to be used for the changelog entry in the PR description for review.
• Re-reviewed Files changed in the Github PR explorer.
• Review Codecov Report in the comment section below once CI passes.

[codecov-commenter]

Codecov Report

Merging #4757 (295c2ca) into main (e443a88) will increase coverage by 0.25%.
Report is 3 commits behind head on main.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4757      +/-   ##
==========================================
+ Coverage   79.49%   79.75%   +0.25%     
==========================================
  Files         189      189              
  Lines       13285    13285              
==========================================
+ Hits        10561    10595      +34     
+ Misses       2296     2262      -34     
  Partials      428      428              

see 1 file with indirect coverage changes

[colin-axner]

Nice! I wonder if there are some duplicate tests that can be removed now. Curious if this covers testing for UnpackInterfaces func needing to be required

[chatton]

Really nice 🥳

[damiannolan]

Nice! I wonder if there are some duplicate tests that can be removed now. Curious if this covers testing for UnpackInterfaces func needing to be required

Indirectly I think it kind of covers it. But its not really explicit. I think its probably worth keeping the existing marshal/unmarshal tests for Any unpacking for now until it can be further worked out.