release: 7.62.2 (#25436)

# 🚀 v7.62.2 Testing & Release Quality Process

Hi Team,  
As part of our new **MetaMask Release Quality Process**, here’s a quick
overview of the key processes, testing strategies, and milestones to
ensure a smooth and high-quality deployment.

---

## 📋 Key Processes

### Testing Strategy
- **Developer Teams:**  
Conduct regression and exploratory testing for your functional areas,
including automated and manual tests for critical workflows.
- **QA Team:**  
Focus on exploratory testing across the wallet, prioritize high-impact
areas, and triage any Sentry errors found during testing.
- **Customer Success Team:**  
Validate new functionalities and provide feedback to support release
monitoring.

### GitHub Signoff
- Each team must **sign off on the Release Candidate (RC)** via GitHub
by the end of the validation timeline (**Tuesday EOD PT**).
- Ensure all tests outlined in the Testing Plan are executed, and any
identified issues are addressed.

### Issue Resolution
- **Resolve all Release Blockers** (Sev0 and Sev1) by **Tuesday EOD
PT**.
- For unresolved blockers, PRs may be reverted, or feature flags
disabled to maintain release quality and timelines.

### Cherry-Picking Criteria
- Only **critical fixes** meeting outlined criteria will be
cherry-picked.
- Developers must ensure these fixes are thoroughly reviewed, tested,
and merged by **Tuesday EOD PT**.

---

## 🗓️ Timeline and Milestones

1. **Today (Friday):** Begin Release Candidate validation.  
2. **Tuesday EOD PT:** Finalize RC with all fixes and cherry-picks.  
3. **Wednesday:** Buffer day for final checks.  
4. **Thursday:** Submit release to app stores and begin rollout to 1% of
users.
5. **Monday:** Scale deployment to 10%.  
6. **Tuesday:** Full rollout to 100%.

---

## ✅ Signoff Checklist

Each team is responsible for signing off via GitHub. Use the checkbox
below to track signoff completion:

# Team sign-off checklist
- [ ] Mobile Platform

This process is a major step forward in ensuring release stability and
quality. Let’s stay aligned and make this release a success! 🚀

Feel free to reach out if you have questions or need clarification. 

Many thanks in advance

# Reference
- Testing plan sheet -
https://docs.google.com/spreadsheets/d/1tsoodlAlyvEUpkkcNcbZ4PM9HuC9cEM80RZeoVv5OCQ/edit?gid=404070372#gid=404070372

Author: joaoloureirop

CHANGELOG.md

@@ -7,6 +7,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [7.62.2]
+
+### Fixed
+
+- fix: cherry-pick 429 rate limiting fix with coin naming convention (#25443)
+- fix(perps): potential rate limit on close positions (#25456)
+
 ## [7.62.1]
 
 ### Fixed
@@ -9910,7 +9917,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - [#957](https://github.com/MetaMask/metamask-mobile/pull/957): fix timeouts (#957)
 - [#954](https://github.com/MetaMask/metamask-mobile/pull/954): Bugfix: onboarding navigation (#954)
 
-[Unreleased]: https://github.com/MetaMask/metamask-mobile/compare/v7.62.1...HEAD
+[Unreleased]: https://github.com/MetaMask/metamask-mobile/compare/v7.62.2...HEAD
+[7.62.2]: https://github.com/MetaMask/metamask-mobile/compare/v7.62.1...v7.62.2
 [7.62.1]: https://github.com/MetaMask/metamask-mobile/compare/v7.62.0...v7.62.1
 [7.62.0]: https://github.com/MetaMask/metamask-mobile/compare/v7.61.6...v7.62.0
 [7.61.6]: https://github.com/MetaMask/metamask-mobile/compare/v7.61.5...v7.61.6

android/app/build.gradle

@@ -187,8 +187,8 @@ android {
         applicationId "io.metamask"
         minSdkVersion rootProject.ext.minSdkVersion
         targetSdkVersion rootProject.ext.targetSdkVersion
-        versionName "7.62.1"
-        versionCode 3561
+        versionName "7.62.2"
+        versionCode 3591
         testBuildType System.getProperty('testBuildType', 'debug')
         testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
         manifestPlaceholders.MM_BRANCH_KEY_TEST = "$System.env.MM_BRANCH_KEY_TEST"

app/components/UI/Perps/Perps.testIds.ts

@@ -472,7 +472,8 @@ export const PerpsTutorialSelectorsIDs = {
 export const PerpsStopLossPromptSelectorsIDs = {
   CONTAINER: 'perps-stop-loss-prompt-container',
   ADD_MARGIN_BUTTON: 'perps-stop-loss-prompt-add-margin-button',
-  TOGGLE: 'perps-stop-loss-prompt-toggle',
+  SET_STOP_LOSS_BUTTON: 'perps-stop-loss-prompt-set-button',
+  SUCCESS_ICON: 'perps-stop-loss-prompt-success-icon',
   LOADING: 'perps-stop-loss-prompt-loading',
 } as const;
 

app/components/UI/Perps/Views/PerpsMarketDetailsView/PerpsMarketDetailsView.test.tsx

@@ -252,21 +252,28 @@ jest.mock('../../hooks/usePerpsOpenOrders', () => ({
   usePerpsOpenOrders: () => mockUsePerpsOpenOrdersImpl(),
 }));
 
-const mockUsePerpsOrderFillsImpl = jest.fn<
+const mockUsePerpsLiveFillsImpl = jest.fn<
   ReturnType<
-    typeof import('../../hooks/usePerpsOrderFills').usePerpsOrderFills
+    typeof import('../../hooks/stream/usePerpsLiveFills').usePerpsLiveFills
   >,
   []
 >(() => ({
-  orderFills: [],
-  isLoading: false,
-  error: null,
-  refresh: jest.fn(),
-  isRefreshing: false,
+  fills: [],
+  isInitialLoading: false,
 }));
 
-jest.mock('../../hooks/usePerpsOrderFills', () => ({
-  usePerpsOrderFills: () => mockUsePerpsOrderFillsImpl(),
+jest.mock('../../hooks/stream/usePerpsLiveFills', () => ({
+  usePerpsLiveFills: () => mockUsePerpsLiveFillsImpl(),
+}));
+
+// Mock Engine for REST fallback tests
+const mockGetOrderFills = jest.fn();
+jest.mock('../../../../../core/Engine', () => ({
+  context: {
+    PerpsController: {
+      getOrderFills: (...args: unknown[]) => mockGetOrderFills(...args),
+    },
+  },
 }));
 
 // Mock for usePerpsMarkets that can be modified per test
@@ -568,6 +575,7 @@ describe('PerpsMarketDetailsView', () => {
       error: null,
       existingPosition: null,
       refreshPosition: jest.fn(),
+      positionOpenedTimestamp: undefined,
     });
 
     // Reset navigation mocks
@@ -600,12 +608,9 @@ describe('PerpsMarketDetailsView', () => {
     };
 
     // Reset order fills mock to default
-    mockUsePerpsOrderFillsImpl.mockReturnValue({
-      orderFills: [],
-      isLoading: false,
-      error: null,
-      refresh: jest.fn(),
-      isRefreshing: false,
+    mockUsePerpsLiveFillsImpl.mockReturnValue({
+      fills: [],
+      isInitialLoading: false,
     });
   });
 
@@ -834,6 +839,7 @@ describe('PerpsMarketDetailsView', () => {
           },
         },
         refreshPosition: jest.fn(),
+        positionOpenedTimestamp: undefined,
       });
 
       const { getByTestId, queryByText, queryByTestId } = renderWithProvider(
@@ -924,6 +930,7 @@ describe('PerpsMarketDetailsView', () => {
         error: null,
         existingPosition: null,
         refreshPosition: mockRefreshPosition, // No-op function for WebSocket positions
+        positionOpenedTimestamp: undefined,
       });
 
       const { getByTestId } = renderWithProvider(
@@ -975,6 +982,7 @@ describe('PerpsMarketDetailsView', () => {
           },
         },
         refreshPosition: mockRefreshPosition,
+        positionOpenedTimestamp: undefined,
       });
 
       const { getByTestId } = renderWithProvider(
@@ -1011,6 +1019,7 @@ describe('PerpsMarketDetailsView', () => {
         error: null,
         existingPosition: null,
         refreshPosition: mockRefreshPosition,
+        positionOpenedTimestamp: undefined,
       });
 
       const { getByTestId } = renderWithProvider(
@@ -1075,6 +1084,7 @@ describe('PerpsMarketDetailsView', () => {
         error: null,
         existingPosition: null,
         refreshPosition: mockRefreshPosition,
+        positionOpenedTimestamp: undefined,
       });
 
       const { getByTestId } = renderWithProvider(
@@ -1692,10 +1702,13 @@ describe('PerpsMarketDetailsView', () => {
     });
   });
 
-  describe('Position opened timestamp calculation', () => {
-    it('computes position opened timestamp from order fills data', () => {
-      // Arrange
-      const timestamp = Date.now();
+  describe('Position opened timestamp', () => {
+    // Note: The timestamp calculation logic has been moved to useHasExistingPosition hook
+    // These tests verify the component correctly uses the hook's positionOpenedTimestamp
+
+    it('uses positionOpenedTimestamp from useHasExistingPosition hook', () => {
+      // Arrange - Hook provides the timestamp
+      const timestamp = Date.now() - 5 * 60 * 1000;
       mockUseHasExistingPosition.mockReturnValue({
         hasPosition: true,
         isLoading: false,
@@ -1718,51 +1731,50 @@ describe('PerpsMarketDetailsView', () => {
           },
         },
         refreshPosition: jest.fn(),
+        positionOpenedTimestamp: timestamp, // Hook now provides this
       });
 
-      mockUsePerpsOrderFillsImpl.mockReturnValue({
-        orderFills: [
-          {
-            orderId: 'order-1',
-            symbol: 'BTC',
-            side: 'buy',
-            direction: 'Open Long',
-            timestamp: timestamp - 2000,
-            size: '0.3',
-            price: '43000',
-            pnl: '0',
-            fee: '0.001',
-            feeToken: 'USDC',
-          },
-          {
-            orderId: 'order-2',
-            symbol: 'BTC',
-            side: 'buy',
-            direction: 'Open Long',
-            timestamp,
-            size: '0.5',
-            price: '44000',
-            pnl: '0',
-            fee: '0.001',
-            feeToken: 'USDC',
-          },
-          {
-            orderId: 'order-3',
-            symbol: 'ETH',
-            side: 'sell',
-            direction: 'Open Short',
-            timestamp: timestamp - 1000,
-            size: '1.0',
-            price: '3000',
-            pnl: '0',
-            fee: '0.001',
-            feeToken: 'USDC',
-          },
-        ],
+      // Act
+      const { getByTestId } = renderWithProvider(
+        <PerpsConnectionProvider>
+          <PerpsMarketDetailsView />
+        </PerpsConnectionProvider>,
+        {
+          state: initialState,
+        },
+      );
+
+      // Assert - component renders with position data
+      expect(
+        getByTestId(PerpsMarketDetailsViewSelectorsIDs.CONTAINER),
+      ).toBeTruthy();
+    });
+
+    it('handles undefined positionOpenedTimestamp from hook', () => {
+      // Arrange - Hook returns undefined timestamp (e.g., new position)
+      mockUseHasExistingPosition.mockReturnValue({
+        hasPosition: true,
         isLoading: false,
         error: null,
-        refresh: jest.fn(),
-        isRefreshing: false,
+        existingPosition: {
+          symbol: 'BTC',
+          size: '0.5',
+          entryPrice: '44000',
+          positionValue: '22000',
+          unrealizedPnl: '50',
+          marginUsed: '500',
+          leverage: { type: 'isolated', value: 5 },
+          liquidationPrice: '40000',
+          maxLeverage: 20,
+          returnOnEquity: '1.14',
+          cumulativeFunding: {
+            allTime: '0',
+            sinceOpen: '0',
+            sinceChange: '0',
+          },
+        },
+        refreshPosition: jest.fn(),
+        positionOpenedTimestamp: undefined, // No timestamp available yet
       });
 
       // Act
@@ -1775,11 +1787,10 @@ describe('PerpsMarketDetailsView', () => {
         },
       );
 
-      // Assert
+      // Assert - component still renders correctly
       expect(
         getByTestId(PerpsMarketDetailsViewSelectorsIDs.CONTAINER),
       ).toBeTruthy();
-      expect(mockUsePerpsOrderFillsImpl).toHaveBeenCalled();
     });
   });