refactor: verifySeedPhrase should return a Uint8Array

MetaMask · May 8, 2023 · c415b7d · c415b7d
1 parent 09aacdb
commit c415b7d
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/packages/keyring-controller/src/KeyringController.test.ts b/packages/keyring-controller/src/KeyringController.test.ts
@@ -877,6 +877,11 @@ describe('KeyringController', () => {
       const seedPhrase = await keyringController.verifySeedPhrase();
       expect(seedPhrase).toBeDefined();
     });
+
+    it('should return current seedphrase as Uint8Array', async () => {
+      const seedPhrase = await keyringController.verifySeedPhrase();
+      expect(seedPhrase).toBeInstanceOf(Uint8Array);
+    });
   });
 
   describe('validatePassword', () => {

diff --git a/packages/keyring-controller/src/KeyringController.ts b/packages/keyring-controller/src/KeyringController.ts
@@ -623,14 +623,14 @@ export class KeyringController extends BaseController<
    *
    * @returns Whether the verification succeeds.
    */
-  async verifySeedPhrase(): Promise<string> {
+  async verifySeedPhrase(): Promise<Uint8Array> {
     const primaryKeyring = this.#keyring.getKeyringsByType(KeyringTypes.hd)[0];
     /* istanbul ignore if */
     if (!primaryKeyring) {
       throw new Error('No HD keyring found.');
     }
 
-    const seedWords = (await primaryKeyring.serialize()).mnemonic;
+    const seedWords = primaryKeyring.mnemonic;
     const accounts = await primaryKeyring.getAccounts();
     /* istanbul ignore if */
     if (accounts.length === 0) {