migration: remove featureFlags property

app/store/migrations/061.test.ts

@@ -0,0 +1,59 @@
+import migrate from './061';
+import { captureException } from '@sentry/react-native';
+import mockedEngine from '../../core/__mocks__/MockedEngine';
+
+jest.mock('@sentry/react-native', () => ({
+  captureException: jest.fn(),
+}));
+const mockedCaptureException = jest.mocked(captureException);
+
+jest.mock('../../core/Engine', () => ({
+  init: () => mockedEngine.init(),
+}));
+
+describe('Migration #61 - remove featureFlags property from redux state', () => {
+  beforeEach(() => {
+    jest.restoreAllMocks();
+    jest.resetAllMocks();
+  });
+
+  const invalidStates = [
+    {
+      state: null,
+      errorMessage: "FATAL ERROR: Migration 61: Invalid state error: 'object'",
+      scenario: 'state is invalid',
+    },
+  ];
+
+  for (const { errorMessage, scenario, state } of invalidStates) {
+    it(`should capture exception if ${scenario}`, async () => {
+      const newState = await migrate(state);
+
+      expect(newState).toStrictEqual(state);
+      expect(mockedCaptureException).toHaveBeenCalledWith(expect.any(Error));
+      expect(mockedCaptureException.mock.calls[0][0].message).toBe(
+        errorMessage,
+      );
+    });
+  }
+
+  it('remove featureFlags property from redux state', async () => {
+    const oldState = {
+      engine: {
+        backgroundState: {},
+      },
+      featureFlags: {
+        minimumAppVersion: 29,
+      },
+    };
+
+    const expectedState = {
+      engine: {
+        backgroundState: {},
+      },
+    };
+
+    const migratedState = await migrate(oldState);
+    expect(migratedState).toStrictEqual(expectedState);
+  });
+});

app/store/migrations/061.ts

@@ -0,0 +1,12 @@
+import { hasProperty } from '@metamask/utils';
+import { ensureValidState } from "./util";
+
+export default function migrate(state: unknown) {
+  if (!ensureValidState(state, 61)) {
+    return state;
+  }
+  if (hasProperty(state, 'featureFlags')) {
+    delete state.featureFlags;
+  }
+  return state;
+}

app/store/migrations/index.ts

@@ -61,6 +61,7 @@ import migration57 from './057';
 import migration58 from './058';
 import migration59 from './059';
 import migration60 from './060';
+import migration61 from './061';
 
 type MigrationFunction = (state: unknown) => unknown;
 type AsyncMigrationFunction = (state: unknown) => Promise<unknown>;
@@ -133,7 +134,8 @@ export const migrationList: MigrationsList = {
   57: migration57,
   58: migration58,
   59: migration59,
-  60: migration60
+  60: migration60,
+  61: migration61
 };
 
 // Enable both synchronous and asynchronous migrations