Merge branch 'main' into hm/refactor-snaps-ui-tests

Author: hmalik88

.github/CODEOWNERS

@@ -42,6 +42,7 @@ app/reducers/fiatOrders/             @MetaMask/ramp
 # Confirmation Team
 app/components/Views/confirmations                          @MetaMask/confirmations
 app/core/Engine/controllers/gas-fee-controller              @MetaMask/confirmations
+app/core/Engine/controllers/signature-controller            @MetaMask/confirmations
 app/core/Engine/controllers/transaction-controller          @MetaMask/confirmations
 app/core/Analytics/events/confirmations                     @MetaMask/confirmations
 ppom                                                        @MetaMask/confirmations

.github/workflows/automated-rca.yml

@@ -0,0 +1,19 @@
+name: Automated RCA
+
+on:
+  issues:
+    types: [closed]
+
+permissions:
+  issues: write
+  contents: read
+
+jobs:
+  automated-rca:
+    uses: MetaMask/github-tools/.github/workflows/post-gh-rca.yml@115cc6dce7aa32c85cbd77a19e9c04db85fb7920
+    with:
+      google-form-base-url: 'https://docs.google.com/forms/d/e/1FAIpQLSdnPbJISzFlR_aQD2uRpnMKSoGAopuTd_yeZK7J4Q5GzgbsOA/viewform?usp=pp_url&entry.340898780='
+      repo-owner: ${{ github.repository_owner }}
+      repo-name: ${{ github.event.repository.name }}
+      issue-number: ${{ github.event.issue.number }}
+      issue-labels: '["Sev0-urgent", "Sev1-high"]'

CHANGELOG.md

