chore: remove addNewAccountWithoutUpdate

MetaMask · Oct 24, 2024 · 884d344 · 884d344
1 parent 35b900b
commit 884d344
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 45 deletions.
diff --git a/packages/keyring-controller/src/KeyringController.test.ts b/packages/keyring-controller/src/KeyringController.test.ts
@@ -361,32 +361,6 @@ describe('KeyringController', () => {
     });
   });
 
-  describe('addNewAccountWithoutUpdate', () => {
-    it('should add new account without updating', async () => {
-      await withController(async ({ controller, initialState }) => {
-        await controller.addNewAccountWithoutUpdate();
-        expect(initialState.keyrings).toHaveLength(1);
-        expect(initialState.keyrings[0].accounts).not.toStrictEqual(
-          controller.state.keyrings[0].accounts,
-        );
-        expect(controller.state.keyrings[0].accounts).toHaveLength(2);
-        // we make sure that updateIdentities is not called
-        // during this test
-      });
-    });
-
-    it('should throw error with no HD keyring', async () => {
-      await withController(
-        { skipVaultCreation: true },
-        async ({ controller }) => {
-          await expect(controller.addNewAccountWithoutUpdate()).rejects.toThrow(
-            'No HD keyring found',
-          );
-        },
-      );
-    });
-  });
-
   describe('addNewKeyring', () => {
     describe('when there is a builder for the given type', () => {
       it('should add new keyring', async () => {

diff --git a/packages/keyring-controller/src/KeyringController.ts b/packages/keyring-controller/src/KeyringController.ts
@@ -732,25 +732,6 @@ export class KeyringController extends BaseController<
     });
   }
 
-  /**
-   * Adds a new account to the default (first) HD seed phrase keyring without updating identities in preferences.
-   *
-   * @returns Promise resolving to the added account address.
-   */
-  async addNewAccountWithoutUpdate(): Promise<string> {
-    return this.#persistOrRollback(async () => {
-      const primaryKeyring = this.getKeyringsByType('HD Key Tree')[0] as
-        | EthKeyring<Json>
-        | undefined;
-      if (!primaryKeyring) {
-        throw new Error('No HD keyring found');
-      }
-      const [addedAccountAddress] = await primaryKeyring.addAccounts(1);
-      await this.verifySeedPhrase();
-      return addedAccountAddress;
-    });
-  }
-
   /**
    * Effectively the same as creating a new keychain then populating it
    * using the given seed phrase.