chore(migrations): remove nulls from root state

app/scripts/migrations/194.test.ts

@@ -0,0 +1,167 @@
+import { cloneDeep } from 'lodash';
+import { migrate, version } from './194';
+
+const VERSION = version;
+const OLD_VERSION = VERSION - 1;
+
+describe(`migration #${VERSION}`, () => {
+  it('removes seedWords when value is null', async () => {
+    const oldStorage = {
+      meta: { version: OLD_VERSION },
+      data: {
+        seedWords: null,
+        PreferencesController: {
+          showTestNetworks: true,
+        },
+      },
+    };
+
+    const versionedData = cloneDeep(oldStorage);
+    const changedKeys = new Set<string>();
+
+    await migrate(versionedData, changedKeys);
+
+    expect(versionedData.meta.version).toBe(VERSION);
+    expect('seedWords' in versionedData.data).toBe(false);
+    expect(versionedData.data).toStrictEqual({
+      PreferencesController: {
+        showTestNetworks: true,
+      },
+    });
+    expect(changedKeys).toStrictEqual(new Set(['seedWords']));
+  });
+
+  it('keeps seedWords when value is not null', async () => {
+    const oldStorage = {
+      meta: { version: OLD_VERSION },
+      data: {
+        seedWords: 'mock-seed-phrase',
+      },
+    };
+
+    const versionedData = cloneDeep(oldStorage);
+    const changedKeys = new Set<string>();
+
+    await migrate(versionedData, changedKeys);
+
+    expect(versionedData.meta.version).toBe(VERSION);
+    expect(versionedData.data).toStrictEqual(oldStorage.data);
+    expect(changedKeys.size).toBe(0);
+  });
+
+  it('handles state when seedWords property does not exist', async () => {
+    const oldStorage = {
+      meta: { version: OLD_VERSION },
+      data: {
+        PreferencesController: {
+          showTestNetworks: true,
+        },
+      },
+    };
+
+    const versionedData = cloneDeep(oldStorage);
+    const changedKeys = new Set<string>();
+
+    await migrate(versionedData, changedKeys);
+
+    expect(versionedData.meta.version).toBe(VERSION);
+    expect('seedWords' in versionedData.data).toBe(false);
+    expect(versionedData.data).toStrictEqual(oldStorage.data);
+    expect(changedKeys.size).toBe(0);
+  });
+
+  it('removes forgottenPassword when value is null', async () => {
+    const oldStorage = {
+      meta: { version: OLD_VERSION },
+      data: {
+        forgottenPassword: null,
+        PreferencesController: {
+          showTestNetworks: true,
+        },
+      },
+    };
+
+    const versionedData = cloneDeep(oldStorage);
+    const changedKeys = new Set<string>();
+
+    await migrate(versionedData, changedKeys);
+
+    expect(versionedData.meta.version).toBe(VERSION);
+    expect('forgottenPassword' in versionedData.data).toBe(false);
+    expect(versionedData.data).toStrictEqual({
+      PreferencesController: {
+        showTestNetworks: true,
+      },
+    });
+    expect(changedKeys).toStrictEqual(new Set(['forgottenPassword']));
+  });
+
+  it('keeps forgottenPassword when value is not null', async () => {
+    const oldStorage = {
+      meta: { version: OLD_VERSION },
+      data: {
+        forgottenPassword: true,
+      },
+    };
+
+    const versionedData = cloneDeep(oldStorage);
+    const changedKeys = new Set<string>();
+
+    await migrate(versionedData, changedKeys);
+
+    expect(versionedData.meta.version).toBe(VERSION);
+    expect(versionedData.data).toStrictEqual(oldStorage.data);
+    expect(changedKeys.size).toBe(0);
+  });
+
+  it('handles state when forgottenPassword property does not exist', async () => {
+    const oldStorage = {
+      meta: { version: OLD_VERSION },
+      data: {
+        PreferencesController: {
+          showTestNetworks: true,
+        },
+      },
+    };
+
+    const versionedData = cloneDeep(oldStorage);
+    const changedKeys = new Set<string>();
+
+    await migrate(versionedData, changedKeys);
+
+    expect(versionedData.meta.version).toBe(VERSION);
+    expect('forgottenPassword' in versionedData.data).toBe(false);
+    expect(versionedData.data).toStrictEqual(oldStorage.data);
+    expect(changedKeys.size).toBe(0);
+  });
+
+  it('removes both seedWords and forgottenPassword when both are null', async () => {
+    const oldStorage = {
+      meta: { version: OLD_VERSION },
+      data: {
+        seedWords: null,
+        forgottenPassword: null,
+        PreferencesController: {
+          showTestNetworks: true,
+        },
+      },
+    };
+
+    const versionedData = cloneDeep(oldStorage);
+    const changedKeys = new Set<string>();
+
+    await migrate(versionedData, changedKeys);
+
+    expect(versionedData.meta.version).toBe(VERSION);
+    expect('seedWords' in versionedData.data).toBe(false);
+    expect('forgottenPassword' in versionedData.data).toBe(false);
+    expect(versionedData.data).toStrictEqual({
+      PreferencesController: {
+        showTestNetworks: true,
+      },
+    });
+    expect(changedKeys).toStrictEqual(
+      new Set(['seedWords', 'forgottenPassword']),
+    );
+  });
+});

