feat: MUSD-128 conditionally render bridge-time-row based on transaction type (MetaMask#23552)

<!--
Please submit this PR as a draft initially.
Do not mark it as "Ready for review" until the template has been
completely filled out, and PR status checks have passed at least once.
-->

## **Description**
Add conditional rendering of `bridge-time-row` based on transaction
type. For mUSD conversion we want the time estimate hidden.

<!--
Write a short description of the changes included in this pull request,
also include relevant motivation and context. Have in mind the following
questions:
1. What is the reason for the change?
2. What is the improvement/solution?
-->

## **Changelog**

<!--
If this PR is not End-User-Facing and should not show up in the
CHANGELOG, you can choose to either:
1. Write `CHANGELOG entry: null`
2. Label with `no-changelog`

If this PR is End-User-Facing, please write a short User-Facing
description in the past tense like:
`CHANGELOG entry: Added a new tab for users to see their NFTs`
`CHANGELOG entry: Fixed a bug that was causing some NFTs to flicker`

(This helps the Release Engineer do their job more quickly and
accurately)
-->

CHANGELOG entry: add conditional rendering of bridge-time-row by tx type

## **Related issues**

Fixes: [MUSD-128: Hide "estimated time" from quote
screen](https://consensyssoftware.atlassian.net/browse/MUSD-128)

## **Manual testing steps**

```gherkin
Feature: Bridge Time Row Visibility

  Scenario: user views confirmation for mUSD conversion transaction
    Given the user is on the confirmation screen for a mUSD conversion transaction
    When the user views the transaction details
    Then the bridge estimated time row is not displayed

  Scenario: user views loading state for mUSD conversion transaction
    Given the user is on the confirmation screen for a mUSD conversion transaction
    And the bridge quotes are loading
    When the user views the transaction details
    Then the bridge time skeleton loader is not displayed
```

## **Screenshots/Recordings**

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

### **Before**

<!-- [screenshots/recordings] -->
Time estimate is displayed


https://github.com/user-attachments/assets/f910fd34-4f1f-4ca0-b831-159d3c8fa04a

### **After**

<!-- [screenshots/recordings] -->
Time estimate is hidden


https://github.com/user-attachments/assets/3a6d593e-98e2-467f-996c-91421e39996c

## **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.

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> Conditionally hides the bridge estimated time row and skeleton for
`musdConversion` transactions.
> 
> - **Confirmations UI**:
> - Add conditional rendering in `bridge-time-row.tsx` to hide time
estimate and skeleton when transaction type matches
`TransactionType.musdConversion` via `hasTransactionType` and
`HIDE_TYPES`.
> - Preserve existing behavior (show skeleton when loading, show
estimate when quotes exist) for other transaction types; keep same-chain
display as `< 10 sec`.
> - **Tests**:
> - Update `bridge-time-row.test.tsx` to support injecting transaction
`type` and add cases asserting no skeleton/estimate for
`musdConversion`.
> - Retain and verify existing duration formatting and same-chain logic.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
ca077f9. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Author: Matt561

app/components/Views/confirmations/components/rows/bridge-time-row/bridge-time-row.test.tsx

@@ -13,17 +13,27 @@ import {
   TransactionPayQuote,
   TransactionPayTotals,
 } from '@metamask/transaction-pay-controller';
+import { TransactionType } from '@metamask/transaction-controller';
 import { Hex, Json } from '@metamask/utils';
 import { useTransactionPayToken } from '../../../hooks/pay/useTransactionPayToken';
 
 jest.mock('../../../hooks/pay/useTransactionPayData');
 jest.mock('../../../hooks/pay/useTransactionPayToken');
 
-function render() {
+function render(options: { type?: TransactionType } = {}) {
   const state = merge(
     {},
     simpleSendTransactionControllerMock,
     transactionApprovalControllerMock,
+    options.type && {
+      engine: {
+        backgroundState: {
+          TransactionController: {
+            transactions: [{ type: options.type }],
+          },
+        },
+      },
+    },
   );
 
   return renderWithProvider(<BridgeTimeRow />, { state });
@@ -76,7 +86,6 @@ describe('BridgeTimeRow', () => {
     useTransactionPayTotalsMock.mockReturnValue({
       estimatedDuration: 120,
     } as TransactionPayTotals);
-
     useTransactionPayTokenMock.mockReturnValue({
       payToken: { chainId: '0x1' as Hex },
     } as ReturnType<typeof useTransactionPayToken>);
@@ -93,4 +102,22 @@ describe('BridgeTimeRow', () => {
 
     expect(getByTestId(`bridge-time-row-skeleton`)).toBeDefined();
   });
+
+  it('does not render skeleton when transaction type is in HIDE_TYPES', () => {
+    useIsTransactionPayLoadingMock.mockReturnValue(true);
+
+    const { queryByTestId } = render({ type: TransactionType.musdConversion });
+
+    expect(queryByTestId('bridge-time-row-skeleton')).toBeNull();
+  });
+
+  it('does not render when transaction type is in HIDE_TYPES', () => {
+    useTransactionPayTotalsMock.mockReturnValue({
+      estimatedDuration: 60,
+    } as TransactionPayTotals);
+
+    const { queryByText } = render({ type: TransactionType.musdConversion });
+
+    expect(queryByText('1 min')).toBeNull();
+  });
 });

app/components/Views/confirmations/components/rows/bridge-time-row/bridge-time-row.tsx

@@ -13,17 +13,24 @@ import {
 import { useTransactionPayToken } from '../../../hooks/pay/useTransactionPayToken';
 import { useTransactionMetadataRequest } from '../../../hooks/transactions/useTransactionMetadataRequest';
 import { InfoRowSkeleton, InfoRowVariant } from '../../UI/info-row/info-row';
+import { TransactionType } from '@metamask/transaction-controller';
+import { hasTransactionType } from '../../../utils/transaction';
 
 const SAME_CHAIN_DURATION_SECONDS = '< 10';
 
+const HIDE_TYPES = [TransactionType.musdConversion];
+
 export function BridgeTimeRow() {
   const isLoading = useIsTransactionPayLoading();
   const { estimatedDuration } = useTransactionPayTotals() ?? {};
   const quotes = useTransactionPayQuotes();
   const { payToken } = useTransactionPayToken();
-  const { chainId } = useTransactionMetadataRequest() ?? {};
+  const transactionMetadata = useTransactionMetadataRequest();
+  const { chainId } = transactionMetadata ?? {};
 
-  const showEstimate = isLoading || Boolean(quotes?.length);
+  const showEstimate =
+    !hasTransactionType(transactionMetadata, HIDE_TYPES) &&
+    (isLoading || Boolean(quotes?.length));
 
   if (!showEstimate) {
     return null;