From 0c8d002edc6aaa46a6973267dc8e6facfd603fdb Mon Sep 17 00:00:00 2001
From: Vinicius Stevam <vinicius.stevam@consensys.io>
Date: Wed, 18 Dec 2024 09:26:15 +0000
Subject: [PATCH] fix: gasFeeEstimates property undefined

---
 .../hooks/useTransactionFunction.test.js             | 12 ++++++++++++
 .../confirmations/hooks/useTransactionFunctions.js   |  5 +++++
 2 files changed, 17 insertions(+)

diff --git a/ui/pages/confirmations/hooks/useTransactionFunction.test.js b/ui/pages/confirmations/hooks/useTransactionFunction.test.js
index 22984ba71a37..ec823d2acf70 100644
--- a/ui/pages/confirmations/hooks/useTransactionFunction.test.js
+++ b/ui/pages/confirmations/hooks/useTransactionFunction.test.js
@@ -164,4 +164,16 @@ describe('useMaxPriorityFeePerGasInput', () => {
       userFeeLevel: 'dappSuggested',
     });
   });
+
+  it('returns early when gasFeeEstimates is undefined', () => {
+    const mockUpdateTransaction = jest
+      .spyOn(Actions, 'updateTransactionGasFees')
+      .mockImplementation(() => ({ type: '' }));
+
+    const { result } = renderUseTransactionFunctions({
+      gasFeeEstimates: undefined,
+    });
+    result.current.updateTransactionUsingEstimate(GasRecommendations.low);
+    expect(mockUpdateTransaction).not.toHaveBeenCalled();
+  });
 });
diff --git a/ui/pages/confirmations/hooks/useTransactionFunctions.js b/ui/pages/confirmations/hooks/useTransactionFunctions.js
index 17b5165da94e..437beb70f0cd 100644
--- a/ui/pages/confirmations/hooks/useTransactionFunctions.js
+++ b/ui/pages/confirmations/hooks/useTransactionFunctions.js
@@ -201,9 +201,14 @@ export const useTransactionFunctions = ({
 
   const updateTransactionUsingEstimate = useCallback(
     (gasFeeEstimateToUse) => {
+      if (!gasFeeEstimates) {
+        return;
+      }
+
       if (!gasFeeEstimates[gasFeeEstimateToUse]) {
         return;
       }
+
       const { suggestedMaxFeePerGas, suggestedMaxPriorityFeePerGas } =
         gasFeeEstimates[gasFeeEstimateToUse];
       updateTransaction({