Liquidity: Gérer les problèmes d'envoi de pièces sur le compte de réserve

Résumé du bogue

Lorsque quelqu'un envoie des pièces sur le compte de réserve d'un pool, cela peut causer des problèmes, notamment :

• Un pool avec un solde de montant nul sur un côté de la pièce de réserve, ce qui entraîne un calcul incorrect du prix du pool
  
  
  
  • En retirant toutes les pièces du pool et en envoyant ensuite une seule pièce de réserve directement sur le compte de réserve
  
  
• Une panique dans la logique de dépôt, lors du calcul du ratio de pièces de réserve (diviser par zéro erreur avec un pool décrit ci-dessus)
• ...

Version

https://github.com/tendermint/liquidity/releases/tag/v1.2.5

Étapes pour reproduire

L'exécution de ce test provoque une panique :

func TestSwapHalfZeroReserve(t *testing.T) {
    simapp, ctx := createTestInput()
    params := simapp.LiquidityKeeper.GetParams(ctx)
    pool, addr, err := createPool(simapp, ctx, sdk.NewInt(1000000), sdk.NewInt(1000000), DenomX, DenomY)
    require.NoError(t, err)
    pc := simapp.BankKeeper.GetBalance(ctx, addr, pool.PoolCoinDenom)
    _, err = simapp.LiquidityKeeper.WithdrawLiquidityPoolToBatch(ctx, types.NewMsgWithdrawWithinBatch(addr, pool.Id, pc))
    require.NoError(t, err)
    liquidity.BeginBlocker(ctx, simapp.LiquidityKeeper)
    liquidity.EndBlocker(ctx, simapp.LiquidityKeeper)
    require.True(t, simapp.BankKeeper.GetBalance(ctx, pool.GetReserveAccount(), DenomX).IsZero())
    require.True(t, simapp.BankKeeper.GetBalance(ctx, pool.GetReserveAccount(), DenomY).IsZero())
    err = simapp.BankKeeper.SendCoins(ctx, addr, pool.GetReserveAccount(), sdk.NewCoins(sdk.NewInt64Coin(DenomX, 10000)))
    require.NoError(t, err)
    _, err = simapp.LiquidityKeeper.SwapLiquidityPoolToBatch(ctx,
        types.NewMsgSwapWithinBatch(
            addr, pool.Id, types.DefaultSwapTypeId, sdk.NewInt64Coin(DenomX, 1000), DenomY, sdk.MustNewDecFromStr("1.0"), params.SwapFeeRate), 0)
    require.NoError(t, err)
    liquidity.BeginBlocker(ctx, simapp.LiquidityKeeper)
    liquidity.EndBlocker(ctx, simapp.LiquidityKeeper)
}

À l'usage de l'administrateur

• [x] Pas de problème en double
• [x] Libellés appropriés appliqués
• [ ] Contributeurs appropriés tagués
• [ ] Contributeur attribué/auto-attribué