dymensionxyz / dymension

Dymension Hub
https://dymension.xyz
Other
364 stars 341 forks source link

fix(eibc,delaydack): add validation to eibc and delayedack genesis state #967

Closed zale144 closed 3 months ago

zale144 commented 3 months ago

Description


Closes #942

All items are required. Please add a note to the item if the item is not applicable and please add links to any relevant follow-up issues.

PR review checkboxes:

I have...

SDK Checklist

Full security checklist here

----;

For Reviewer:

---;

After reviewer approval:

codecov[bot] commented 3 months ago

Codecov Report

Attention: Patch coverage is 58.62069% with 24 lines in your changes missing coverage. Please review.

Project coverage is 28.72%. Comparing base (c3d9307) to head (30815ca). Report is 10 commits behind head on main.

:exclamation: Current head 30815ca differs from pull request most recent head f18b751

Please upload reports for the commit f18b751 to get more accurate results.

Files Patch % Lines
x/common/types/rollapp_packet.go 28.57% 10 Missing and 5 partials :warning:
x/eibc/types/params.go 52.63% 6 Missing and 3 partials :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #967 +/- ## ========================================== - Coverage 29.15% 28.72% -0.44% ========================================== Files 243 256 +13 Lines 34148 33695 -453 ========================================== - Hits 9957 9679 -278 + Misses 22710 22664 -46 + Partials 1481 1352 -129 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.