chore(runway): cherry-pick feat: customize fetchInterval for remoteFeatureFlagController to 15min (#13343)

- feat: customize fetchInterval for remoteFeatureFlagController to 15min
(#13341)

## **Description**

- add a new app constant for the 15 min fetch interval
- add fetch interval param using the constant
- add test that controller builder function passes fetch interval param
- remove out of scope unit tests (this was testing the controller. This
is [already tested in controller

itself](https://github.com/MetaMask/core/blob/main/packages/remote-feature-flag-controller/src/remote-feature-flag-controller.test.ts)
and adds a dangerous and blocking link to non local code.

## **Related issues**

Fixes mobile app part of
https://github.com/MetaMask/MetaMask-planning/issues/4098

## **Manual testing steps**

(Manual testing only be done by someone with Launch Darkly access)
1. activate a remote feature flag
2. Load app
3. Check feature is activated
4. disable remote feature flag
5. Wait for 15 min
6. Reload the app
7. Check feature is deactivated

## **Screenshots/Recordings**

<!-- If applicable, add screenshots and/or recordings to visualize the
before and after of your change. -->

### **Before**

<!-- [screenshots/recordings] -->

### **After**

<!-- [screenshots/recordings] -->

## **Pre-merge author checklist**

- [x] I’ve followed [MetaMask Contributor
Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask Mobile
Coding

Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [x] I've completed the PR template to the best of my ability
- [x] I’ve included tests if applicable
- [x] I’ve documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [x] I’ve applied the right labels on the PR (see [labeling

guidelines](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.

## **Pre-merge reviewer checklist**

- [ ] I've manually tested the PR (e.g. pull and build branch, run the
app, test code being changed).
- [ ] I confirm that this PR addresses all acceptance criteria described
in the ticket it closes and includes the necessary testing evidence such
as recordings and or screenshots.
[508a5f7](508a5f7)

---------

Co-authored-by: metamaskbot <[email protected]>

Author: runway-github[bot]

app/core/AppConstants.ts

@@ -5,7 +5,9 @@ import { DEFAULT_SERVER_URL } from '@metamask/sdk-communication-layer';
 const DEVELOPMENT = 'development';
 const PORTFOLIO_URL =
   process.env.MM_PORTFOLIO_URL || 'https://portfolio.metamask.io';
-const SECURITY_ALERTS_API_URL = process.env.SECURITY_ALERTS_API_URL ?? 'https://security-alerts.api.cx.metamask.io';
+const SECURITY_ALERTS_API_URL =
+  process.env.SECURITY_ALERTS_API_URL ??
+  'https://security-alerts.api.cx.metamask.io';
 
 export default {
   IS_DEV: process.env?.NODE_ENV === DEVELOPMENT,
@@ -137,7 +139,9 @@ export default {
       'https://support.metamask.io/transactions-and-gas/transactions/smart-transactions/',
     STAKING_RISK_DISCLOSURE: 'https://consensys.io/staking-risk-disclosures',
   },
-  DECODING_API_URL: process.env.DECODING_API_URL || 'https://signature-insights.api.cx.metamask.io/v1',
+  DECODING_API_URL:
+    process.env.DECODING_API_URL ||
+    'https://signature-insights.api.cx.metamask.io/v1',
   ERRORS: {
     INFURA_BLOCKED_MESSAGE:
       'EthQuery - RPC Error - This service is not available in your country',
@@ -222,5 +226,6 @@ export default {
   FEATURE_FLAGS_API: {
     BASE_URL: 'https://client-config.api.cx.metamask.io',
     VERSION: 'v1',
+    DEFAULT_FETCH_INTERVAL: 15 * 60 * 1000, // 15 minutes
   },
 } as const;

app/core/Engine/controllers/RemoteFeatureFlagController/types.ts

@@ -1,7 +1,11 @@
-import { RemoteFeatureFlagControllerMessenger, RemoteFeatureFlagControllerState } from '@metamask/remote-feature-flag-controller';
+import {
+  RemoteFeatureFlagControllerMessenger,
+  RemoteFeatureFlagControllerState,
+} from '@metamask/remote-feature-flag-controller';
 
 export interface RemoteFeatureFlagInitParamTypes {
   state?: RemoteFeatureFlagControllerState;
   messenger: RemoteFeatureFlagControllerMessenger;
   disabled: boolean;
+  fetchInterval?: number;
 }

app/core/Engine/controllers/RemoteFeatureFlagController/utils.test.ts

@@ -5,6 +5,21 @@ import {
 } from '@metamask/remote-feature-flag-controller';
 import { createRemoteFeatureFlagController } from './utils';
 
+const mockUpdateRemoteFeatureFlags = jest.fn().mockResolvedValue(undefined);
+
+jest.mock('@metamask/remote-feature-flag-controller', () => {
+  const originalModule = jest.requireActual(
+    '@metamask/remote-feature-flag-controller',
+  );
+  return {
+    ...originalModule,
+    RemoteFeatureFlagController: jest.fn().mockImplementation((params) => ({
+      updateRemoteFeatureFlags: mockUpdateRemoteFeatureFlags, // Ensures it returns a resolved promise
+      ...params, // Ensure that fetchInterval and other params are stored
+    })),
+  };
+});
+
 describe('RemoteFeatureFlagController utils', () => {
   let messenger: RemoteFeatureFlagControllerMessenger;
 
@@ -15,86 +30,40 @@ describe('RemoteFeatureFlagController utils', () => {
   });
 
   describe('createRemoteFeatureFlagController', () => {
-    it('creates controller with initial undefined state', () => {
-      const controller = createRemoteFeatureFlagController({
-        state: undefined,
-        messenger,
-        disabled: false,
-      });
-
-      expect(controller).toBeDefined();
-
-      // Initializing with am empty object should return an empty obj?
-      expect(controller.state).toStrictEqual({
-        cacheTimestamp: 0,
-        remoteFeatureFlags: {},
-      });
-    });
-
-    it('internal state matches initial state', () => {
-      const initialState = {
-        remoteFeatureFlags: {
-          testFlag: true,
-        },
-        cacheTimestamp: 123,
-      };
-
-      const controller = createRemoteFeatureFlagController({
-        state: initialState,
-        messenger,
-        disabled: false,
-      });
-
-      expect(controller.state).toStrictEqual(initialState);
-    });
-
     it('calls updateRemoteFeatureFlags when enabled', () => {
-      const spy = jest.spyOn(
-        RemoteFeatureFlagController.prototype,
-        'updateRemoteFeatureFlags',
-      );
-
       createRemoteFeatureFlagController({
         state: undefined,
         messenger,
         disabled: false,
       });
 
-      expect(spy).toHaveBeenCalled();
+      expect(mockUpdateRemoteFeatureFlags).toHaveBeenCalled();
     });
 
     it('does not call updateRemoteFeatureFlagscontroller when controller is disabled', () => {
-      const spy = jest.spyOn(
-        RemoteFeatureFlagController.prototype,
-        'updateRemoteFeatureFlags',
-      );
-
       createRemoteFeatureFlagController({
         state: undefined,
         messenger,
         disabled: true,
       });
 
-      expect(spy).not.toHaveBeenCalled();
+      expect(mockUpdateRemoteFeatureFlags).not.toHaveBeenCalled();
     });
 
-    it('controller keeps initial extra data into its state', () => {
-      const initialState = {
-        extraData: true,
-      };
+    it('passes fetchInterval to RemoteFeatureFlagController', async () => {
+      const fetchInterval = 6000;
 
-      const controller = createRemoteFeatureFlagController({
-        // @ts-expect-error giving a wrong initial state
-        state: initialState,
+      createRemoteFeatureFlagController({
+        state: undefined,
         messenger,
         disabled: false,
+        fetchInterval,
       });
 
-      expect(controller.state).toStrictEqual({
-        cacheTimestamp: 0,
-        extraData: true,
-        remoteFeatureFlags: {},
-      });
+      // Ensure the constructor was called with fetchInterval
+      expect(RemoteFeatureFlagController).toHaveBeenCalledWith(
+        expect.objectContaining({ fetchInterval }),
+      );
     });
   });
 });

app/core/Engine/controllers/RemoteFeatureFlagController/utils.ts

@@ -9,30 +9,39 @@ import {
 import Logger from '../../../../util/Logger';
 
 import { RemoteFeatureFlagInitParamTypes } from './types';
+import AppConstants from '../../../AppConstants';
 
 const getFeatureFlagAppEnvironment = () => {
   const env = process.env.METAMASK_ENVIRONMENT;
   switch (env) {
-    case 'local': return EnvironmentType.Development;
-    case 'pre-release': return EnvironmentType.ReleaseCandidate;
-    case 'production': return EnvironmentType.Production;
-    default: return EnvironmentType.Development;
+    case 'local':
+      return EnvironmentType.Development;
+    case 'pre-release':
+      return EnvironmentType.ReleaseCandidate;
+    case 'production':
+      return EnvironmentType.Production;
+    default:
+      return EnvironmentType.Development;
   }
 };
 
 const getFeatureFlagAppDistribution = () => {
   const dist = process.env.METAMASK_BUILD_TYPE;
   switch (dist) {
-    case 'main': return DistributionType.Main;
-    case 'flask': return DistributionType.Flask;
-    default: return DistributionType.Main;
+    case 'main':
+      return DistributionType.Main;
+    case 'flask':
+      return DistributionType.Flask;
+    default:
+      return DistributionType.Main;
   }
 };
 
 export const createRemoteFeatureFlagController = ({
   state,
   messenger,
   disabled,
+  fetchInterval = AppConstants.FEATURE_FLAGS_API.DEFAULT_FETCH_INTERVAL,
 }: RemoteFeatureFlagInitParamTypes) => {
   const remoteFeatureFlagController = new RemoteFeatureFlagController({
     messenger,
@@ -46,6 +55,7 @@ export const createRemoteFeatureFlagController = ({
         distribution: getFeatureFlagAppDistribution(),
       },
     }),
+    fetchInterval,
   });
 
   if (disabled) {