diff --git a/ui/components/app/edit-gas-fee-popover/edit-gas-fee-popover.test.js b/ui/components/app/edit-gas-fee-popover/edit-gas-fee-popover.test.js index 7fa800668..165cb3de0 100644 --- a/ui/components/app/edit-gas-fee-popover/edit-gas-fee-popover.test.js +++ b/ui/components/app/edit-gas-fee-popover/edit-gas-fee-popover.test.js @@ -79,7 +79,7 @@ const render = ({ txProps, contextProps } = {}) => { describe('EditGasFeePopover', () => { it('should renders low / medium / high options', () => { - render({ txProps: { dappSuggestedGasFees: {} } }); + render({ txProps: { dappSuggestedGasFees: { maxFeePerGas: '0x5208' } } }); expect(screen.queryByText('🐢')).toBeInTheDocument(); expect(screen.queryByText('🦊')).toBeInTheDocument(); diff --git a/ui/components/app/edit-gas-fee-popover/edit-gas-item/edit-gas-item.js b/ui/components/app/edit-gas-fee-popover/edit-gas-item/edit-gas-item.js index 49e7d7443..ddd80c963 100644 --- a/ui/components/app/edit-gas-fee-popover/edit-gas-item/edit-gas-item.js +++ b/ui/components/app/edit-gas-fee-popover/edit-gas-item/edit-gas-item.js @@ -64,7 +64,8 @@ const EditGasItem = ({ priorityLevel }) => { if ( priorityLevel === PRIORITY_LEVELS.DAPP_SUGGESTED && - !dappSuggestedGasFees + !dappSuggestedGasFees?.maxFeePerGas && + !dappSuggestedGasFees?.gasPrice ) { return null; } diff --git a/ui/components/app/edit-gas-fee-popover/edit-gas-item/edit-gas-item.test.js b/ui/components/app/edit-gas-fee-popover/edit-gas-item/edit-gas-item.test.js index d3baf6a3b..469348e2a 100644 --- a/ui/components/app/edit-gas-fee-popover/edit-gas-item/edit-gas-item.test.js +++ b/ui/components/app/edit-gas-fee-popover/edit-gas-item/edit-gas-item.test.js @@ -150,6 +150,23 @@ describe('EditGasItem', () => { expect(screen.queryByTitle('0.0000315 ETH')).toBeInTheDocument(); }); + it('should not renders site gas estimate option for priorityLevel dappSuggested if site does not provided gas estimates', () => { + renderComponent({ + componentProps: { priorityLevel: 'dappSuggested' }, + transactionProps: {}, + }); + expect( + screen.queryByRole('button', { name: 'dappSuggested' }), + ).not.toBeInTheDocument(); + renderComponent({ + componentProps: { priorityLevel: 'dappSuggested' }, + transactionProps: { dappSuggestedGasFees: { gas: '0x59682f10' } }, + }); + expect( + screen.queryByRole('button', { name: 'dappSuggested' }), + ).not.toBeInTheDocument(); + }); + it('should renders advance gas estimate option for priorityLevel custom', () => { renderComponent({ componentProps: { priorityLevel: 'custom' },