fix: cp-7.54.0 quotes on Optimism crashing app (MetaMask#18486)

<!--
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**

<!--
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?
-->

This PR fixes an issue where the app would crash when
`quote.gasFee.effective.amount` was in scientific notation. You could
observe this on networks such as Optimism.

## **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: null

## **Related issues**

Fixes: MetaMask#18454

## **Manual testing steps**

```gherkin
Feature: Getting quotes on Optimism

  Scenario: user requests a quote on Optimism
    Given user has requested a quote

    When user requests a quote
    Then they should receive a quote and app should not crash
```

## **Screenshots/Recordings**

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

### **Before**

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

### **After**

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



https://github.com/user-attachments/assets/ca420125-5d07-4b10-9283-bedc5aff585f



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

Author: infiniteflower

app/components/UI/Bridge/hooks/useHasSufficientGas/index.ts

@@ -8,6 +8,7 @@ import { ethers } from 'ethers';
 import { CaipChainId, Hex } from '@metamask/utils';
 import { useBridgeQuoteData } from '../useBridgeQuoteData';
 import { getNativeSourceToken } from '../useInitialSourceToken';
+import { BigNumber } from 'bignumber.js';
 
 interface Props {
   quote: ReturnType<typeof useBridgeQuoteData>['activeQuote'];
@@ -40,10 +41,15 @@ export const useHasSufficientGas = ({ quote }: Props): boolean | null => {
     decimals: sourceChainNativeAsset?.decimals,
   });
 
+  // quote.gasFee.effective.amount might be in scientific notation (e.g. 9.200359292e-8), so we need to handle that
+  const effectiveGasFee = quote?.gasFee.effective
+    ? new BigNumber(quote.gasFee.effective.amount).toFixed()
+    : null;
+
   const atomicGasFee =
-    quote?.gasFee.effective && !gasIncluded
+    effectiveGasFee && !gasIncluded
       ? ethers.utils.parseUnits(
-          quote.gasFee.effective.amount,
+          effectiveGasFee,
           sourceChainNativeAsset?.decimals,
         )
       : null;

app/components/UI/Bridge/hooks/useHasSufficientGas/useHasSufficientGas.test.ts

@@ -91,6 +91,33 @@ describe('useHasSufficientGas', () => {
         expect(result.current).toBe(false);
       });
 
+      it('should handle scientific notation in effective gas fee', () => {
+        const mockQuote: ReturnType<typeof useBridgeQuoteData>['activeQuote'] =
+          {
+            quote: {
+              gasIncluded: false,
+              srcChainId: '0x1',
+            },
+            gasFee: {
+              effective: { amount: '9.200359292e-8' }, // Scientific notation
+            },
+          } as unknown as ReturnType<typeof useBridgeQuoteData>['activeQuote'];
+
+        // User has 0.001 ETH (more than enough for the tiny gas fee)
+        mockUseLatestBalance.mockReturnValue({
+          displayBalance: '0.001',
+          atomicBalance: BigNumber.from('1000000000000000'), // 0.001 ETH in wei
+        });
+
+        const { result } = renderHookWithProvider(
+          () => useHasSufficientGas({ quote: mockQuote }),
+          { state: {} },
+        );
+
+        // Should return true since 0.001 ETH > 0.00000009200359292 ETH
+        expect(result.current).toBe(true);
+      });
+
       it('should return null when gas token balance is not available', () => {
         const mockQuote: ReturnType<typeof useBridgeQuoteData>['activeQuote'] =
           {