mirror of
https://github.com/kremalicious/metamask-extension.git
synced 2024-11-27 12:56:01 +01:00
662c19d4da
* added warning fix added warning for send screen fix changed condition fix fixed test review updates * review updates * fix * fixed failing test * added check for transaction type * fixed failing unit test * added description for localization * review updates * review updates
316 lines
11 KiB
JavaScript
316 lines
11 KiB
JavaScript
import React from 'react';
|
|
import { fireEvent } from '@testing-library/react';
|
|
import { renderWithProvider } from '../../../../test/jest';
|
|
import { submittedPendingTransactionsSelector } from '../../../selectors/transactions';
|
|
import { TRANSACTION_TYPES } from '../../../../shared/constants/transaction';
|
|
import { useGasFeeContext } from '../../../contexts/gasFee';
|
|
import configureStore from '../../../store/store';
|
|
import TransactionAlerts from './transaction-alerts';
|
|
|
|
jest.mock('../../../selectors/transactions', () => {
|
|
return {
|
|
...jest.requireActual('../../../selectors/transactions'),
|
|
submittedPendingTransactionsSelector: jest.fn(),
|
|
};
|
|
});
|
|
|
|
jest.mock('../../../contexts/gasFee');
|
|
|
|
function render({
|
|
componentProps = {},
|
|
useGasFeeContextValue = {},
|
|
submittedPendingTransactionsSelectorValue = null,
|
|
}) {
|
|
useGasFeeContext.mockReturnValue(useGasFeeContextValue);
|
|
submittedPendingTransactionsSelector.mockReturnValue(
|
|
submittedPendingTransactionsSelectorValue,
|
|
);
|
|
const store = configureStore({});
|
|
return renderWithProvider(<TransactionAlerts {...componentProps} />, store);
|
|
}
|
|
|
|
describe('TransactionAlerts', () => {
|
|
describe('when supportsEIP1559V2 from useGasFeeContext is truthy', () => {
|
|
describe('if hasSimulationError from useGasFeeContext is true', () => {
|
|
it('informs the user that a simulation of the transaction failed', () => {
|
|
const { getByText } = render({
|
|
useGasFeeContextValue: {
|
|
supportsEIP1559V2: true,
|
|
hasSimulationError: true,
|
|
},
|
|
});
|
|
|
|
expect(
|
|
getByText(
|
|
'We were not able to estimate gas. There might be an error in the contract and this transaction may fail.',
|
|
),
|
|
).toBeInTheDocument();
|
|
});
|
|
|
|
describe('if the user has not acknowledged the failure', () => {
|
|
it('offers the user an option to bypass the warning', () => {
|
|
const { getByText } = render({
|
|
useGasFeeContextValue: {
|
|
supportsEIP1559V2: true,
|
|
hasSimulationError: true,
|
|
},
|
|
});
|
|
expect(getByText('I want to proceed anyway')).toBeInTheDocument();
|
|
});
|
|
|
|
it('calls setUserAcknowledgedGasMissing if the user bypasses the warning', () => {
|
|
const setUserAcknowledgedGasMissing = jest.fn();
|
|
const { getByText } = render({
|
|
useGasFeeContextValue: {
|
|
supportsEIP1559V2: true,
|
|
hasSimulationError: true,
|
|
},
|
|
componentProps: { setUserAcknowledgedGasMissing },
|
|
});
|
|
fireEvent.click(getByText('I want to proceed anyway'));
|
|
expect(setUserAcknowledgedGasMissing).toHaveBeenCalled();
|
|
});
|
|
});
|
|
|
|
describe('if the user has already acknowledged the failure', () => {
|
|
it('does not offer the user an option to bypass the warning', () => {
|
|
const { queryByText } = render({
|
|
useGasFeeContextValue: {
|
|
supportsEIP1559V2: true,
|
|
hasSimulationError: true,
|
|
},
|
|
componentProps: { userAcknowledgedGasMissing: true },
|
|
});
|
|
expect(
|
|
queryByText('I want to proceed anyway'),
|
|
).not.toBeInTheDocument();
|
|
});
|
|
});
|
|
});
|
|
|
|
describe('if hasSimulationError from useGasFeeContext is falsy', () => {
|
|
it('does not inform the user that a simulation of the transaction failed', () => {
|
|
const { queryByText } = render({
|
|
useGasFeeContextValue: {
|
|
supportsEIP1559V2: true,
|
|
},
|
|
});
|
|
expect(
|
|
queryByText(
|
|
'We were not able to estimate gas. There might be an error in the contract and this transaction may fail.',
|
|
),
|
|
).not.toBeInTheDocument();
|
|
});
|
|
});
|
|
|
|
describe('if the length of pendingTransactions is 1', () => {
|
|
it('informs the user that they have a pending transaction', () => {
|
|
const { getByText } = render({
|
|
useGasFeeContextValue: { supportsEIP1559V2: true },
|
|
submittedPendingTransactionsSelectorValue: [{ some: 'transaction' }],
|
|
});
|
|
expect(
|
|
getByText('You have (1) pending transaction.'),
|
|
).toBeInTheDocument();
|
|
});
|
|
});
|
|
|
|
describe('if the length of pendingTransactions is more than 1', () => {
|
|
it('informs the user that they have pending transactions', () => {
|
|
const { getByText } = render({
|
|
useGasFeeContextValue: { supportsEIP1559V2: true },
|
|
submittedPendingTransactionsSelectorValue: [
|
|
{ some: 'transaction' },
|
|
{ some: 'transaction' },
|
|
],
|
|
});
|
|
expect(
|
|
getByText('You have (2) pending transactions.'),
|
|
).toBeInTheDocument();
|
|
});
|
|
});
|
|
|
|
describe('if the length of pendingTransactions is 0', () => {
|
|
it('does not inform the user that they have pending transactions', () => {
|
|
const { queryByText } = render({
|
|
useGasFeeContextValue: { supportsEIP1559V2: true },
|
|
submittedPendingTransactionsSelectorValue: [],
|
|
});
|
|
expect(
|
|
queryByText('You have (0) pending transactions.'),
|
|
).not.toBeInTheDocument();
|
|
});
|
|
});
|
|
|
|
describe('if balanceError from useGasFeeContext is true', () => {
|
|
it('informs the user that they have insufficient funds', () => {
|
|
const { getByText } = render({
|
|
useGasFeeContextValue: {
|
|
supportsEIP1559V2: true,
|
|
balanceError: true,
|
|
},
|
|
componentProps: {
|
|
nativeCurrency: 'ETH',
|
|
networkName: 'Ropsten',
|
|
showBuyModal: jest.fn(),
|
|
chainId: '0x1',
|
|
type: TRANSACTION_TYPES.DEPLOY_CONTRACT,
|
|
},
|
|
});
|
|
expect(
|
|
getByText(
|
|
/You do not have enough ETH in your account to pay for transaction fees on Ropsten network./u,
|
|
),
|
|
).toBeInTheDocument();
|
|
});
|
|
});
|
|
|
|
describe('if balanceError from useGasFeeContext is falsy', () => {
|
|
it('does not inform the user that they have insufficient funds', () => {
|
|
const { queryByText } = render({
|
|
useGasFeeContextValue: {
|
|
supportsEIP1559V2: true,
|
|
balanceError: false,
|
|
},
|
|
});
|
|
expect(queryByText('Insufficient funds.')).not.toBeInTheDocument();
|
|
});
|
|
});
|
|
|
|
describe('if estimateUsed from useGasFeeContext is "low"', () => {
|
|
it('informs the user that the current transaction is queued', () => {
|
|
const { getByText } = render({
|
|
useGasFeeContextValue: {
|
|
supportsEIP1559V2: true,
|
|
estimateUsed: 'low',
|
|
},
|
|
});
|
|
expect(
|
|
getByText('Future transactions will queue after this one.'),
|
|
).toBeInTheDocument();
|
|
});
|
|
});
|
|
|
|
describe('if estimateUsed from useGasFeeContext is not "low"', () => {
|
|
it('does not inform the user that the current transaction is queued', () => {
|
|
const { queryByText } = render({
|
|
useGasFeeContextValue: {
|
|
supportsEIP1559V2: true,
|
|
estimateUsed: 'something_else',
|
|
},
|
|
});
|
|
expect(
|
|
queryByText('Future transactions will queue after this one.'),
|
|
).not.toBeInTheDocument();
|
|
});
|
|
});
|
|
|
|
describe('if isNetworkBusy from useGasFeeContext is truthy', () => {
|
|
it('informs the user that the network is busy', () => {
|
|
const { getByText } = render({
|
|
useGasFeeContextValue: {
|
|
supportsEIP1559V2: true,
|
|
isNetworkBusy: true,
|
|
},
|
|
});
|
|
expect(
|
|
getByText(
|
|
'Network is busy. Gas prices are high and estimates are less accurate.',
|
|
),
|
|
).toBeInTheDocument();
|
|
});
|
|
});
|
|
|
|
describe('if isNetworkBusy from useGasFeeContext is falsy', () => {
|
|
it('does not inform the user that the network is busy', () => {
|
|
const { queryByText } = render({
|
|
useGasFeeContextValue: {
|
|
supportsEIP1559V2: true,
|
|
isNetworkBusy: false,
|
|
},
|
|
});
|
|
expect(
|
|
queryByText(
|
|
'Network is busy. Gas prices are high and estimates are less accurate.',
|
|
),
|
|
).not.toBeInTheDocument();
|
|
});
|
|
});
|
|
});
|
|
|
|
describe('when supportsEIP1559V2 from useGasFeeContext is falsy', () => {
|
|
describe('if hasSimulationError from useGasFeeContext is true', () => {
|
|
it('does not inform the user that a simulation of the transaction failed', () => {
|
|
const { queryByText } = render({
|
|
useGasFeeContextValue: {
|
|
supportsEIP1559V2: false,
|
|
hasSimulationError: true,
|
|
},
|
|
});
|
|
|
|
expect(
|
|
queryByText(
|
|
'We were not able to estimate gas. There might be an error in the contract and this transaction may fail.',
|
|
),
|
|
).not.toBeInTheDocument();
|
|
});
|
|
});
|
|
|
|
describe('if the length of pendingTransactions is at least 1', () => {
|
|
it('informs the user that they have a pending transaction', () => {
|
|
const { queryByText } = render({
|
|
useGasFeeContextValue: { supportsEIP1559V2: false },
|
|
submittedPendingTransactionsSelectorValue: [{ some: 'transaction' }],
|
|
});
|
|
expect(
|
|
queryByText('You have (1) pending transaction.'),
|
|
).not.toBeInTheDocument();
|
|
});
|
|
});
|
|
|
|
describe('if balanceError from useGasFeeContext is true', () => {
|
|
it('informs the user that they have insufficient funds', () => {
|
|
const { queryByText } = render({
|
|
useGasFeeContextValue: {
|
|
supportsEIP1559V2: false,
|
|
balanceError: true,
|
|
},
|
|
});
|
|
expect(queryByText('Insufficient funds.')).not.toBeInTheDocument();
|
|
});
|
|
});
|
|
|
|
describe('if estimateUsed from useGasFeeContext is "low"', () => {
|
|
it('informs the user that the current transaction is queued', () => {
|
|
const { queryByText } = render({
|
|
useGasFeeContextValue: {
|
|
supportsEIP1559V2: false,
|
|
estimateUsed: 'low',
|
|
},
|
|
});
|
|
expect(
|
|
queryByText(
|
|
'Future transactions will queue after this one. This price was last seen was some time ago.',
|
|
),
|
|
).not.toBeInTheDocument();
|
|
});
|
|
});
|
|
|
|
describe('if isNetworkBusy from useGasFeeContext is truthy', () => {
|
|
it('does not inform the user that the network is busy', () => {
|
|
const { queryByText } = render({
|
|
useGasFeeContextValue: {
|
|
supportsEIP1559V2: false,
|
|
isNetworkBusy: true,
|
|
},
|
|
});
|
|
expect(
|
|
queryByText(
|
|
'Network is busy. Gas prices are high and estimates are less accurate.',
|
|
),
|
|
).not.toBeInTheDocument();
|
|
});
|
|
});
|
|
});
|
|
});
|