From 0645ed47123f742faa2e110bc6daf1f2320dedd8 Mon Sep 17 00:00:00 2001
From: Khushboo Mehta <mehtakhushboo@hotmail.com>
Date: Fri, 14 Jun 2024 13:34:20 +0200
Subject: [PATCH] fix(@desktop/wallet): solving the issues with SwapInputPanel
 list in the issue

fixes  #15162
---
 storybook/pages/ErrorTagPage.qml              |   5 +
 storybook/pages/SwapInputPanelPage.qml        |  22 +-
 storybook/qmlTests/tests/tst_ErrorTag.qml     |   1 +
 .../qmlTests/tests/tst_SwapInputPanel.qml     | 191 ++++++++++++++++--
 storybook/qmlTests/tests/tst_SwapModal.qml    | 118 +++++------
 .../Wallet/panels/SwapInputPanel.qml          |  21 +-
 .../Wallet/popups/swap/SwapModal.qml          |  21 +-
 ui/imports/shared/controls/ErrorTag.qml       |   3 +-
 .../shared/popups/send/views/AmountToSend.qml |  29 +--
 9 files changed, 288 insertions(+), 123 deletions(-)

diff --git a/storybook/pages/ErrorTagPage.qml b/storybook/pages/ErrorTagPage.qml
index dabc31820..259012c89 100644
--- a/storybook/pages/ErrorTagPage.qml
+++ b/storybook/pages/ErrorTagPage.qml
@@ -27,6 +27,7 @@ SplitView {
             anchors.centerIn: parent
             text: ctrlText.text
             buttonText: ctrlButtonText.text
+            buttonVisible: buttonVisible.checked
             asset.name: ctrlAssetName.text
             loading: ctrlLoading.checked
             onButtonClicked: logs.logEvent("ErrorTag::onButtonClicked", [], arguments)
@@ -85,6 +86,10 @@ SplitView {
                 id: ctrlLoading
                 text: "Loading"
             }
+            Switch {
+                id: buttonVisible
+                text: "Button Visible"
+            }
             Item { Layout.fillHeight: true }
         }
     }
