fix(ramps): address Bugbot async handler; expand OrderDetails callback tests

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: amitabh94

app/components/UI/Ramp/Views/Checkout/Checkout.tsx

@@ -138,7 +138,7 @@ const Checkout = () => {
   ]);
 
   const handleNavigationStateChange = useCallback(
-    async (navState: WebViewNavigation) => {
+    (navState: WebViewNavigation) => {
       if (
         !hasCallbackFlow ||
         isRedirectionHandledRef.current ||

app/components/UI/Ramp/Views/OrderDetails/OrderDetails.test.tsx

@@ -189,6 +189,21 @@ describe('OrderDetails', () => {
     expect(mockRefreshOrder).toHaveBeenCalled();
   });
 
+  it('shows localized error when pending order refresh rejects with non-Error', async () => {
+    mockUseParams.mockReturnValue({ orderId: 'ord-pending' });
+    mockGetOrderById.mockReturnValue({
+      ...mockOrder,
+      status: RampsOrderStatus.Pending,
+    });
+    mockRefreshOrder.mockRejectedValue('not-an-error');
+
+    const { getByText } = render();
+
+    await waitFor(() => {
+      expect(getByText('ramps_order_details.error_message')).toBeOnTheScreen();
+    });
+  });
+
   it('tracks RAMPS_SCREEN_VIEWED when order is displayed', async () => {
     render();
     await waitFor(() => {
@@ -256,6 +271,126 @@ describe('OrderDetails', () => {
     });
   });
 
+  it('resets to build quote when callback returns no order', async () => {
+    mockUseParams.mockReturnValue({
+      callbackUrl: 'https://callback.example?x=1',
+      providerCode: 'moonpay',
+      walletAddress: '0x123',
+    });
+    mockGetOrderById.mockReturnValue(undefined);
+    mockGetOrderFromCallback.mockResolvedValue(null);
+
+    render();
+
+    await waitFor(() => {
+      expect(mockReset).toHaveBeenCalled();
+    });
+    const resetArg = mockReset.mock.calls[0][0] as {
+      routes: { name: string }[];
+    };
+    expect(resetArg.routes[0].name).toBe(Routes.RAMP.BUILD_QUOTE);
+  });
+
+  it('resets to build quote when callback order is in a bailed status', async () => {
+    mockUseParams.mockReturnValue({
+      callbackUrl: 'https://callback.example?x=1',
+      providerCode: 'moonpay',
+      walletAddress: '0x123',
+    });
+    mockGetOrderById.mockReturnValue(undefined);
+    mockGetOrderFromCallback.mockResolvedValue({
+      providerOrderId: 'ord-bail',
+      status: RampsOrderStatus.Precreated,
+      provider: { id: 'moonpay' },
+      walletAddress: '0x123',
+    });
+
+    render();
+
+    await waitFor(() => {
+      expect(mockReset).toHaveBeenCalled();
+    });
+    expect(mockAddOrder).not.toHaveBeenCalled();
+  });
+
+  it('uses route cryptocurrency for toast when callback order has no crypto symbol', async () => {
+    const { showV2OrderToast } = jest.requireMock(
+      '../../utils/v2OrderToast',
+    ) as { showV2OrderToast: jest.Mock };
+    const orderWithoutCryptoSymbol = {
+      providerOrderId: 'ord-cb-2',
+      status: RampsOrderStatus.Completed,
+      cryptoAmount: '1',
+      provider: { id: 'moonpay' },
+      walletAddress: '0x123',
+    };
+    mockUseParams.mockReturnValue({
+      callbackUrl: 'https://callback.example?x=1',
+      providerCode: 'moonpay',
+      walletAddress: '0x123',
+      cryptocurrency: 'SOL',
+    });
+    mockGetOrderById.mockReturnValue(undefined);
+    mockGetOrderFromCallback.mockResolvedValue(orderWithoutCryptoSymbol);
+
+    render();
+
+    await waitFor(() => {
+      expect(showV2OrderToast).toHaveBeenCalledWith(
+        expect.objectContaining({
+          orderId: 'ord-cb-2',
+          cryptocurrency: 'SOL',
+        }),
+      );
+    });
+  });
+
+  it('does not emit status metrics when callback order status matches prior order', async () => {
+    const { showV2OrderToast } = jest.requireMock(
+      '../../utils/v2OrderToast',
+    ) as { showV2OrderToast: jest.Mock };
+    const completedOrder = {
+      providerOrderId: 'ord-same',
+      status: RampsOrderStatus.Completed,
+      cryptoCurrency: { symbol: 'ETH' },
+      cryptoAmount: '0.1',
+      provider: { id: 'moonpay' },
+      walletAddress: '0x123',
+    };
+    mockUseParams.mockReturnValue({
+      callbackUrl: 'https://callback.example?x=1',
+      providerCode: 'moonpay',
+      walletAddress: '0x123',
+    });
+    mockGetOrderById.mockImplementation((id: string) =>
+      id === 'ord-same' ? completedOrder : undefined,
+    );
+    mockGetOrderFromCallback.mockResolvedValue(completedOrder);
+
+    render();
+
+    await waitFor(() => {
+      expect(showV2OrderToast).toHaveBeenCalled();
+    });
+    expect(mockHandleOrderStatusChangedForMetrics).not.toHaveBeenCalled();
+  });
+
+  it('shows localized error when callback fetch rejects with Error that has no message', async () => {
+    mockUseParams.mockReturnValue({
+      callbackUrl: 'https://callback.example?x=1',
+      providerCode: 'moonpay',
+      walletAddress: '0x123',
+    });
+    mockGetOrderById.mockReturnValue(undefined);
+    mockGetOrderFromCallback.mockRejectedValue(new Error(''));
+
+    const { getByText } = render();
+
+    await waitFor(() => {
+      expect(getByText('ramps_order_details.error_message')).toBeOnTheScreen();
+    });
+  });
+
   it('shows error state with retry when initial callback fetch fails', async () => {
     mockUseParams.mockReturnValue({
       callbackUrl: 'metamask://on-ramp/providers/paypal?orderId=abc',