diff --git a/cypress/e2e/flows.test.ts b/cypress/e2e/flows.test.ts index c8826067..d6bc8184 100644 --- a/cypress/e2e/flows.test.ts +++ b/cypress/e2e/flows.test.ts @@ -133,19 +133,12 @@ describe('Select tokens & input amount', () => { describe('Input & output amount validation', () => { it('should type an input amount & wait for output amount', () => { cy.visit('/swap'); - //Select input asset -/* cy.get('[data-testid="swap__input__panel"]').within(() => { - cy.get('[data-testid="swap__token__select"]').click(); - }); - cy.get('[data-testid="currency__list__XLM"]').click(); - */ //Select output asset cy.get('[data-testid="swap__output__panel"]').within(() => { cy.get('[data-testid="swap__token__select"]').click(); }); - cy.get('[data-testid="token-search-input"]').type('ngnt'); - cy.get('[data-testid="currency__list__NGNT"]').click(); - + cy.get('[data-testid="token-search-input"]').type('usdc'); + cy.get('[data-testid="currency__list__USDC"]').click(); //Input amount cy.get('[data-testid="swap__input__panel"]').within(() => { @@ -153,7 +146,6 @@ describe('Input & output amount validation', () => { }); //await for calcs cy.wait(5000); - cy.screenshot() //Get the output amount cy.get('[data-testid="swap-output-input-panel"]').invoke('val').as('outputAmount'); //Get the input amount @@ -168,8 +160,8 @@ describe('Input & output amount validation', () => { cy.get('[data-testid="swap__input__panel"]').within(() => { cy.get('[data-testid="swap__token__select"]').click(); }); - cy.get('[data-testid="token-search-input"]').type('ngnt'); - cy.get('[data-testid="currency__list__NGNT"]').click(); + cy.get('[data-testid="token-search-input"]').type('usdc'); + cy.get('[data-testid="currency__list__USDC"]').click(); //Input amount cy.get('[data-testid="swap__output__panel"]').within(() => { cy.get('.token-amount-input').type('{backspace}'); @@ -177,16 +169,63 @@ describe('Input & output amount validation', () => { cy.get('.token-amount-input').type('1'); }); cy.wait(2500); - cy.screenshot() cy.get('[data-testid="swap-input-input-panel"]').invoke('val').then((inputAmount: any)=>{ - const belowOutput = Math.floor(parseFloat(outputAmount) * 0.9); - const aboveOutput = Math.ceil(parseFloat(outputAmount) * 1.1); + const belowOutput = Math.floor(parseFloat(outputAmount) * 0.5); + const aboveOutput = Math.ceil(parseFloat(outputAmount) * 1.5); expect(parseFloat(inputAmount)).within(belowOutput, aboveOutput) }) }) }) }); + +describe('Slippage tolerance config', ()=>{ + it('should change slippage tolerance', ()=>{ + cy.visit('/swap'); + cy.get('[data-testid="open-settings-dialog-button"]').click(); + cy.get('[data-testid="max-slippage-settings"]').click(); + cy.get('[data-testid="slippage-input"]').type('1.05'); + cy.get('[data-testid="max-slippage-settings"]').contains('1.05%'); + }) + it('Show display an alert if slippage is too high', ()=>{ + cy.visit('/swap'); + cy.get('[data-testid="open-settings-dialog-button"]').click(); + cy.get('[data-testid="max-slippage-settings"]').click(); + cy.get('[data-testid="slippage-input"]').type('10'); + cy.get('[data-testid="max-slippage-settings"]').contains('10%'); + cy.get('[data-testid="slippage-alert"]').should('exist'); + cy.get('[data-testid="slippage-alert-too-high"]').should('exist'); + }) + it('Show display an alert if slippage is too low', ()=>{ + cy.visit('/swap'); + cy.get('[data-testid="open-settings-dialog-button"]').click(); + cy.get('[data-testid="max-slippage-settings"]').click(); + cy.get('[data-testid="slippage-input"]').type('0.01'); + cy.get('[data-testid="max-slippage-settings"]').contains('0.01%'); + cy.get('[data-testid="slippage-alert"]').should('exist'); + cy.get('[data-testid="slippage-alert-too-low"]').should('exist'); + }) + it('should keep the slippage tolerance after closing the settings', ()=>{ + cy.visit('/swap'); + cy.get('[data-testid="open-settings-dialog-button"]').click(); + cy.get('[data-testid="max-slippage-settings"]').click(); + cy.get('[data-testid="slippage-input"]').type('1.05'); + cy.get('[data-testid="open-settings-dialog-button"]').click(); + cy.get('[data-testid="max-slippage-settings"]').should('not.exist'); + cy.get('[data-testid="open-settings-dialog-button"]').click(); + cy.get('[data-testid="max-slippage-settings"]').should('exist') + cy.get('[data-testid="max-slippage-settings"]').contains('1.05%'); + }) + it('Should set the slippage tolerance to auto when pressing "auto" button', ()=>{ + cy.visit('/swap'); + cy.get('[data-testid="open-settings-dialog-button"]').click(); + cy.get('[data-testid="max-slippage-settings"]').click(); + cy.get('[data-testid="slippage-input"]').type('1.05'); + cy.get('[data-testid="max-slippage-settings"]').contains('1.05%'); + cy.get('[data-testid="slippage-auto-button"]').click(); + cy.get('[data-testid="max-slippage-settings"]').contains('Auto'); + }) +}) // Navigation flow describe('Navigation flow', () => { it('should render the navbar', () => { diff --git a/src/components/Settings/MaxSlippageSettings/index.tsx b/src/components/Settings/MaxSlippageSettings/index.tsx index 603199cd..77facf7f 100644 --- a/src/components/Settings/MaxSlippageSettings/index.tsx +++ b/src/components/Settings/MaxSlippageSettings/index.tsx @@ -103,15 +103,16 @@ export default function MaxSlippageSettings({ autoSlippage }: { autoSlippage: nu function setCustomSlippage() { // When switching to custom slippage, use `auto` value as a default. - setUserSlippageTolerance(autoSlippage); selectSlippageInput(); } - const removeTrailingZeroes = (num: number) => { - return num + const removeTrailingZeroes = (num: number | SlippageTolerance.Auto) => { + if (num === SlippageTolerance.Auto) return; + const parsedNum = num .toFixed(2) .toString() .replace(/0{1,}$/, ''); + return parsedNum[parsedNum.length - 1] === '.' ? parsedNum.slice(0, -1) : parsedNum; }; useEffect(function () { @@ -139,8 +140,8 @@ export default function MaxSlippageSettings({ autoSlippage }: { autoSlippage: nu } button={ - - {isAuto ? <>Auto : `${removeTrailingZeroes(userSlippageTolerance)}%`} + + {isAuto ? 'Auto' : `${removeTrailingZeroes(userSlippageTolerance)}%`} } > @@ -154,7 +155,7 @@ export default function MaxSlippageSettings({ autoSlippage }: { autoSlippage: nu }} isActive={isAuto} > - + Auto @@ -173,11 +174,11 @@ export default function MaxSlippageSettings({ autoSlippage }: { autoSlippage: nu onChange={(e) => parseSlippageInput(e.target.value)} onBlur={() => { // When the input field is blurred, reset the input field to the default value - setSlippageInput(DEFAULT_SLIPPAGE_INPUT_VALUE); - setSlippageError(false); + //setSlippageInput(DEFAULT_SLIPPAGE_INPUT_VALUE); + //setSlippageError(false); }} onFocus={setCustomSlippage} - type="number" + type="text" step="0.1" /> % @@ -185,14 +186,14 @@ export default function MaxSlippageSettings({ autoSlippage }: { autoSlippage: nu {tooLow || tooHigh ? ( - + {tooLow ? ( -
+
Slippage below {MINIMUM_RECOMMENDED_SLIPPAGE.toFixed(2)}% may result in a failed transaction
) : ( -
Your transaction may be frontrun and result in an unfavorable trade.
+
Your transaction may be frontrun and result in an unfavorable trade.
)}