From 5f6b020693b6497c40728e80c281b5cc3028c542 Mon Sep 17 00:00:00 2001
From: Michele Esposito <34438276+mikesposito@users.noreply.github.com>
Date: Tue, 5 Nov 2024 14:35:53 +0100
Subject: [PATCH] chore: remove `addNewAccountWithoutUpdate` (#4845)

---
 packages/keyring-controller/CHANGELOG.md      |  4 +++
 packages/keyring-controller/jest.config.js    |  4 +--
 .../src/KeyringController.test.ts             | 26 -------------------
 .../src/KeyringController.ts                  | 19 --------------
 4 files changed, 6 insertions(+), 47 deletions(-)

diff --git a/packages/keyring-controller/CHANGELOG.md b/packages/keyring-controller/CHANGELOG.md
index 13d066a87a..8e9a19a7f6 100644
--- a/packages/keyring-controller/CHANGELOG.md
+++ b/packages/keyring-controller/CHANGELOG.md
@@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Removed
+
+- **BREAKING** Remove `addNewAccountWithoutUpdate` method ([#4845](https://github.com/MetaMask/core/pull/4845))
+
 ## [17.3.1]
 
 ### Changed
diff --git a/packages/keyring-controller/jest.config.js b/packages/keyring-controller/jest.config.js
index 912580f270..3dbee99897 100644
--- a/packages/keyring-controller/jest.config.js
+++ b/packages/keyring-controller/jest.config.js
@@ -19,8 +19,8 @@ module.exports = merge(baseConfig, {
     global: {
       branches: 95.51,
       functions: 100,
-      lines: 99.08,
-      statements: 99.09,
+      lines: 99.07,
+      statements: 99.08,
     },
   },
 
diff --git a/packages/keyring-controller/src/KeyringController.test.ts b/packages/keyring-controller/src/KeyringController.test.ts
index 10a2c5311f..d8e6218725 100644
--- 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
index 3e15c3f5b5..375397fd60 100644
--- 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.