fix seq typo

Finschia · Apr 29, 2024 · d1618ce · d1618ce
1 parent c464ac9
commit d1618ce
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 7 deletions.
diff --git a/x/fbridge/keeper/transfer.go b/x/fbridge/keeper/transfer.go
@@ -25,10 +25,10 @@ func (k Keeper) handleBridgeTransfer(ctx sdk.Context, sender sdk.AccAddress, amo
 		panic(fmt.Errorf("cannot burn coins after a successful send to a module account: %v", err))
 	}
 
-	nextSeq := k.GetNextSequence(ctx) + 1
-	k.setNextSequence(ctx, nextSeq)
+	seq := k.GetNextSequence(ctx)
+	k.setNextSequence(ctx, seq+1)
 
-	return nextSeq, nil
+	return seq, nil
 }
 
 func (k Keeper) GetNextSequence(ctx sdk.Context) uint64 {

diff --git a/x/fbridge/keeper/transfer_test.go b/x/fbridge/keeper/transfer_test.go
@@ -27,14 +27,16 @@ func TestHandleBridgeTransfer(t *testing.T) {
 	bankKeeper.EXPECT().BurnCoins(ctx, types.ModuleName, token).Return(nil)
 
 	k := NewKeeper(encCfg.Codec, key, authKeeper, bankKeeper, denom, "gov")
-	beforeSeq := uint64(2)
+	targetSeq := uint64(2)
 	bz := make([]byte, 8)
-	binary.BigEndian.PutUint64(bz, beforeSeq)
+	binary.BigEndian.PutUint64(bz, targetSeq)
 	ctx.KVStore(key).Set(types.KeyNextSeqSend, bz)
 
-	afterSeq, err := k.handleBridgeTransfer(ctx, sender, amt)
+	handledSeq, err := k.handleBridgeTransfer(ctx, sender, amt)
 	require.NoError(t, err)
-	require.Equal(t, beforeSeq+1, afterSeq)
+	require.Equal(t, targetSeq, handledSeq)
+	afterSeq := k.GetNextSequence(ctx)
+	require.Equal(t, targetSeq+1, afterSeq)
 }
 
 func TestIsValidEthereumAddress(t *testing.T) {