From bccabb808065e76927264c34930861db9dcfb0d1 Mon Sep 17 00:00:00 2001
From: Jonathan Chappelow <chappjc@protonmail.com>
Date: Tue, 7 Feb 2023 09:23:37 -0600
Subject: [PATCH] client/asset/eth: add (*assetWallet).amtString method

---
 client/asset/eth/eth.go | 24 ++++++++++++++++--------
 1 file changed, 16 insertions(+), 8 deletions(-)

diff --git a/client/asset/eth/eth.go b/client/asset/eth/eth.go
index 93481f39a1..96d2e815e6 100644
--- a/client/asset/eth/eth.go
+++ b/client/asset/eth/eth.go
@@ -431,10 +431,10 @@ type baseWallet struct {
 // assetWallet satisfies the dex.Wallet interface.
 type assetWallet struct {
 	*baseWallet
-	assetID    uint32
-	tipChange  func(error)
-	log        dex.Logger
-	atomicUnit string
+	assetID   uint32
+	tipChange func(error)
+	log       dex.Logger
+	ui        dex.UnitInfo
 
 	lockedFunds struct {
 		mtx                sync.RWMutex
@@ -681,7 +681,7 @@ func NewWallet(assetCFG *asset.WalletConfig, logger dex.Logger, net dex.Network)
 		contractors:        make(map[uint32]contractor),
 		evmify:             dexeth.GweiToWei,
 		atomize:            dexeth.WeiToGwei,
-		atomicUnit:         dexeth.UnitInfo.AtomicUnit,
+		ui:                 dexeth.UnitInfo,
 		maxSwapsInTx:       gasCeil / maxProportionOfBlockGasLimitToUse / maxSwapGas,
 		maxRedeemsInTx:     gasCeil / maxProportionOfBlockGasLimitToUse / maxRedeemGas,
 	}
@@ -963,7 +963,7 @@ func (w *ETHWallet) OpenTokenWallet(tokenCfg *asset.TokenConfig) (asset.Wallet,
 		contractors:        make(map[uint32]contractor),
 		evmify:             token.AtomicToEVM,
 		atomize:            token.EVMToAtomic,
-		atomicUnit:         token.UnitInfo.AtomicUnit,
+		ui:                 token.UnitInfo,
 		maxSwapsInTx:       gasCeil / maxProportionOfBlockGasLimitToUse / maxSwapGas,
 		maxRedeemsInTx:     gasCeil / maxProportionOfBlockGasLimitToUse / maxRedeemGas,
 	}
@@ -994,6 +994,10 @@ func (eth *baseWallet) OwnsDepositAddress(address string) (bool, error) {
 	return addr == eth.addr, nil
 }
 
+func (w *assetWallet) amtString(amt uint64) string {
+	return fmt.Sprintf("%s %s", w.ui.ConventionalString(amt), w.ui.Conventional.Unit)
+}
+
 // fundReserveType represents the various uses for which funds need to be locked:
 // initiations, redemptions, and refunds.
 type fundReserveType uint32
@@ -1042,7 +1046,7 @@ func (w *assetWallet) lockFunds(amt uint64, t fundReserveType) error {
 	if balance.Available < amt {
 
 		return fmt.Errorf("attempting to lock more %s for %s than is currently available. %d > %d %s",
-			dex.BipIDSymbol(w.assetID), t, amt, balance.Available, w.atomicUnit)
+			dex.BipIDSymbol(w.assetID), t, amt, balance.Available, w.ui.AtomicUnit)
 	}
 
 	w.lockedFunds.mtx.Lock()
@@ -1340,6 +1344,8 @@ func (w *ETHWallet) FundOrder(ord *asset.Order) (asset.Coins, []dex.Bytes, error
 	// some work for the caller as well. We can't just always do it that way and
 	// remove RedeemN, since we can't guarantee that the redemption asset is in
 	// our fee-family. though it could still be an AccountRedeemer.
+	w.log.Debugf("Locking %s to swap %s in up to %d swaps at a fee rate of %d gwei/gas using up to %d gas per swap",
+		w.amtString(ethToLock), w.amtString(ord.Value), ord.MaxSwapCount, ord.MaxFeeRate, g.Swap)
 
 	coin := w.createFundingCoin(ethToLock)
 
@@ -1381,6 +1387,8 @@ func (w *TokenWallet) FundOrder(ord *asset.Order) (asset.Coins, []dex.Bytes, err
 		}
 	}()
 
+	w.log.Debugf("Locking %s to swap %s in up to %d swaps at a fee rate of %d gwei/gas using up to %d gas per swap",
+		w.parent.amtString(ethToLock), w.amtString(ord.Value), ord.MaxSwapCount, ord.MaxFeeRate, g.Swap)
 	if err := w.parent.lockFunds(ethToLock, initiationReserve); err != nil {
 		return nil, nil, err
 	}
@@ -2635,7 +2643,7 @@ func (w *TokenWallet) canSend(value uint64, isPreEstimate bool) (uint64, *big.In
 		}
 		avail := bal.Available
 		if avail < value {
-			return 0, nil, fmt.Errorf("not enough tokens: have %[1]d %[3]s need %[2]d %[3]s", avail, value, w.atomicUnit)
+			return 0, nil, fmt.Errorf("not enough tokens: have %[1]d %[3]s need %[2]d %[3]s", avail, value, w.ui.AtomicUnit)
 		}
 
 		ethBal, err := w.parent.Balance()