diff --git a/storybook/pages/SwapInputPanelPage.qml b/storybook/pages/SwapInputPanelPage.qml
index 51320ffc4..a18d61f0e 100644
--- a/storybook/pages/SwapInputPanelPage.qml
+++ b/storybook/pages/SwapInputPanelPage.qml
@@ -86,7 +86,8 @@ SplitView {
                 swapSide: SwapInputPanel.SwapSide.Pay
                 fiatInputInteractive: ctrlFiatInputInteractive.checked
                 swapExchangeButtonWidth: swapButton.width
-                loading: ctrlLoading.checked
+                mainInputLoading: ctrlMainInputLoading.checked
+                bottomTextLoading: ctrlBottomTextLoading.checked
             }
 
             SwapInputPanel {
@@ -107,7 +108,8 @@ SplitView {
                 swapSide: SwapInputPanel.SwapSide.Receive
                 fiatInputInteractive: ctrlFiatInputInteractive.checked
                 swapExchangeButtonWidth: swapButton.width
-                loading: ctrlLoading.checked
+                mainInputLoading: ctrlMainInputLoading.checked
+                bottomTextLoading: ctrlBottomTextLoading.checked
             }
 
             SwapExchangeButton {
@@ -191,8 +193,12 @@ SplitView {
                 checked: false
             }
             Switch {
-                id: ctrlLoading
-                text: "Loading"
+                id: ctrlMainInputLoading
+                text: "mainInputLoading"
+            }
+            Switch {
+                id: ctrlBottomTextLoading
+                text: "bottomTextLoading"
             }
 
             Label {
@@ -200,16 +206,16 @@ SplitView {
                 font.weight: Font.Medium
                 text: "<b>Pay:</b><ul><li>Symbol: %1<li>Amount: %2<li>Valid: %3"
                   .arg(payPanel.selectedHoldingId || "N/A")
-                  .arg(payPanel.cryptoValue.toString())
-                  .arg(payPanel.cryptoValueValid ? "true" : "false")
+                  .arg(payPanel.value.toString())
+                  .arg(payPanel.valueValid ? "true" : "false")
             }
             Label {
                 Layout.fillWidth: true
                 font.weight: Font.Medium
                 text: "<b>Receive:</b><ul><li>Symbol: %1<li>Amount: %2<li>Valid: %3"
                   .arg(receivePanel.selectedHoldingId || "N/A")
-                  .arg(receivePanel.cryptoValue.toString())
-                  .arg(receivePanel.cryptoValueValid ? "true" : "false")
+                  .arg(receivePanel.value.toString())
+                  .arg(receivePanel.valueValid ? "true" : "false")
             }
 
             Item { Layout.fillHeight: true }
diff --git a/storybook/qmlTests/tests/tst_ErrorTag.qml b/storybook/qmlTests/tests/tst_ErrorTag.qml
index d84ec31f9..81f7fe7b0 100644
--- a/storybook/qmlTests/tests/tst_ErrorTag.qml
+++ b/storybook/qmlTests/tests/tst_ErrorTag.qml
@@ -48,6 +48,7 @@ Item {
         function test_buttonClick() {
             verify(!!controlUnderTest)
             controlUnderTest.buttonText = "Buy crypto"
+            controlUnderTest.buttonVisible = true
             const button = findChild(controlUnderTest, "rightComponentButton")
             verify(!!button)
             tryCompare(button, "visible", true)
diff --git a/storybook/qmlTests/tests/tst_SwapInputPanel.qml b/storybook/qmlTests/tests/tst_SwapInputPanel.qml
index 4bfbf630a..dbc576b07 100644
--- a/storybook/qmlTests/tests/tst_SwapInputPanel.qml
+++ b/storybook/qmlTests/tests/tst_SwapInputPanel.qml
@@ -2,6 +2,8 @@ import QtQuick 2.15
 import QtTest 1.15
 
 import StatusQ 0.1
+import StatusQ.Core 0.1
+import StatusQ.Controls 0.1
 import StatusQ.Core.Utils 0.1
 
 import AppLayouts.Wallet.stores 1.0
@@ -71,8 +73,8 @@ Item {
             tryCompare(controlUnderTest, "swapSide", SwapInputPanel.SwapSide.Pay)
             tryCompare(controlUnderTest, "caption", qsTr("Pay"))
             tryCompare(controlUnderTest, "selectedHoldingId", "")
-            tryCompare(controlUnderTest, "cryptoValue", 0)
-            tryCompare(controlUnderTest, "cryptoValueRaw", "0")
+            tryCompare(controlUnderTest, "value", 0)
+            tryCompare(controlUnderTest, "rawValue", "0")
         }
 
         function test_basicSetupReceiveSide() {
@@ -85,8 +87,8 @@ Item {
             tryCompare(controlUnderTest, "swapSide", SwapInputPanel.SwapSide.Receive)
             tryCompare(controlUnderTest, "caption", qsTr("Receive"))
             tryCompare(controlUnderTest, "selectedHoldingId", "")
-            tryCompare(controlUnderTest, "cryptoValue", 0)
-            tryCompare(controlUnderTest, "cryptoValueRaw", "0")
+            tryCompare(controlUnderTest, "value", 0)
+            tryCompare(controlUnderTest, "rawValue", "0")
         }
 
         function test_basicSetupWithInitialProperties() {
@@ -101,8 +103,8 @@ Item {
 
             tryCompare(controlUnderTest, "swapSide", SwapInputPanel.SwapSide.Pay)
             tryCompare(controlUnderTest, "selectedHoldingId", "STT")
-            tryCompare(controlUnderTest, "cryptoValue", 10000000.0000001)
-            verify(controlUnderTest.cryptoValueValid)
+            tryCompare(controlUnderTest, "value", 10000000.0000001)
+            verify(controlUnderTest.valueValid)
         }
 
         function test_setTokenKeyAndAmounts_data() {
@@ -129,8 +131,8 @@ Item {
             tryCompare(controlUnderTest, "selectedHoldingId", tokenSymbol)
             if (!valid)
                 expectFail(data.tag, "Invalid data expected to fail: %1".arg(tokenAmount))
-            tryCompare(controlUnderTest, "cryptoValue", parseFloat(tokenAmount))
-            tryCompare(controlUnderTest, "cryptoValueValid", true)
+            tryCompare(controlUnderTest, "value", parseFloat(tokenAmount))
+            tryCompare(controlUnderTest, "valueValid", true)
 
             const holdingSelector = findChild(controlUnderTest, "holdingSelector")
             verify(!!holdingSelector)
@@ -167,8 +169,8 @@ Item {
             keyClick(Qt.Key_2)
 
             tryCompare(amountToSendInput.input, "text", "1000000,00000042")
-            tryCompare(controlUnderTest, "cryptoValue", 1000000.00000042)
-            verify(controlUnderTest.cryptoValueValid)
+            tryCompare(controlUnderTest, "value", 1000000.00000042)
+            verify(controlUnderTest.valueValid)
         }
 
         function test_selectSTTHoldingAndTypeAmount() {
@@ -201,8 +203,8 @@ Item {
             keyClick(Qt.Key_4)
             keyClick(Qt.Key_2)
 
-            tryCompare(controlUnderTest, "cryptoValue", 1.42)
-            verify(controlUnderTest.cryptoValueValid)
+            tryCompare(controlUnderTest, "value", 1.42)
+            verify(controlUnderTest.valueValid)
         }
 
         // verify that when "fiatInputInteractive" mode is on, the Max send button text shows fiat currency symbol (e.g. "1.2 USD")
@@ -217,6 +219,7 @@ Item {
             waitForRendering(maxTagButton)
             verify(maxTagButton.visible)
             verify(!maxTagButton.text.endsWith("ETH"))
+            compare(maxTagButton.type, StatusBaseButton.Type.Normal)
             mouseClick(maxTagButton)
 
             const amountToSendInput = findChild(controlUnderTest, "amountToSendInput")
@@ -242,6 +245,7 @@ Item {
             verify(!!maxTagButton)
             waitForRendering(maxTagButton)
             verify(maxTagButton.visible)
+            compare(maxTagButton.type, StatusBaseButton.Type.Normal)
             verify(!maxTagButton.text.endsWith("ETH"))
         }
 
@@ -255,6 +259,7 @@ Item {
             verify(!!maxTagButton)
             waitForRendering(maxTagButton)
             verify(maxTagButton.visible)
+            compare(maxTagButton.type, StatusBaseButton.Type.Normal)
             mouseClick(maxTagButton)
 
             const amountToSendInput = findChild(controlUnderTest, "amountToSendInput")
@@ -263,15 +268,16 @@ Item {
             const maxValue = amountToSendInput.maxInputBalance
 
             tryCompare(amountToSendInput.input, "text", maxValue.toLocaleString(Qt.locale(), 'f', -128))
-            tryCompare(controlUnderTest, "cryptoValue", maxValue)
-            verify(controlUnderTest.cryptoValueValid)
+            tryCompare(controlUnderTest, "value", maxValue)
+            verify(controlUnderTest.valueValid)
         }
 
         function test_loadingState() {
             controlUnderTest = createTemporaryObject(componentUnderTest, root)
             verify(!!controlUnderTest)
 
-            controlUnderTest.loading = true
+            controlUnderTest.mainInputLoading = true
+            controlUnderTest.bottomTextLoading = true
 
             const amountToSendInput = findChild(controlUnderTest, "amountToSendInput")
             verify(!!amountToSendInput)
@@ -292,5 +298,160 @@ Item {
             verify(!!bottomItemTextLoadingComponent)
             verify(bottomItemTextLoadingComponent.visible)
         }
+
+        function test_max_button_when_different_tokens_clicked() {
+            controlUnderTest = createTemporaryObject(componentUnderTest, root)
+            verify(!!controlUnderTest)
+
+            controlUnderTest.mainInputLoading = true
+            controlUnderTest.bottomTextLoading = true
+
+            const maxTagButton = findChild(controlUnderTest, "maxTagButton")
+            verify(!!maxTagButton)
+            verify(!maxTagButton.visible)
+
+            const holdingSelector = findChild(controlUnderTest, "holdingSelector")
+            verify(!!holdingSelector)
+
+            const assetSelectorList = findChild(holdingSelector, "assetSelectorList")
+            verify(!!assetSelectorList)
+
+            const assetSelectorButton = findChild(controlUnderTest, "assetSelectorButton")
+            verify(!!assetSelectorButton)
+
+            const amountToSendInput = findChild(controlUnderTest, "amountToSendInput")
+            verify(!!amountToSendInput)
+
+            const bottomItemText = findChild(amountToSendInput, "bottomItemText")
+            verify(!!bottomItemText)
+
+            for (let i= 0; i < d.adaptor.processedAssetsModel.count; i++) {
+                let modelItemToTest = ModelUtils.get(d.adaptor.processedAssetsModel, i)
+                mouseClick(assetSelectorButton)
+                waitForRendering(assetSelectorList)
+
+                let delToTest = assetSelectorList.itemAtIndex(i)
+                verify(!!delToTest)
+                mouseClick(delToTest, 40, 40) // center might be covered by tags
+
+                waitForRendering(maxTagButton)
+                verify(maxTagButton.visible)
+                verify(!maxTagButton.text.endsWith(modelItemToTest.symbol))
+                compare(maxTagButton.type, modelItemToTest.currentBalance === 0 ? StatusBaseButton.Type.Danger : StatusBaseButton.Type.Normal)
+
+                // check input value and state
+                mouseClick(maxTagButton)
+                waitForRendering(amountToSendInput)
+
+                compare(amountToSendInput.input.text, modelItemToTest.currentBalance === 0 ? "" : maxTagButton.maxSafeValueAsString)
+                compare(controlUnderTest.value, maxTagButton.maxSafeValue)
+                verify(modelItemToTest.currentBalance === 0 ? !controlUnderTest.valueValid : controlUnderTest.valueValid)
+                compare(bottomItemText.text, d.adaptor.formatCurrencyAmount(
+                            maxTagButton.maxSafeValue * amountToSendInput.selectedHolding.marketDetails.currencyPrice.amount,
+                            d.adaptor.currencyStore.currentCurrency))
+            }
+        }
+
+        function test_input_greater_than_max_balance() {
+            controlUnderTest = createTemporaryObject(componentUnderTest, root)
+            verify(!!controlUnderTest)
+
+            controlUnderTest.mainInputLoading = true
+            controlUnderTest.bottomTextLoading = true
+
+            const maxTagButton = findChild(controlUnderTest, "maxTagButton")
+            verify(!!maxTagButton)
+            verify(!maxTagButton.visible)
+
+            const holdingSelector = findChild(controlUnderTest, "holdingSelector")
+            verify(!!holdingSelector)
+
+            const assetSelectorList = findChild(holdingSelector, "assetSelectorList")
+            verify(!!assetSelectorList)
+
+            const assetSelectorButton = findChild(controlUnderTest, "assetSelectorButton")
+            verify(!!assetSelectorButton)
+
+            const amountToSendInput = findChild(controlUnderTest, "amountToSendInput")
+            verify(!!amountToSendInput)
+
+            const bottomItemText = findChild(amountToSendInput, "bottomItemText")
+            verify(!!bottomItemText)
+
+            // enter 5.42 as entered amount
+            keyClick(Qt.Key_5)
+            keyClick(Qt.Key_Period)
+            keyClick(Qt.Key_4)
+            keyClick(Qt.Key_2)
+
+            let numberTested = LocaleUtils.numberFromLocaleString("5.42", amountToSendInput.input.locale)
+
+            compare(amountToSendInput.input.text, "5.42")
+
+            for (let i= 0; i < d.adaptor.processedAssetsModel.count; i++) {
+                let modelItemToTest = ModelUtils.get(d.adaptor.processedAssetsModel, i)
+                mouseClick(assetSelectorButton)
+                waitForRendering(assetSelectorList)
+
+                let delToTest = assetSelectorList.itemAtIndex(i)
+                verify(!!delToTest)
+                mouseClick(delToTest, 40, 40) // center might be covered by tags
+
+                // check input value and state
+                waitForRendering(amountToSendInput)
+
+                compare(amountToSendInput.input.text, "5.42")
+                compare(bottomItemText.text, d.adaptor.formatCurrencyAmount(
+                            numberTested * amountToSendInput.selectedHolding.marketDetails.currencyPrice.amount,
+                            d.adaptor.currencyStore.currentCurrency))
+                compare(controlUnderTest.value, numberTested)
+                compare(controlUnderTest.rawValue, AmountsArithmetic.fromNumber(amountToSendInput.input.text, modelItemToTest.decimals).toString())
+                compare(controlUnderTest.valueValid, numberTested <= maxTagButton.maxSafeValue)
+                compare(controlUnderTest.selectedHoldingId, modelItemToTest.tokensKey)
+                compare(controlUnderTest.amountEnteredGreaterThanBalance, numberTested > maxTagButton.maxSafeValue)
+            }
+        }
+
+        function test_if_values_are_reset_after_setting_tokenAmount_as_empty() {
+            const tokenKeyToTest = "ETH"
+            let numberTestedString = "1.0001"
+            let modelItemToTest = ModelUtils.getByKey(d.adaptor.processedAssetsModel, "tokensKey", tokenKeyToTest)
+            controlUnderTest = createTemporaryObject(componentUnderTest, root, {
+                                                         swapSide: SwapInputPanel.SwapSide.Pay,
+                                                         tokenKey: tokenKeyToTest,
+                                                         tokenAmount: numberTestedString
+                                                     })
+            verify(!!controlUnderTest)
+            waitForRendering(controlUnderTest)
+
+
+            const amountToSendInput = findChild(controlUnderTest, "amountToSendInput")
+            verify(!!amountToSendInput)
+
+            const bottomItemText = findChild(amountToSendInput, "bottomItemText")
+            verify(!!bottomItemText)
+
+            let numberTested = LocaleUtils.numberFromLocaleString(numberTestedString, amountToSendInput.input.locale)
+
+            compare(amountToSendInput.input.text, numberTestedString)
+            compare(controlUnderTest.value, numberTested)
+            compare(controlUnderTest.rawValue, AmountsArithmetic.fromNumber(amountToSendInput.input.text, modelItemToTest.decimals).toString())
+            compare(controlUnderTest.valueValid, true)
+            compare(controlUnderTest.selectedHoldingId, tokenKeyToTest)
+            compare(controlUnderTest.amountEnteredGreaterThanBalance, false)
+
+            numberTestedString = ""
+            numberTested = 0
+            mouseClick(amountToSendInput)
+            controlUnderTest.tokenAmount = numberTestedString
+            waitForRendering(amountToSendInput)
+
+            compare(amountToSendInput.input.text, numberTestedString)
+            compare(controlUnderTest.value, numberTested)
+            compare(controlUnderTest.rawValue, AmountsArithmetic.fromNumber(numberTested, modelItemToTest.decimals).toString())
+            compare(controlUnderTest.valueValid, false)
+            compare(controlUnderTest.selectedHoldingId, tokenKeyToTest)
+            compare(controlUnderTest.amountEnteredGreaterThanBalance, false)
+        }
     }
 }
diff --git a/storybook/qmlTests/tests/tst_SwapModal.qml b/storybook/qmlTests/tests/tst_SwapModal.qml
index 224cf4863..784c05f2f 100644
--- a/storybook/qmlTests/tests/tst_SwapModal.qml
+++ b/storybook/qmlTests/tests/tst_SwapModal.qml
@@ -120,19 +120,18 @@ Item {
             compare(root.swapAdaptor.swapOutputData.hasError, false)
 
             // verfy input and output panels
-            verify(!payPanel.loading)
+            verify(!payPanel.mainInputLoading)
+            verify(!payPanel.bottomTextLoading)
             compare(payPanel.selectedHoldingId, root.swapFormData.fromTokensKey)
-            compare(payPanel.cryptoValue, Number(root.swapFormData.fromTokenAmount))
-            compare(payPanel.cryptoValueRaw, SQUtils.AmountsArithmetic.fromNumber(root.swapFormData.fromTokenAmount, root.swapAdaptor.fromToken.decimals).toString())
-            verify(payPanel.cryptoValueValid)
-            verify(receivePanel.loading)
+            compare(payPanel.value, Number(root.swapFormData.fromTokenAmount))
+            compare(payPanel.rawValue, SQUtils.AmountsArithmetic.fromNumber(root.swapFormData.fromTokenAmount, root.swapAdaptor.fromToken.decimals).toString())
+            verify(payPanel.valueValid)
+            verify(receivePanel.mainInputLoading)
+            verify(receivePanel.bottomTextLoading)
             verify(!receivePanel.interactive)
             compare(receivePanel.selectedHoldingId, root.swapFormData.toTokenKey)
-            /* TODO: there is bug which prevents us from testing this right now
-            The value is not updated after  setting tokenAmount to empty string in the receive input panel
-            https://github.com/status-im/status-desktop/issues/15162
-            compare(receivePanel.cryptoValue, 0)
-            compare(receivePanel.cryptoValueRaw, "0") */
+            compare(receivePanel.value, 0)
+            compare(receivePanel.rawValue, "0")
         }
         // end helper functions -------------------------------------------------------------
 
@@ -554,12 +553,14 @@ Item {
             compare(signButton.text, qsTr("Swap"))
 
             // verfy input and output panels
-            verify(!payPanel.loading)
-            verify(!receivePanel.loading)
+            verify(!payPanel.mainInputLoading)
+            verify(!payPanel.bottomTextLoading)
+            verify(!receivePanel.mainInputLoading)
+            verify(!receivePanel.bottomTextLoading)
             verify(!receivePanel.interactive)
             compare(receivePanel.selectedHoldingId, root.swapFormData.toTokenKey)
-            compare(receivePanel.cryptoValue, 0)
-            compare(receivePanel.cryptoValueRaw, "0")
+            compare(receivePanel.value, 0)
+            compare(receivePanel.rawValue, "0")
 
             // edit some params to retry swap
             root.swapFormData.fromTokenAmount = "0.00011"
@@ -596,12 +597,13 @@ Item {
 
             // verfy input and output panels
             waitForRendering(receivePanel)
-            verify(payPanel.cryptoValueValid)
-            verify(!receivePanel.loading)
+            verify(payPanel.valueValid)
+            verify(!receivePanel.mainInputLoading)
+            verify(!receivePanel.bottomTextLoading)
             verify(!receivePanel.interactive)
             compare(receivePanel.selectedHoldingId, root.swapFormData.toTokenKey)
-            compare(receivePanel.cryptoValue, root.swapStore.getWei2Eth(txRoutes.amountToReceive, root.swapAdaptor.toToken.decimals))
-            compare(receivePanel.cryptoValueRaw, SQUtils.AmountsArithmetic.fromNumber(root.swapAdaptor.swapOutputData.toTokenAmount, root.swapAdaptor.toToken.decimals).toString())
+            compare(receivePanel.value, root.swapStore.getWei2Eth(txRoutes.amountToReceive, root.swapAdaptor.toToken.decimals))
+            compare(receivePanel.rawValue, SQUtils.AmountsArithmetic.fromNumber(root.swapAdaptor.swapOutputData.toTokenAmount, root.swapAdaptor.toToken.decimals).toString())
 
             // edit some params to retry swap
             root.swapFormData.fromTokenAmount = "0.012"
@@ -638,12 +640,13 @@ Item {
 
             // verfy input and output panels
             waitForRendering(receivePanel)
-            verify(payPanel.cryptoValueValid)
-            verify(!receivePanel.loading)
+            verify(payPanel.valueValid)
+            verify(!receivePanel.mainInputLoading)
+            verify(!receivePanel.bottomTextLoading)
             verify(!receivePanel.interactive)
             compare(receivePanel.selectedHoldingId, root.swapFormData.toTokenKey)
-            compare(receivePanel.cryptoValue, root.swapStore.getWei2Eth(txRoutes.amountToReceive, root.swapAdaptor.toToken.decimals))
-            compare(receivePanel.cryptoValueRaw, SQUtils.AmountsArithmetic.fromNumber(root.swapAdaptor.swapOutputData.toTokenAmount, root.swapAdaptor.toToken.decimals).toString())
+            compare(receivePanel.value, root.swapStore.getWei2Eth(txRoutes.amountToReceive, root.swapAdaptor.toToken.decimals))
+            compare(receivePanel.rawValue, SQUtils.AmountsArithmetic.fromNumber(root.swapAdaptor.swapOutputData.toTokenAmount, root.swapAdaptor.toToken.decimals).toString())
         }
 
         function test_modal_pay_input_default() {
@@ -680,9 +683,9 @@ Item {
             verify(!holdingSelectorsTokenIcon.visible)
             verify(!maxTagButton.visible)
             compare(payPanel.selectedHoldingId, "")
-            compare(payPanel.cryptoValue, 0)
-            compare(payPanel.cryptoValueRaw, "0")
-            verify(!payPanel.cryptoValueValid)
+            compare(payPanel.value, 0)
+            compare(payPanel.rawValue, "0")
+            verify(!payPanel.valueValid)
 
             closeAndVerfyModal()
         }
@@ -731,9 +734,9 @@ Item {
             verify(maxTagButton.visible)
             compare(maxTagButton.text, qsTr("Max. %1").arg(root.swapAdaptor.currencyStore.formatCurrencyAmount(Math.trunc(WalletUtils.calculateMaxSafeSendAmount(expectedToken.currentBalance, expectedToken.symbol)*100)/100, expectedToken.symbol, {noSymbol: true})))
             compare(payPanel.selectedHoldingId, expectedToken.symbol)
-            compare(payPanel.cryptoValue, valueToExchange)
-            compare(payPanel.cryptoValueRaw, SQUtils.AmountsArithmetic.fromNumber(valueToExchangeString, expectedToken.decimals).toString())
-            verify(payPanel.cryptoValueValid)
+            compare(payPanel.value, valueToExchange)
+            compare(payPanel.rawValue, SQUtils.AmountsArithmetic.fromNumber(valueToExchangeString, expectedToken.decimals).toString())
+            verify(payPanel.valueValid)
 
             closeAndVerfyModal()
         }
@@ -778,9 +781,9 @@ Item {
                 verify(!holdingSelectorsTokenIcon.visible)
                 verify(!maxTagButton.visible)
                 compare(payPanel.selectedHoldingId, invalidValue)
-                compare(payPanel.cryptoValue, 0)
-                compare(payPanel.cryptoValueRaw, SQUtils.AmountsArithmetic.fromNumber("0", 0).toString())
-                verify(!payPanel.cryptoValueValid)
+                compare(payPanel.value, 0)
+                compare(payPanel.rawValue, SQUtils.AmountsArithmetic.fromNumber("0", 0).toString())
+                verify(!payPanel.valueValid)
 
                 closeAndVerfyModal()
             }
@@ -822,7 +825,7 @@ Item {
             compare(amountToSendInput.input.text, valueToExchangeString)
             compare(amountToSendInput.input.placeholderText, LocaleUtils.numberToLocaleString(0))
             verify(amountToSendInput.input.input.edit.cursorVisible)
-            compare(bottomItemText.text, root.swapAdaptor.currencyStore.formatCurrencyAmount(0 * expectedToken.marketDetails.currencyPrice.amount, root.swapAdaptor.currencyStore.currentCurrency))
+            compare(bottomItemText.text, root.swapAdaptor.currencyStore.formatCurrencyAmount(valueToExchange * expectedToken.marketDetails.currencyPrice.amount, root.swapAdaptor.currencyStore.currentCurrency))
             compare(holdingSelector.selectedItem, expectedToken)
             compare(holdingSelectorsContentItemText.text, expectedToken.symbol)
             compare(holdingSelectorsTokenIcon.image.source, Constants.tokenIcon(expectedToken.symbol))
@@ -830,9 +833,9 @@ Item {
             verify(maxTagButton.visible)
             compare(maxTagButton.text, qsTr("Max. %1").arg(root.swapAdaptor.currencyStore.formatCurrencyAmount(Math.trunc(WalletUtils.calculateMaxSafeSendAmount(expectedToken.currentBalance, expectedToken.symbol)*100)/100, expectedToken.symbol, {noSymbol: true})))
             compare(payPanel.selectedHoldingId, expectedToken.symbol)
-            compare(payPanel.cryptoValue, 0)
-            compare(payPanel.cryptoValueRaw, SQUtils.AmountsArithmetic.fromNumber("0", expectedToken.decimals).toString())
-            verify(!payPanel.cryptoValueValid)
+            compare(payPanel.value, valueToExchange)
+            compare(payPanel.rawValue, SQUtils.AmountsArithmetic.fromNumber(valueToExchangeString, expectedToken.decimals).toString())
+            verify(!payPanel.valueValid)
 
             closeAndVerfyModal()
         }
@@ -864,12 +867,9 @@ Item {
                 let maxPossibleValue = Math.trunc(WalletUtils.calculateMaxSafeSendAmount(expectedToken.currentBalance, expectedToken.symbol)*100)/100
                 compare(maxTagButton.text, qsTr("Max. %1").arg(root.swapAdaptor.currencyStore.formatCurrencyAmount(maxPossibleValue, expectedToken.symbol, {noSymbol: true})))
                 compare(payPanel.selectedHoldingId, expectedToken.symbol)
-                compare(payPanel.cryptoValueValid, valueToExchange <= maxPossibleValue)
-                /* TODO: there is bug which prevents us from testing this right now
-                When value entered is greater than balance then fiat and crytpo values are not calculated
-                https://github.com/status-im/status-desktop/issues/15162
-                compare(payPanel.cryptoValue, valueToExchange)
-                compare(payPanel.cryptoValueRaw, SQUtils.AmountsArithmetic.fromNumber(valueToExchangeString, expectedToken.decimals).toString()) */
+                compare(payPanel.valueValid, valueToExchange <= maxPossibleValue)
+                compare(payPanel.value, valueToExchange)
+                compare(payPanel.rawValue, SQUtils.AmountsArithmetic.fromNumber(valueToExchangeString, expectedToken.decimals).toString())
             }
 
             closeAndVerfyModal()
@@ -904,9 +904,9 @@ Item {
             compare(holdingSelectorsContentItemText.text, qsTr("Select asset"))
             verify(!maxTagButton.visible)
             compare(receivePanel.selectedHoldingId, "")
-            compare(receivePanel.cryptoValue, 0)
-            compare(receivePanel.cryptoValueRaw, "0")
-            verify(!receivePanel.cryptoValueValid)
+            compare(receivePanel.value, 0)
+            compare(receivePanel.rawValue, "0")
+            verify(!receivePanel.valueValid)
 
             closeAndVerfyModal()
         }
@@ -955,9 +955,9 @@ Item {
             verify(holdingSelectorsTokenIcon.visible)
             verify(!maxTagButton.visible)
             compare(receivePanel.selectedHoldingId, expectedToken.symbol)
-            compare(receivePanel.cryptoValue, valueToReceive)
-            compare(receivePanel.cryptoValueRaw, SQUtils.AmountsArithmetic.fromNumber(valueToReceiveString, expectedToken.decimals).toString())
-            verify(receivePanel.cryptoValueValid)
+            compare(receivePanel.value, valueToReceive)
+            compare(receivePanel.rawValue, SQUtils.AmountsArithmetic.fromNumber(valueToReceiveString, expectedToken.decimals).toString())
+            verify(receivePanel.valueValid)
 
             closeAndVerfyModal()
         }
@@ -1104,24 +1104,18 @@ Item {
                     let maxPossibleValue = Math.trunc(WalletUtils.calculateMaxSafeSendAmount(expectedToken.currentBalance, expectedToken.symbol)*100)/100
                     compare(maxTagButton.text, qsTr("Max. %1").arg(maxPossibleValue === 0 ? Qt.locale().zeroDigit : root.swapAdaptor.currencyStore.formatCurrencyAmount(maxPossibleValue, expectedToken.symbol, {noSymbol: true, minDecimals: 0})))
                     compare(payPanel.selectedHoldingId, expectedToken.symbol)
-                    /* TODO bug in max button not shown in red when max is 0 value and
-                    bug in swapInputModal that in case value entered is greater than maxPossibleValue then value is reset to 0, making the cryptoValueValid to false
-                    https://github.com/status-im/status-desktop/issues/15162 */
-                    compare(payPanel.cryptoValueValid, (valueToExchangeString === amountToSendInput.input.text) && !!root.swapFormData.fromTokenAmount && valueToExchange <= maxPossibleValue)
+                    compare(payPanel.valueValid, !!root.swapFormData.fromTokenAmount && valueToExchange <= maxPossibleValue)
 
-                    /* TODO: there is bug which prevents us from testing this right now
-                    The value is not updated after setting tokenAmount to empty string in the receive input panel
-                    https://github.com/status-im/status-desktop/issues/15162
-                    compare(payPanel.cryptoValue, valueToExchange)
-                    compare(payPanel.cryptoValueRaw, SQUtils.AmountsArithmetic.fromNumber(valueToExchangeString, expectedToken.decimals).toString())*/
+                    compare(payPanel.value, valueToExchange)
+                    compare(payPanel.rawValue, !!valueToExchangeString ? SQUtils.AmountsArithmetic.fromNumber(valueToExchangeString, expectedToken.decimals).toString(): "0")
 
-                    /* TODO: check if tag is visible in case amount entered to exchange is greater than max balance to send
-                    https://github.com/status-im/status-desktop/issues/15162 */
-                    let errortext = /*valueToExchange > maxPossibleValue ? qsTr("Insufficient funds for swap"): */qsTr("An error has occured, please try again")
-                    let buttonText = /*valueToExchange > maxPossibleValue ? qsTr("Buy crypto"):*/ ""
-                    compare(errorTag.visible, false/*valueToExchange > maxPossibleValue*/)
+                    // check if tag is visible in case amount entered to exchange is greater than max balance to send
+                    let amountEnteredGreaterThanMaxBalance = valueToExchange > maxPossibleValue
+                    let errortext = amountEnteredGreaterThanMaxBalance ? qsTr("Insufficient funds for swap"): qsTr("An error has occured, please try again")
+                    compare(errorTag.visible, amountEnteredGreaterThanMaxBalance)
                     compare(errorTag.text, errortext)
-                    compare(errorTag.buttonText, buttonText)
+                    compare(errorTag.buttonText, qsTr("Buy crypto"))
+                    compare(errorTag.buttonVisible, amountEnteredGreaterThanMaxBalance)
                 }
 
                 closeAndVerfyModal()
diff --git a/ui/app/AppLayouts/Wallet/panels/SwapInputPanel.qml b/ui/app/AppLayouts/Wallet/panels/SwapInputPanel.qml
index bd1c79c70..77eb275fd 100644
--- a/ui/app/AppLayouts/Wallet/panels/SwapInputPanel.qml
+++ b/ui/app/AppLayouts/Wallet/panels/SwapInputPanel.qml
@@ -32,23 +32,21 @@ Control {
     onTokenKeyChanged: reevaluateSelectedId()
     property string tokenAmount
     onTokenAmountChanged: {
-        if (!!tokenAmount)
-            Qt.callLater(() => amountToSendInput.input.text = SQUtils.AmountsArithmetic.fromString(tokenAmount).toLocaleString(locale, 'f', -128))
+        Qt.callLater(() => amountToSendInput.input.text = !!tokenAmount ? SQUtils.AmountsArithmetic.fromString(tokenAmount).toLocaleString(locale, 'f', -128): "")
     }
 
     property int swapSide: SwapInputPanel.SwapSide.Pay
     property bool fiatInputInteractive
-    property bool loading
+    property bool mainInputLoading
+    property bool bottomTextLoading
     property bool interactive: true
 
     // output API
     readonly property string selectedHoldingId: d.selectedHoldingId
-    readonly property double cryptoValue: amountToSendInput.cryptoValueToSendFloat
-    readonly property string cryptoValueRaw: amountToSendInput.cryptoValueToSend
-    readonly property bool cryptoValueValid: amountToSendInput.inputNumberValid
-    /* TODO: this does not work as expected because of bug -
-       https://github.com/status-im/status-desktop/issues/15162 */
-    readonly property bool amountEnteredGreaterThanBalance: cryptoValue > maxSendButton.maxSafeValue
+    readonly property double value: amountToSendInput.cryptoValueToSendFloat
+    readonly property string rawValue: amountToSendInput.cryptoValueToSend
+    readonly property bool valueValid: amountToSendInput.inputNumberValid
+    readonly property bool amountEnteredGreaterThanBalance: value > maxSendButton.maxSafeValue
     function reevaluateSelectedId() {
         if (!!tokenKey) {
             Qt.callLater(d.setSelectedHoldingId, tokenKey, Constants.TokenType.ERC20)
@@ -205,7 +203,8 @@ Control {
                                                                                                                        : maxSendButton.maxSafeValue
                 currentCurrency: root.currencyStore.currentCurrency
                 formatCurrencyAmount: root.currencyStore.formatCurrencyAmount
-                loading: root.loading
+                mainInputLoading: root.mainInputLoading
+                bottomTextLoading: root.bottomTextLoading
             }
         }
         ColumnLayout {
@@ -256,7 +255,7 @@ Control {
 
                 value: d.maxInputBalance
                 symbol: d.inputSymbol
-                valid: amountToSendInput.input.valid || !amountToSendInput.input.text
+                valid: (amountToSendInput.input.valid || !amountToSendInput.input.text) && value > 0
                 formatCurrencyAmount: (amount, symbol) => root.currencyStore.formatCurrencyAmount(amount, symbol, {noSymbol: !amountToSendInput.inputIsFiat})
 
                 visible: d.isSelectedHoldingValidAsset && root.swapSide === SwapInputPanel.SwapSide.Pay
diff --git a/ui/app/AppLayouts/Wallet/popups/swap/SwapModal.qml b/ui/app/AppLayouts/Wallet/popups/swap/SwapModal.qml
index 72b4e43e8..f014fe4b0 100644
--- a/ui/app/AppLayouts/Wallet/popups/swap/SwapModal.qml
+++ b/ui/app/AppLayouts/Wallet/popups/swap/SwapModal.qml
@@ -37,13 +37,15 @@ StatusDialog {
     QtObject {
         id: d
          property var debounceFetchSuggestedRoutes: Backpressure.debounce(root, 1000, function() {
-                 root.swapAdaptor.fetchSuggestedRoutes(payPanel.cryptoValueRaw)
+                 root.swapAdaptor.fetchSuggestedRoutes(payPanel.rawValue)
          })
 
         function fetchSuggestedRoutes() {
-            root.swapAdaptor.newFetchReset()
-            root.swapAdaptor.swapProposalLoading = true
-            debounceFetchSuggestedRoutes()
+            if (payPanel.valueValid) {
+                root.swapAdaptor.newFetchReset()
+                root.swapAdaptor.swapProposalLoading = true
+                debounceFetchSuggestedRoutes()
+            }
         }
     }
 
@@ -151,7 +153,7 @@ StatusDialog {
                 tokenAmount: {
                     // Only update if there is different in amount displayed
                     if (root.swapInputParamsForm.fromTokenAmount !==
-                            SQUtils.AmountsArithmetic.fromString(cryptoValue).toLocaleString(locale, 'f', -128)){
+                            SQUtils.AmountsArithmetic.fromString(value).toLocaleString(locale, 'f', -128)){
                         return root.swapInputParamsForm.fromTokenAmount
                     }
                     return payPanel.tokenAmount
@@ -161,7 +163,8 @@ StatusDialog {
                 swapExchangeButtonWidth: swapButton.width
 
                 onSelectedHoldingIdChanged: root.swapInputParamsForm.fromTokensKey = selectedHoldingId
-                onCryptoValueChanged: root.swapInputParamsForm.fromTokenAmount = cryptoValue.toLocaleString(locale, 'f', -128)
+                onValueChanged: root.swapInputParamsForm.fromTokenAmount = value.toLocaleString(locale, 'f', -128)
+                onValueValidChanged: d.fetchSuggestedRoutes()
             }
 
             SwapInputPanel {
@@ -184,7 +187,8 @@ StatusDialog {
                 swapSide: SwapInputPanel.SwapSide.Receive
                 swapExchangeButtonWidth: swapButton.width
 
-                loading: root.swapAdaptor.swapProposalLoading
+                mainInputLoading: root.swapAdaptor.swapProposalLoading
+                bottomTextLoading: root.swapAdaptor.swapProposalLoading
 
                 onSelectedHoldingIdChanged: root.swapInputParamsForm.toTokenKey = selectedHoldingId
 
@@ -236,7 +240,8 @@ StatusDialog {
                 }
                 return qsTr("An error has occured, please try again")
             }
-            buttonText: payPanel.amountEnteredGreaterThanBalance ? qsTr("Buy crypto"): ""
+            buttonText: qsTr("Buy crypto")
+            buttonVisible: payPanel.amountEnteredGreaterThanBalance
             onButtonClicked: Global.openBuyCryptoModalRequested()
         }
     }
diff --git a/ui/imports/shared/controls/ErrorTag.qml b/ui/imports/shared/controls/ErrorTag.qml
index 10bd425d8..fc8595ce0 100644
--- a/ui/imports/shared/controls/ErrorTag.qml
+++ b/ui/imports/shared/controls/ErrorTag.qml
@@ -13,6 +13,7 @@ InformationTag {
 
     property string text
     property string buttonText
+    property bool buttonVisible
 
     signal buttonClicked()
 
@@ -54,7 +55,7 @@ InformationTag {
         horizontalPadding: 8
 
         width: visible || root.loading ? implicitWidth : 0
-        visible: !!text
+        visible: root.buttonVisible
 
         size: StatusBaseButton.Size.Tiny
         font.pixelSize: priv.fontPixelSize
diff --git a/ui/imports/shared/popups/send/views/AmountToSend.qml b/ui/imports/shared/popups/send/views/AmountToSend.qml
index 9e92ad7e1..70f1debcf 100644
--- a/ui/imports/shared/popups/send/views/AmountToSend.qml
+++ b/ui/imports/shared/popups/send/views/AmountToSend.qml
@@ -51,7 +51,8 @@ ColumnLayout {
     property var formatCurrencyAmount:
         (amount, symbol, options = null, locale = null) => {}
 
-    property bool loading
+    property bool mainInputLoading
+    property bool bottomTextLoading
 
     signal reCalculateSuggestedRoute()
 
@@ -88,9 +89,6 @@ ColumnLayout {
         readonly property string selectedSymbol: !!root.selectedHolding && !!root.selectedHolding.symbol ? root.selectedHolding.symbol: ""
 
         readonly property string cryptoValueRawToSend: {
-            if (!root.inputNumberValid)
-                return "0"
-
             return SQUtils.AmountsArithmetic.fromNumber(
                         d.cryptoValueToSend, root.multiplierIndex).toString()
         }
@@ -106,7 +104,8 @@ ColumnLayout {
                                                topAmountToSendInput.locale)
 
         readonly property double inputNumber:
-            root.inputNumberValid ? d.parsedInput : 0
+            // we should still calculate if value entered is greater than max safe value
+            !!input.text && !isNaN(d.parsedInput) && d.parsedInput >= 0 ? d.parsedInput : 0
 
         readonly property Timer waitTimer: Timer {
             interval: 1000
@@ -146,15 +145,9 @@ ColumnLayout {
                     errorMessage: ""
 
                     validate: (text) => {
-                                  var num = 0
-                                  try {
-                                      num = Number.fromLocaleString(topAmountToSendInput.locale, text)
-                                  } catch (e) {
-                                      console.warn(e, "(Error parsing number from text: %1)".arg(text))
-                                      return false
-                                  }
-
-                                  return num > 0 && num <= root.maxInputBalance
+                                  var num = LocaleUtils.numberFromLocaleString(topAmountToSendInput.text,
+                                                                               topAmountToSendInput.locale)
+                                  return !isNaN(num) && num > 0 && num <= root.maxInputBalance
                               }
                 }
             ]
@@ -173,13 +166,13 @@ ColumnLayout {
                     d.waitTimer.restart()
             }
 
-            visible: !root.loading
+            visible: !root.mainInputLoading
         }
         LoadingComponent {
             objectName: "topAmountToSendInputLoadingComponent"
             Layout.preferredWidth: topAmountToSendInput.width
             Layout.preferredHeight: topAmountToSendInput.height
-            visible: root.loading
+            visible: root.mainInputLoading
         }
     }
 
@@ -215,13 +208,13 @@ ColumnLayout {
                 d.waitTimer.restart()
             }
         }
-        visible: !root.loading
+        visible: !root.bottomTextLoading
     }
 
     LoadingComponent {
         objectName: "bottomItemTextLoadingComponent"
         Layout.preferredWidth: bottomItem.width
         Layout.preferredHeight: bottomItem.height
-        visible: root.loading
+        visible: root.bottomTextLoading
     }
 }