Revert "Add AccountSelector component (#2764)"

This reverts commit 73d00a0.
MetaMask · Oct 7, 2024 · edb935d · edb935d
1 parent 72e23f0
commit edb935d
Show file tree

Hide file tree

Showing 5 changed files with 0 additions and 165 deletions.
diff --git a/packages/snaps-sdk/src/jsx/components/form/AccountSelector.test.tsx b/packages/snaps-sdk/src/jsx/components/form/AccountSelector.test.tsx
diff --git a/packages/snaps-sdk/src/jsx/components/form/AccountSelector.ts b/packages/snaps-sdk/src/jsx/components/form/AccountSelector.ts
diff --git a/packages/snaps-sdk/src/jsx/components/form/index.ts b/packages/snaps-sdk/src/jsx/components/form/index.ts
@@ -1,4 +1,3 @@
-import type { AccountSelectorElement } from './AccountSelector';
 import type { ButtonElement } from './Button';
 import type { CheckboxElement } from './Checkbox';
 import type { DropdownElement } from './Dropdown';
@@ -12,7 +11,6 @@ import type { RadioGroupElement } from './RadioGroup';
 import type { SelectorElement } from './Selector';
 import type { SelectorOptionElement } from './SelectorOption';
 
-export * from './AccountSelector';
 export * from './Button';
 export * from './Checkbox';
 export * from './Dropdown';
@@ -27,7 +25,6 @@ export * from './Selector';
 export * from './SelectorOption';
 
 export type StandardFormElement =
-  | AccountSelectorElement
   | ButtonElement
   | CheckboxElement
   | FormElement

diff --git a/packages/snaps-sdk/src/jsx/validation.test.tsx b/packages/snaps-sdk/src/jsx/validation.test.tsx
@@ -32,7 +32,6 @@ import {
   Selector,
   SelectorOption,
   Section,
-  AccountSelector,
 } from './components';
 import {
   AddressStruct,
@@ -70,7 +69,6 @@ import {
   SelectorStruct,
   SectionStruct,
   NotificationComponentsStruct,
-  AccountSelectorStruct,
 } from './validation';
 
 describe('KeyStruct', () => {
@@ -958,69 +956,6 @@ describe('FileInputStruct', () => {
   });
 });
 
-describe('AccountSelectorStruct', () => {
-  it.each([
-    <AccountSelector
-      name="account"
-      title="From Account"
-      chainId="bip122:000000000019d6689c085ae165831e93"
-      selectedAddress="128Lkh3S7CkDTBZ8W7BbpsN3YYizJMp8p6"
-    />,
-    <AccountSelector
-      name="account"
-      title="From Account"
-      chainId="eip155:1"
-      selectedAddress="0x1234567890123456789012345678901234567890"
-    />,
-  ])('validates an account picker element', (value) => {
-    expect(is(value, AccountSelectorStruct)).toBe(true);
-  });
-
-  it.each([
-    'foo',
-    42,
-    null,
-    undefined,
-    {},
-    [],
-    // @ts-expect-error - Invalid props.
-    <AccountSelector />,
-    // @ts-expect-error - Invalid props.
-    <AccountSelector>
-      <Text>foo</Text>
-    </AccountSelector>,
-    // @ts-expect-error - Invalid props.
-    <AccountSelector name="account" />,
-    // @ts-expect-error - Invalid props.
-    <AccountSelector title="From Account" />,
-    // @ts-expect-error - Invalid props.
-    <AccountSelector chainId="bip122:000000000019d6689c085ae165831e93" />,
-    // @ts-expect-error - Invalid props.
-    <AccountSelector selectedAddress="128Lkh3S7CkDTBZ8W7BbpsN3YYizJMp8p6" />,
-    <AccountSelector
-      name="account"
-      title="From Account"
-      chainId="foo:bar"
-      selectedAddress="0x1234567890123456789012345678901234567890"
-    />,
-    <AccountSelector
-      name="account"
-      title="From Account"
-      chainId="eip155:1"
-      selectedAddress="0x123"
-    />,
-    <Text>foo</Text>,
-    <Box>
-      <Text>foo</Text>
-    </Box>,
-    <Row label="label">
-      <Image src="<svg />" alt="alt" />
-    </Row>,
-  ])('does not validate "%p"', (value) => {
-    expect(is(value, AddressStruct)).toBe(false);
-  });
-});
-
 describe('SelectorStruct', () => {
   it.each([
     <Selector name="foo" title="Title">

diff --git a/packages/snaps-sdk/src/jsx/validation.ts b/packages/snaps-sdk/src/jsx/validation.ts
@@ -20,9 +20,7 @@ import {
   refine,
 } from '@metamask/superstruct';
 import {
-  CaipAccountAddressStruct,
   CaipAccountIdStruct,
-  CaipChainIdStruct,
   hasProperty,
   HexChecksumAddressStruct,
   isPlainObject,
@@ -49,7 +47,6 @@ import type {
   StringElement,
 } from './component';
 import {
-  type AccountSelectorElement,
   type AddressElement,
   type BoldElement,
   type BoxElement,
@@ -334,22 +331,6 @@ export const FileInputStruct: Describe<FileInputElement> = element(
   },
 );
 
-/**
- * A struct for the {@link AccountSelectorElement} type.
- */
-export const AccountSelectorStruct: Describe<AccountSelectorElement> = element(
-  'AccountSelector',
-  {
-    name: string(),
-    title: string(),
-    chainId: CaipChainIdStruct as unknown as Struct<
-      Infer<typeof CaipChainIdStruct>,
-      Infer<typeof CaipChainIdStruct>
-    >,
-    selectedAddress: CaipAccountAddressStruct,
-  },
-);
-
 /**
  * A subset of JSX elements that represent the tuple Box + Input of the Field children.
  */
@@ -843,7 +824,6 @@ export const JSXElementStruct: Describe<JSXElement> = typedUnion([
   SelectorStruct,
   SelectorOptionStruct,
   SectionStruct,
-  AccountSelectorStruct,
 ]);
 
 /**