Liquidity: Manejar problemas de envío de monedas a la cuenta de reserva.

Resumen de error

Cuando alguien envía monedas a la cuenta de reserva de un grupo, puede causar algunos problemas que incluyen:

• Un grupo con saldo de monto cero en una cara de la moneda de reserva, lo que lleva a un cálculo incorrecto del precio del grupo
  
  
  
  • Retirando todas las monedas del grupo y luego enviando solo una moneda de reserva directamente a la cuenta de reserva
  
  
• Un pánico en la lógica de depósito, al calcular la relación de la moneda de reserva (dividir por error cero con un grupo descrito anteriormente)
• ...

Versión

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

Pasos para reproducir

Ejecutar esta prueba provoca pánico:

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)
}

Para uso administrativo

• [x] No es un problema duplicado
• [x] Se han aplicado las etiquetas adecuadas
• [] Colaboradores apropiados etiquetados
• [] Colaborador asignado / autoasignado