app/scripts/migrations/194.ts

@@ -0,0 +1,38 @@
+import { hasProperty } from '@metamask/utils';
+import type { Migrate } from './types';
+
+export const version = 194;
+
+/**
+ * Migration that removes `null` seedWords and forgottenPassword values from the persisted state.
+ *
+ * If the `seedWords` property exists on the data object and its value is
+ * `null`, this migration deletes the property and records `seedWords` in
+ * the set of changed keys.
+ *
+ * If the `forgottenPassword` property exists on the data object and its value is
+ * `null`, this migration deletes the property and records `forgottenPassword` in
+ * the set of changed keys.
+ *
+ * @param versionedData - The versioned data object to migrate.
+ * @param changedKeys - A set used to record keys that were modified.
+ */
+export const migrate = (async (versionedData, changedKeys) => {
+  versionedData.meta.version = version;
+
+  if (
+    hasProperty(versionedData.data, 'seedWords') &&
+    versionedData.data.seedWords === null
+  ) {
+    delete versionedData.data.seedWords;
+    changedKeys.add('seedWords');
+  }
+
+  if (
+    hasProperty(versionedData.data, 'forgottenPassword') &&
+    versionedData.data.forgottenPassword === null
+  ) {
+    delete versionedData.data.forgottenPassword;
+    changedKeys.add('forgottenPassword');
+  }
+}) satisfies Migrate;

app/scripts/migrations/index.js

@@ -229,6 +229,7 @@ const migrations = [
   require('./191'),
   require('./192'),
   require('./193'),
+  require('./194'),
 ];
 
 export default migrations;

test/e2e/fixtures/default-fixture.json

@@ -181,7 +181,7 @@
       "announcements": {}
     },
     "AppMetadataController": {
-      "currentMigrationVersion": 193,
+      "currentMigrationVersion": 194,
       "firstTimeInfo": {},
       "previousAppVersion": "",
       "previousMigrationVersion": 0
@@ -1119,6 +1119,6 @@
   },
   "meta": {
     "storageKind": "split",
-    "version": 193
+    "version": 194
   }
 }

test/e2e/fixtures/onboarding-fixture.json

@@ -36,7 +36,7 @@
       "announcements": {}
     },
     "AppMetadataController": {
-      "currentMigrationVersion": 193,
+      "currentMigrationVersion": 194,
       "firstTimeInfo": {
         "date": 0,
         "version": "13.17.0"
@@ -2151,6 +2151,6 @@
   },
   "meta": {
     "storageKind": "split",
-    "version": 193
+    "version": 194
   }
 }

test/e2e/tests/metrics/state-snapshots/errors-before-init-opt-in-ui-state.json

@@ -317,5 +317,5 @@
     },
     "config": "object"
   },
-  "meta": { "version": 193 }
+  "meta": { "version": 194 }
 }