search

babylonlabs-io / babylon

Other
24 stars 17 forks source link

backport: Fix invalid minUnbondingTime for verifying inclusion proof (#289) #290

Closed gitferry closed 4 days ago

gitferry commented 4 days ago

The bug is caused due to setting params.MinStakingTime to the btcstaking module's keeper method VerifyInclusionProofAndGetHeight other than params.MinUnbondingTime. In the fix, we remove minUnbondingTime from the parameter list of VerifyInclusionProofAndGetHeight as it should retrieve the parameter within the method.

This PR also added comprehensive tests for VerifyInclusionProofAndGetHeight