test: MMQA - 1711 - [Mobile] Update Predict e2e tests to enable features that were disabled due to feature flag updates (#29154)

<!--
Please submit this PR as a draft initially.

Do not mark it as "Ready for review" until this PR meets the canonical
Definition of Ready For Review in `docs/readme/ready-for-review.md`.
-->

## **Description**

Brings Predict E2E test feature flag mocks into parity with production
values.

Two remote feature flag overrides in predict smoke tests diverged from
production after a feature flag registry sync ([PR
#28444](#28444)). This
PR aligns the mocks:

- **`exploreSectionsOrder`**: Replaced the explicit section ordering
arrays with `{}` in 4 predict smoke specs. On production this flag is
empty; the app falls back to hardcoded defaults which produce the same
layout. The Homepage component does not consume this flag at all — it
only affects TrendingView/Explore, which these tests don't exercise.
- **`predictLiveSports`**: Added `'nba'` to the leagues list (now
`['nfl', 'nba']`) to match production. This causes NBA markets (Spurs
vs. Pelicans) to render via `PredictGameDetailsContent`, which uses a
different cash-out button test ID
(`predict-picks-cash-out-button-{positionId}`). Updated the
`PredictDetailsPage` page object with a new
`tapGameCashOutButton(positionId)` method and updated
`predict-cash-out.spec.ts` to use it.

## **Changelog**

CHANGELOG entry: null

## **Related issues**

Fixes:
[MMQA-1711](https://consensyssoftware.atlassian.net/browse/MMQA-1711)

## **Manual testing steps**

<!-- Gherkin scenarios for this change -->

**Feature: Predict E2E prod parity**

**Scenario: Cash out on NBA game market uses correct button**
- Given the predict smoke tests use production-parity feature flag mocks
- When the predict-cash-out test runs against Spurs vs. Pelicans
- Then the test taps the game-details cash-out button (PredictPickItem
testID)
- And the cash-out flow completes successfully

**Scenario: Predict tests work without explicit exploreSectionsOrder**
- Given exploreSectionsOrder is set to `{}` in all predict smoke specs
- When any predict smoke test runs (cash-out, open-position,
geo-restriction, withdraw)
- Then the Homepage renders with the default section order
- And the test navigates to predictions by test ID without issue

## **Screenshots/Recordings**

### **Before**

N/A — test-only changes, no UI impact.

### **After**

N/A — test-only changes, no UI impact.

## **Pre-merge author checklist**

- [ ] 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).
- [ ] I've completed the PR template to the best of my ability
- [ ] I've included tests if applicable
- [ ] I've documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [ ] 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.

[MMQA-1711]:
https://consensyssoftware.atlassian.net/browse/MMQA-1711?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Test-only changes that adjust feature-flag mocks and E2E selectors;
low product risk, but could affect Predict smoke test stability if
testIDs/position IDs drift.
> 
> **Overview**
> Aligns Predict E2E remote feature-flag mocks with current production
defaults by removing the `predictLiveSports` override from
`remoteFeatureFlagPredictEnabled`.
> 
> Updates Predict smoke specs to interact with the *game-details* UI:
adds a shared `SPURS_PELICANS_POSITION_ID`, introduces
`PredictDetailsPage.tapGameCashOutButton(positionId)` and
`tapGameBetYesButton()`, and switches cash-out/position-opening tests to
use the new testIDs for NBA/game markets.
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
f452736. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Author: LeVinhGithub

tests/api-mocking/mock-responses/feature-flags-mocks.ts

@@ -132,13 +132,6 @@ export const remoteFeatureFlagPredictEnabled = (enabled = true) => ({
     enabled: false,
     minimumVersion: '7.60.0',
   },
-  // Exclude 'nba' — NBA markets render PredictGameDetailsContent which uses a different cash-out testID not yet supported by E2E page objects
-  predictLiveSports: {
-    versions: {
-      '7.67.0': { enabled: true, leagues: ['nfl'] },
-      '7.70.0': { enabled: true, leagues: ['nfl'] },
-    },
-  },
 });
 
 export const remoteFeatureFlagHomepageSectionsV1Enabled = (enabled = true) => ({

tests/api-mocking/mock-responses/polymarket/polymarket-constants.ts

@@ -39,6 +39,10 @@ export const CONDITIONAL_TOKENS_CONTRACT_ADDRESS =
 export const POLYGON_EIP7702_CONTRACT_ADDRESS =
   '0x63c0c19a282a1B52b07dD5a65b58948A07DAE32B';
 
+// Predict position ID for Spurs vs. Pelicans
+export const SPURS_PELICANS_POSITION_ID =
+  '110743925263777693447488608878982152642205002490046349037358337248548507433643';
+
 // EIP-7702 format: 0xef01 (magic byte) + 00 (padding) + 20-byte contract address
 // This format indicates an EOA is upgraded with EIP-7702
 export const EIP7702_CODE_FORMAT = (contractAddress: string): string => {

tests/page-objects/Predict/PredictDetailsPage.ts

@@ -14,6 +14,13 @@ import {
   PredictMarketDetailsSelectorsIDs,
   PredictMarketDetailsSelectorsText,
 } from '../../../app/components/UI/Predict/Predict.testIds';
+import { PREDICT_PICK_ITEM_TEST_IDS } from '../../../app/components/UI/Predict/components/PredictPicks/PredictPickItem.testIds';
+import {
+  PREDICT_GAME_DETAILS_FOOTER,
+  PREDICT_GAME_DETAILS_FOOTER_TEST_IDS,
+} from '../../../app/components/UI/Predict/components/PredictGameDetailsFooter/PredictGameDetailsFooter.testIds';
+import { PREDICT_ACTION_BUTTONS_TEST_IDS } from '../../../app/components/UI/Predict/components/PredictActionButtons/PredictActionButtons.testIds';
+import { PREDICT_BET_BUTTONS_TEST_IDS } from '../../../app/components/UI/Predict/components/PredictActionButtons/PredictBetButtons.testIds';
 
 class PredictDetailsPage {
   get container(): EncapsulatedElementType {
@@ -208,6 +215,14 @@ class PredictDetailsPage {
     });
   }
 
+  get gameBetYesButton(): EncapsulatedElementType {
+    const testID = `${PREDICT_GAME_DETAILS_FOOTER}${PREDICT_GAME_DETAILS_FOOTER_TEST_IDS.ACTION_BUTTONS}${PREDICT_ACTION_BUTTONS_TEST_IDS.PREDICT_BET_BUTTON}${PREDICT_BET_BUTTONS_TEST_IDS.PREDICT_BET_BUTTON_YES}`;
+    return encapsulated({
+      detox: () => Matchers.getElementByID(testID),
+      appium: () => PlaywrightMatchers.getElementById(testID, { exact: true }),
+    });
+  }
+
   async waitForScreenToDisplay(): Promise<void> {
     await Assertions.expectElementToBeVisible(this.container, {
       description: 'Predict market details screen',
@@ -246,12 +261,32 @@ class PredictDetailsPage {
     });
   }
 
+  getGameCashOutButton(positionId: string): EncapsulatedElementType {
+    const testID = `${PREDICT_PICK_ITEM_TEST_IDS.PREDICT_PICKS_CASH_OUT_BUTTON}-${positionId}`;
+    return encapsulated({
+      detox: () => Matchers.getElementByID(testID),
+      appium: () => PlaywrightMatchers.getElementById(testID, { exact: true }),
+    });
+  }
+
+  async tapGameCashOutButton(positionId: string): Promise<void> {
+    await UnifiedGestures.waitAndTap(this.getGameCashOutButton(positionId), {
+      description: 'Game details cash out button',
+    });
+  }
+
   async tapOpenPositionValue(): Promise<void> {
     await UnifiedGestures.waitAndTap(this.getOpenPositionValueButton(), {
       description: 'Celtics outcome button',
     });
   }
 
+  async tapGameBetYesButton(): Promise<void> {
+    await UnifiedGestures.waitAndTap(this.gameBetYesButton, {
+      description: 'Game bet yes button',
+    });
+  }
+
   async tapPositionAmount(amount: string): Promise<void> {
     const digits = amount.split('');
 

tests/smoke/predict/predict-cash-out.spec.ts

@@ -24,6 +24,7 @@ import TabBarComponent from '../../page-objects/wallet/TabBarComponent';
 import ActivitiesView from '../../page-objects/Transactions/ActivitiesView';
 import PredictActivityDetails from '../../page-objects/Transactions/predictionsActivityDetails';
 import { predictCashOutFlowAnalyticsExpectations } from '../../helpers/analytics/expectations/predict-cash-out.analytics';
+import { SPURS_PELICANS_POSITION_ID } from '../../api-mocking/mock-responses/polymarket/polymarket-constants';
 
 /*
 Test Scenario: Cash out on open position - Spurs vs. Pelicans
@@ -68,7 +69,9 @@ describe(SmokePredictions('Predictions'), () => {
         await Assertions.expectElementToBeVisible(PredictDetailsPage.container);
         await POLYMARKET_POST_CASH_OUT_MOCKS(mockServer);
 
-        await PredictDetailsPage.tapCashOutButton();
+        await PredictDetailsPage.tapGameCashOutButton(
+          SPURS_PELICANS_POSITION_ID,
+        );
         await Assertions.expectElementToBeVisible(PredictCashOutPage.container);
 
         await Assertions.expectElementToBeVisible(

tests/smoke/predict/predict-geo-restriction.spec.ts

@@ -22,6 +22,9 @@ import {
   POLYMARKET_GEO_BLOCKED_MOCKS,
 } from '../../api-mocking/mock-responses/polymarket/polymarket-mocks';
 import PredictAddFunds from '../../page-objects/Predict/PredictAddFunds';
+import { getEventsPayloads } from '../../helpers/analytics/helpers';
+import SoftAssert from '../../framework/SoftAssert';
+import { SPURS_PELICANS_POSITION_ID } from '../../api-mocking/mock-responses/polymarket/polymarket-constants';
 import {
   geoBlockedPredictActionExpectations,
   geoBlockedCashoutExpectations,
@@ -119,7 +122,9 @@ describe(
           await WalletView.scrollAndTapPredictionsPosition(
             'Spurs vs. Pelicans',
           );
-          await PredictDetailsPage.tapCashOutButton();
+          await PredictDetailsPage.tapGameCashOutButton(
+            SPURS_PELICANS_POSITION_ID,
+          );
 
           await PredictUnavailableView.expectVisible();
           await PredictUnavailableView.tapGotIt();

tests/smoke/predict/predict-open-position.spec.ts

@@ -78,7 +78,7 @@ describe(SmokePredictions('Predictions'), () => {
           positionDetails.category,
           positionDetails.marketIndex,
         );
-        await PredictDetailsPage.tapOpenPositionValue();
+        await PredictDetailsPage.tapGameBetYesButton();
 
         await POLYMARKET_POST_OPEN_POSITION_MOCKS(mockServer);
 
@@ -89,18 +89,6 @@ describe(SmokePredictions('Predictions'), () => {
 
         await PredictDetailsPage.tapOpenPosition();
 
-        await Assertions.expectElementToBeVisible(
-          PredictDetailsPage.positionsTab,
-          {
-            description:
-              'Position tab should appear after opening a new position',
-          },
-        );
-
-        await Assertions.expectTextDisplayed(positionDetails.name, {
-          description: 'Position card for Celtics vs. Nets should appear',
-        });
-
         await PredictDetailsPage.tapBackButton();
         await Assertions.expectTextDisplayed(positionDetails.newBalance, {
           description: `USDC balance should display ${positionDetails.newBalance} after opening position`,