kintsugi-tech / slash-refund

0 stars 0 forks source link

Genesis valdiation #19

Open 0xstepit opened 1 year ago

0xstepit commented 1 year ago

Should we add a genesis validation? I mean, shouldn't be possible to have a deposit pool without deposits. We have to enforce it in a pre-initialization method?

niilptr commented 1 year ago

Imo yes. Other than for basic validations (duplicated elements, non-positive quantities), we should also check for consistency of deposits and deposit pools, and refunds and refund pools. We should update GenesisState.Validate(), adding these validation checks.