@@ -10,11 +10,17 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Added
 
 - feat(multi-srp): enable multi-srp in main and beta ([#14558](https://github.com/MetaMask/metamask-mobile/pull/14558))
+- feat(ramp): Update ramp data flow to fetch cryptos before payment methods ([#14437](https://github.com/MetaMask/metamask-mobile/pull/14437))
 - feat(bridge): add destination account picker ([#14656](https://github.com/MetaMask/metamask-mobile/pull/14656))
 - feat(bridge): add Solana assets to bridge token pickers ([#14365](https://github.com/MetaMask/metamask-mobile/pull/14365))
 - feat: add AppMetadataController controller ([#14513](https://github.com/MetaMask/metamask-mobile/pull/14513))
 - feat(bridge): implement bridge quote fetching ([#14413](https://github.com/MetaMask/metamask-mobile/pull/14413))
 
+
+### Changed
+
+- feat(bridge): Handle Solana vs EVM swap and bridge routing ([#14614](https://github.com/MetaMask/metamask-mobile/pull/14614))
+
 ## [7.44.0]
 
 ### Added

app/components/Snaps/SnapUIAddressInput/SnapUIAddressInput.test.tsx

@@ -0,0 +1,257 @@
+import React from 'react';
+import { fireEvent, render } from '@testing-library/react-native';
+import { SnapUIAddressInput } from './SnapUIAddressInput';
+import { useSnapInterfaceContext } from '../SnapInterfaceContext';
+import { useDisplayName } from '../SnapUIAddress/useDisplayName';
+import { INPUT_TEST_ID } from '../../../component-library/components/Form/TextField/foundation/Input/Input.constants';
+import renderWithProvider from '../../../util/test/renderWithProvider';
+
+const mockInitialState = {
+  settings: {
+    useBlockieIcon: false,
+  },
+};
+
+jest.mock('../SnapInterfaceContext');
+jest.mock('../SnapUIAddress/useDisplayName');
+
+describe('SnapUIAddressInput', () => {
+  const mockHandleInputChange = jest.fn();
+  const mockSetCurrentFocusedInput = jest.fn();
+  const mockGetValue = jest.fn();
+
+  // Define a valid Ethereum chain ID for testing
+  const testChainId = `eip155:0`;
+  const testAddress = '0xabcd5d886577d5081b0c52e242ef29e70be3e7bc';
+
+  beforeEach(() => {
+    (useSnapInterfaceContext as jest.Mock).mockReturnValue({
+      handleInputChange: mockHandleInputChange,
+      getValue: mockGetValue,
+      setCurrentFocusedInput: mockSetCurrentFocusedInput,
+      focusedInput: null,
+    });
+
+    (useDisplayName as jest.Mock).mockReturnValue(undefined);
+
+    jest.clearAllMocks();
+  });
+
+  it('will render', () => {
+    const { getByTestId } = render(
+      <SnapUIAddressInput name="testAddress" chainId={testChainId} />,
+    );
+
+    expect(getByTestId(INPUT_TEST_ID)).toBeTruthy();
+  });
+
+  it('supports existing state', () => {
+    mockGetValue.mockReturnValue(`${testChainId}:${testAddress}`);
+
+    const { getByDisplayValue } = renderWithProvider(
+      <SnapUIAddressInput name="testAddress" chainId={testChainId} />,
+      { state: mockInitialState },
+    );
+
+    expect(getByDisplayValue(testAddress)).toBeTruthy();
+  });
+
+  it('can accept input', () => {
+    const { getByTestId } = renderWithProvider(
+      <SnapUIAddressInput name="testAddress" chainId={testChainId} />,
+      { state: mockInitialState },
+    );
+
+    const textfield = getByTestId(INPUT_TEST_ID);
+    fireEvent.changeText(textfield, '0x');
+    expect(mockHandleInputChange).toHaveBeenCalledWith(
+      'testAddress',
+      `${testChainId}:0x`,
+      undefined,
+    );
+  });
+
+  it('supports a placeholder', () => {
+    const placeholder = 'Enter ETH address';
+    const { getByTestId } = renderWithProvider(
+      <SnapUIAddressInput
+        name="testAddress"
+        chainId={testChainId}
+        placeholder={placeholder}
+      />,
+      { state: mockInitialState },
+    );
+
+    const textfield = getByTestId(INPUT_TEST_ID);
+    expect(textfield.props.placeholder).toBe(placeholder);
+  });
+
+  it('supports the disabled prop', () => {
+    const { getByTestId } = renderWithProvider(
+      <SnapUIAddressInput
+        name="testAddress"
+        chainId={testChainId}
+        disabled
+      />,
+      { state: mockInitialState },
+    );
+
+    const textfield = getByTestId(INPUT_TEST_ID);
+    expect(textfield.props.editable).toBe(false);
+  });
+
+  it('will render within a field', () => {
+    const label = 'Address Field';
+    const { getByText } = renderWithProvider(
+      <SnapUIAddressInput
+        name="testAddress"
+        chainId={testChainId}
+        label={label}
+      />,
+      { state: mockInitialState },
+    );
+
+    expect(getByText(label)).toBeTruthy();
+  });
+
+  it('will render a matched address name', () => {
+    mockGetValue.mockReturnValue(`${testChainId}:${testAddress}`);
+    const displayName = 'Vitalik.eth';
+    (useDisplayName as jest.Mock).mockReturnValue(displayName);
+
+    const { getByText } = renderWithProvider(
+      <SnapUIAddressInput name="testAddress" chainId={testChainId} />,
+      { state: mockInitialState },
+    );
+
+    expect(getByText(displayName)).toBeTruthy();
+    expect(getByText(testAddress)).toBeTruthy();
+  });
+
+  it('will render avatar when displayAvatar is true', () => {
+    mockGetValue.mockReturnValue(`${testChainId}:${testAddress}`);
+    const displayName = 'Vitalik.eth';
+    (useDisplayName as jest.Mock).mockReturnValue(displayName);
+
+    const { toJSON } = renderWithProvider(
+      <SnapUIAddressInput
+        name="testAddress"
+        chainId={testChainId}
+        displayAvatar
+      />,
+      { state: mockInitialState },
+    );
+
+
+    const tree = JSON.stringify(toJSON());
+    expect(tree.includes('RNSVGSvgView')).toBe(true);
+  });
+
+  it('will not render avatar when displayAvatar is false', () => {
+    mockGetValue.mockReturnValue(`${testChainId}:${testAddress}`);
+    const displayName = 'Vitalik.eth';
+    (useDisplayName as jest.Mock).mockReturnValue(displayName);
+
+    const { toJSON } = renderWithProvider(
+      <SnapUIAddressInput
+        name="testAddress"
+        chainId={testChainId}
+        displayAvatar={false}
+      />,
+      { state: mockInitialState },
+    );
+
+    const tree = JSON.stringify(toJSON());
+    expect(tree.includes('RNSVGSvgView')).toBe(false);
+  });
+
+  it('renders with an invalid CAIP Account ID', () => {
+    mockGetValue.mockReturnValue('eip155:0:https://foobar.baz/foobar');
+
+    const { toJSON } = renderWithProvider(<SnapUIAddressInput name="input" chainId="eip155:0" displayAvatar={false} />);
+
+    expect(toJSON()).toMatchSnapshot();
+  });
+
+  it('renders the matched address info in a disabled state', () => {
+    mockGetValue.mockReturnValue(`${testChainId}:${testAddress}`);
+    const displayName = 'Vitalik.eth';
+    (useDisplayName as jest.Mock).mockReturnValue(displayName);
+
+    const { getByText, toJSON, getByTestId } = renderWithProvider(
+      <SnapUIAddressInput name="testAddress" chainId={testChainId} disabled />,
+      { state: mockInitialState },
+    );
+
+    expect(getByText(displayName)).toBeTruthy();
+    expect(getByText(testAddress)).toBeTruthy();
+    const closeButton = getByTestId('snap-ui-address-input__clear-button');
+    expect(closeButton).toBeTruthy();
+
+    const tree = JSON.stringify(toJSON());
+
+    expect(tree).toContain('"opacity":0.5');
+    expect(tree).toContain('"disabled":true');
+    expect(toJSON()).toMatchSnapshot();
+  });
+
+  it('does not render the matched address info if there is an error', () => {
+    mockGetValue.mockReturnValue(`${testChainId}:${testAddress}`);
+    const displayName = 'Vitalik.eth';
+    (useDisplayName as jest.Mock).mockReturnValue(displayName);
+
+    const { queryByText, queryByTestId, getByText, toJSON } = renderWithProvider(
+      <SnapUIAddressInput name="testAddress" chainId={testChainId} error="Error" />,
+      { state: mockInitialState },
+    );
+
+    expect(queryByText(displayName)).toBeFalsy();
+
+    const input = queryByTestId(INPUT_TEST_ID);
+
+    expect(input.props.value).toBe(testAddress);
+    expect(getByText('Error')).toBeTruthy();
+
+    const tree = JSON.stringify(toJSON());
+
+    expect(tree.includes('RNSVGSvgView')).toBe(true);
+  });
+
+  it('disables clear button for the input when disabled', () => {
+    mockGetValue.mockReturnValue(`${testChainId}:${testAddress}`);
+
+    const { getByTestId, toJSON } = renderWithProvider(
+      <SnapUIAddressInput name="testAddress" chainId={testChainId} disabled />,
+    );
+
+    const input = getByTestId(INPUT_TEST_ID);
+    expect(input.props.editable).toBe(false);
+    expect(input.props.value).toBe(testAddress);
+
+    const closeButton = getByTestId('snap-ui-address-input__clear-button');
+    expect(closeButton).toBeTruthy();
+
+    const tree = JSON.stringify(toJSON());
+    expect(tree).toContain('"disabled":true');
+  });
+
+  it('disables clear button for the matched address info when disabled', () => {
+    mockGetValue.mockReturnValue(`${testChainId}:${testAddress}`);
+    const displayName = 'Vitalik.eth';
+    (useDisplayName as jest.Mock).mockReturnValue(displayName);
+
+    const { getByText, getByTestId, toJSON } = renderWithProvider(
+      <SnapUIAddressInput name="testAddress" chainId={testChainId} disabled />,
+      { state: mockInitialState },
+    );
+
+    expect(getByText(displayName)).toBeTruthy();
+    expect(getByText(testAddress)).toBeTruthy();
+
+    const closeButton = getByTestId('snap-ui-address-input__clear-button');
+    expect(closeButton).toBeTruthy();
+
+    const tree = JSON.stringify(toJSON());
+    expect(tree).toContain('"disabled":true');
+  });
+});