feat(btc): use new snap account flow for Bitcoin accounts

ui/components/multichain/create-named-snap-account/create-named-snap-account.test.tsx

@@ -4,18 +4,12 @@ import { fireEvent, renderWithProvider, waitFor } from '../../../../test/jest';
 import configureStore from '../../../store/store';
 import mockState from '../../../../test/data/mock-state.json';
 import messages from '../../../../app/_locales/en/messages.json';
-import { createMockInternalAccount } from '../../../../test/jest/mocks';
 import { ETH_EOA_METHODS } from '../../../../shared/constants/eth-methods';
 import {
   CreateNamedSnapAccount,
   CreateNamedSnapAccountProps,
 } from './create-named-snap-account';
 
-const mockAddress = '0x3f9658179a5c053bb2faaf7badbb95f6c9be0fa7';
-const mockAccount = createMockInternalAccount({
-  address: mockAddress,
-  name: 'New account',
-});
 const mockSnapSuggestedAccountName = 'Suggested Account Name';
 
 jest.mock('../../../store/actions', () => ({
@@ -61,7 +55,6 @@ const mockSnapAccount2 = {
 const render = (
   props: CreateNamedSnapAccountProps = {
     onActionComplete: jest.fn().mockResolvedValue({ success: true }),
-    address: mockAccount.address,
     snapSuggestedAccountName: mockSnapSuggestedAccountName,
   },
 ) => {
@@ -105,7 +98,6 @@ describe('CreateNamedSnapAccount', () => {
     const onActionComplete = jest.fn();
     const { getByText, getByPlaceholderText } = render({
       onActionComplete,
-      address: mockAccount.address,
       snapSuggestedAccountName: mockSnapSuggestedAccountName,
     });
 
@@ -150,7 +142,6 @@ describe('CreateNamedSnapAccount', () => {
     const onActionComplete = jest.fn();
     const { getByText, getByPlaceholderText } = render({
       onActionComplete,
-      address: mockAccount.address,
     });
 
     fireEvent.click(getByText(messages.addAccount.message));
@@ -170,7 +161,6 @@ describe('CreateNamedSnapAccount', () => {
     const onActionComplete = jest.fn();
     const { getByText } = render({
       onActionComplete,
-      address: mockAccount.address,
       snapSuggestedAccountName: mockSnapSuggestedAccountName,
     });
 

ui/components/multichain/create-named-snap-account/create-named-snap-account.tsx

@@ -1,16 +1,12 @@
-import React, { useCallback, useMemo } from 'react';
+import React, { useCallback } from 'react';
 import { useSelector } from 'react-redux';
 import { useHistory } from 'react-router-dom';
 import { InternalAccount } from '@metamask/keyring-api';
 import { KeyringTypes } from '@metamask/keyring-controller';
-import { CreateAccount, CreateBtcAccount } from '..';
+import { CreateAccount } from '..';
 import { Box, ModalHeader } from '../../component-library';
 import { useI18nContext } from '../../../hooks/useI18nContext';
 import { getMostRecentOverviewPage } from '../../../ducks/history/history';
-import {
-  isBtcMainnetAddress,
-  isBtcTestnetAddress,
-} from '../../../../shared/lib/multichain';
 import { getNextAvailableAccountName } from '../../../store/actions';
 
 export type CreateNamedSnapAccountProps = {
@@ -22,11 +18,6 @@ export type CreateNamedSnapAccountProps = {
     reject?: boolean,
   ) => Promise<void>;
 
-  /**
-   * Address of the account to create
-   */
-  address: string;
-
   /**
    * Suggested account name from the snap
    */
@@ -35,17 +26,12 @@ export type CreateNamedSnapAccountProps = {
 
 export const CreateNamedSnapAccount: React.FC<CreateNamedSnapAccountProps> = ({
   onActionComplete,
-  address,
   snapSuggestedAccountName,
 }) => {
   const t = useI18nContext();
   const history = useHistory();
 
   const mostRecentOverviewPage = useSelector(getMostRecentOverviewPage);
-  const isBtcAccount = useMemo(
-    () => isBtcMainnetAddress(address) || isBtcTestnetAddress(address),
-    [address],
-  );
 
   const rejectAction = useCallback(async (completed: boolean) => {
     await onActionComplete({ success: completed });
@@ -81,23 +67,13 @@ export const CreateNamedSnapAccount: React.FC<CreateNamedSnapAccountProps> = ({
       <ModalHeader padding={4} onClose={onClose}>
         {t('addAccountToMetaMask')}
       </ModalHeader>
-      {isBtcAccount ? (
-        <CreateBtcAccount
-          // onActionComplete is called when a user rejects the action
-          // onCreateAccount will call onActionComplete if a user confirms the action
-          onActionComplete={rejectAction}
-          onCreateAccount={onCreateAccount}
-          address={address}
-        />
-      ) : (
-        <CreateAccount
-          // onActionComplete is called when a user rejects the action
-          // onCreateAccount will call onActionComplete if a user confirms the action
-          onActionComplete={rejectAction}
-          onCreateAccount={onCreateAccount}
-          getNextAvailableAccountName={getNextAccountName}
-        />
-      )}
+      <CreateAccount
+        // onActionComplete is called when a user rejects the action
+        // onCreateAccount will call onActionComplete if a user confirms the action
+        onActionComplete={rejectAction}
+        onCreateAccount={onCreateAccount}
+        getNextAvailableAccountName={getNextAccountName}
+      />
     </Box>
   );
 };

ui/components/multichain/index.js

@@ -18,7 +18,6 @@ export { ProductTour } from './product-tour-popover';
 export { AccountDetails } from './account-details';
 export { CreateAccount } from './create-account';
 export { CreateEthAccount } from './create-eth-account';
-export { CreateBtcAccount } from './create-btc-account';
 export { ConnectedAccountsMenu } from './connected-accounts-menu';
 export { ImportAccount } from './import-account';
 export { ImportNftsModal } from './import-nfts-modal';

ui/pages/confirmations/confirmation/templates/create-named-snap-account.js

@@ -18,7 +18,7 @@ import {
  */
 function getValues(pendingApproval, t, actions, _history, _data, contexts) {
   const { origin: snapId, snapName, requestData } = pendingApproval;
-  const { address, snapSuggestedAccountName } = requestData;
+  const { snapSuggestedAccountName } = requestData;
   const { trackEvent } = contexts;
 
   const trackSnapAccountEvent = (event) => {
@@ -52,7 +52,6 @@ function getValues(pendingApproval, t, actions, _history, _data, contexts) {
         key: 'create-named-snap-account',
         props: {
           onActionComplete,
-          address,
           snapSuggestedAccountName,
         },
       },