fix: fix the rewards collects for bid

app/upgrades.go

@@ -65,7 +65,7 @@
 
 			// update leverage and metoken params to include burn auction fee share.
 			lparams := app.LeverageKeeper.GetParams(ctx)
-			lparams.RewardsAuctionFee = sdk.MustNewDecFromStr("0.01")
+			lparams.RewardsAuctionFee = sdk.MustNewDecFromStr("0.05") // NOTE: this is for testing!
 			if err := app.LeverageKeeper.SetParams(ctx, lparams); err != nil {
 				return nil, err
 			}

proto/umee/auction/v1/auction.proto

@@ -17,7 +17,7 @@ message RewardsParams {
 
 // Bid records a user bid
 message Bid {
-  bytes  bidder = 1 [(cosmos_proto.scalar) = "cosmos.AddressString"];
+  string bidder = 1 [(cosmos_proto.scalar) = "cosmos.AddressString"];
   string amount = 2 [(cosmos_proto.scalar) = "cosmos.Int", (gogoproto.customtype) = "cosmossdk.io/math.Int", (gogoproto.nullable) = false];
 }
 

proto/umee/auction/v1/query.proto

@@ -3,6 +3,7 @@ package umee.auction.v1;
 
 import "google/protobuf/timestamp.proto";
 import "google/api/annotations.proto";
+import "cosmos_proto/cosmos.proto";
 import "gogoproto/gogo.proto";
 import "umee/auction/v1/auction.proto";
 import "cosmos/base/v1beta1/coin.proto";
@@ -42,7 +43,7 @@ message QueryRewardsAuction {
 message QueryRewardsAuctionResponse {
   uint32 id = 1;
   // highest bidder
-  string                   bidder           = 2;
+  string bidder = 2 [(cosmos_proto.scalar) = "cosmos.AddressString"];
   cosmos.base.v1beta1.Coin bid              = 3 [(gogoproto.nullable) = false];
   repeated cosmos.base.v1beta1.Coin rewards = 4 [(gogoproto.nullable) = false];
   google.protobuf.Timestamp         ends_at = 5 [(gogoproto.nullable) = false, (gogoproto.stdtime) = true];

x/auction/genesis.go

@@ -10,7 +10,7 @@
 // DefaultGenesis creates a default genesis state
 func DefaultGenesis() *GenesisState {
 	return &GenesisState{
-		RewardsParams:   RewardsParams{BidDuration: 14 * 24 * 3600}, // 14 days
+		RewardsParams:   RewardsParams{BidDuration: 4 * 3600}, // 4 hours. NOTE: this is for testing!
 		RewardAuctionId: 0,
 		RewardsAuctions: []RewardsKV{},
 		RewardsBids:     []BidKV{},

x/auction/keeper/genesis_test.go

@@ -44,8 +44,8 @@ func TestGenesis(t *testing.T) {
 		{4, auction.Rewards{newCoin(412313), randTime()}},
 	}
 	gs.RewardsBids = []auction.BidKV{
-		{1, auction.Bid{accs.Alice, sdkmath.NewInt(153252)}},
-		{2, auction.Bid{accs.Alice, sdkmath.NewInt(8521)}},
+		{1, auction.Bid{accs.Alice.String(), sdkmath.NewInt(153252)}},
+		{2, auction.Bid{accs.Alice.String(), sdkmath.NewInt(8521)}},
 	}
 	check(gs)
 }

x/auction/keeper/grpc_query.go

@@ -37,7 +37,7 @@
 	bid, id := q.Keeper(&ctx).getRewardsBid(msg.Id)
 	r := &auction.QueryRewardsAuctionResponse{Id: id}
 	if bid != nil {
-		r.Bidder = sdk.AccAddress(bid.Bidder).String()
+		r.Bidder = bid.Bidder
 		r.Bid = coin.UmeeInt(bid.Amount)
 	}
 	rewards, _ := q.Keeper(&ctx).getRewardsAuction(msg.Id)

x/auction/keeper/rewards.go

@@ -1,15 +1,13 @@
 package keeper
 
 import (
-	"bytes"
 	"errors"
 	"fmt"
 	"strconv"
 	"time"
 
 	sdk "github.com/cosmos/cosmos-sdk/types"
 
-	"github.com/umee-network/umee/v6/util"
 	"github.com/umee-network/umee/v6/util/coin"
 	"github.com/umee-network/umee/v6/util/sdkutil"
 	"github.com/umee-network/umee/v6/util/store"
@@ -30,7 +28,11 @@
 	newCoins := k.bank.GetAllBalances(*k.ctx, k.accs.RewardsCollect)
 	bid, _ := k.getRewardsBid(id)
 	if bid != nil && len(bid.Bidder) != 0 {
-		err := k.sendCoins(k.accs.RewardsCollect, bid.Bidder, a.Rewards)
+		bidderAccAddr, err := sdk.AccAddressFromBech32(bid.Bidder)
+		if err != nil {
+			return err
+		}
+		err = k.sendCoins(k.accs.RewardsCollect, bidderAccAddr, a.Rewards)
 		if err != nil {
 			return fmt.Errorf("can't send coins to finalize the auction [%w]", err)
 		}
@@ -79,25 +81,33 @@
 		return err
 	}
 
+	toAuction := msg.Amount
+	if lastBid != nil {
+		if msg.Sender == lastBid.Bidder {
+			// bidder updates his last bid: send only diff
+			toAuction = msg.Amount.SubAmount(lastBid.Amount)
+		} else {
+			returned := coin.UmeeInt(lastBid.Amount)
+			bidderAccAddr, err := sdk.AccAddressFromBech32(lastBid.Bidder)
+			if err != nil {
+				return err
+			}
+			if err = k.sendFromModule(bidderAccAddr, returned); err != nil {
+				return err
+			}
+		}
+	}
+
 	sender, err := sdk.AccAddressFromBech32(msg.Sender)
-	util.Panic(err)
+	if err != nil {
+		return err
+	}
 
-	if !bytes.Equal(sender, lastBid.Bidder) {
-		returned := coin.UmeeInt(lastBid.Amount)
-		if err = k.sendFromModule(lastBid.Bidder, returned); err != nil {
-			return err
-		}
-		if err = k.sendToModule(sender, msg.Amount); err != nil {
-			return err
-		}
-	} else {
-		diff := msg.Amount.SubAmount(lastBid.Amount)
-		if err = k.sendToModule(sender, diff); err != nil {
-			return err
-		}
+	if err = k.sendToModule(sender, toAuction); err != nil {
+		return err
 	}
 
-	bid := auction.Bid{Bidder: sender, Amount: msg.Amount.Amount}
+	bid := auction.Bid{Bidder: msg.Sender, Amount: msg.Amount.Amount}
 	return store.SetValue(k.store, key, &bid, keyMsg)
 }
 
@@ -111,8 +121,7 @@
 }
 
 func (k Keeper) getAllRewardsBids() ([]auction.BidKV, error) {
-	elems, err := store.LoadAllKV[*store.Uint32, store.Uint32, *auction.Bid](
-		k.store, keyPrefixRewardsBid)
+	elems, err := store.LoadAllKV[*store.Uint32, store.Uint32, *auction.Bid](k.store, keyPrefixRewardsBid)
 	if err != nil {
 		return nil, err
 	}
@@ -122,7 +131,6 @@
 		bids[i].Bid = elems[i].Val
 	}
 	return bids, nil
-
 }
 
 func (k Keeper) storeAllRewardsBids(elems []auction.BidKV) error {

x/leverage/keeper/oracle.go

@@ -297,7 +297,7 @@
 	}
 	if !toAuctionCheck.IsZero() {
 		auction.EmitFundRewardsAuction(&ctx, toOracleCheck)
-		return send(ctx, types.ModuleName, auction.ModuleName, toAuctionCheck)
+		return k.bankKeeper.SendCoinsFromModuleToAccount(ctx, types.ModuleName, k.rewardsAuction, toAuctionCheck)
 	}
 
 	return nil