fix: Metrics

src/SmartTransactionsController.test.ts

@@ -233,6 +233,9 @@
         minedTx: 'not_mined',
         minedHash: '',
       },
+      accountHardwareType: 'Ledger Hardware',
+      accountType: 'hardware',
+      deviceModel: 'ledger',
     },
   ];
 };
@@ -262,6 +265,9 @@
           '0x55ad39634ee10d417b6e190cfd3736098957e958879cffe78f1f00f4fd2654d6',
         minedTx: 'success',
       },
+      accountHardwareType: 'Ledger Hardware',
+      accountType: 'hardware',
+      deviceModel: 'ledger',
     },
   ];
 };
@@ -616,7 +622,21 @@
       smartTransactionsController.trackStxStatusChange(
         smartTransaction as SmartTransaction,
       );
-      expect(trackMetaMetricsEventSpy).toHaveBeenCalled();
+      expect(trackMetaMetricsEventSpy).toHaveBeenCalledWith(
+        expect.objectContaining({
+          event: 'STX Status Updated',
+          category: 'Transactions',
+          properties: expect.objectContaining({
+            stx_status: SmartTransactionStatuses.PENDING,
+            is_smart_transaction: true,
+          }),
+          sensitiveProperties: expect.objectContaining({
+            account_hardware_type: 'Ledger Hardware',
+            account_type: 'hardware',
+            device_model: 'ledger',
+          }),
+        }),
+      );
     });
 
     it('does not track if smartTransaction and prevSmartTransaction have the same status', () => {
@@ -630,15 +650,32 @@
 
     it('tracks status change if smartTransaction and prevSmartTransaction have different statuses', () => {
       const smartTransaction = {
-        ...createStateAfterPending()[0],
+        ...createStateAfterSuccess()[0],
         swapMetaData: {},
       };
-      const prevSmartTransaction = { ...smartTransaction, status: '' };
+      const prevSmartTransaction = {
+        ...smartTransaction,
+        status: SmartTransactionStatuses.PENDING,
+      };
       smartTransactionsController.trackStxStatusChange(
         smartTransaction as SmartTransaction,
         prevSmartTransaction as SmartTransaction,
       );
-      expect(trackMetaMetricsEventSpy).toHaveBeenCalled();
+      expect(trackMetaMetricsEventSpy).toHaveBeenCalledWith(
+        expect.objectContaining({
+          event: 'STX Status Updated',
+          category: 'Transactions',
+          properties: expect.objectContaining({
+            stx_status: SmartTransactionStatuses.SUCCESS,
+            is_smart_transaction: true,
+          }),
+          sensitiveProperties: expect.objectContaining({
+            account_hardware_type: 'Ledger Hardware',
+            account_type: 'hardware',
+            device_model: 'ledger',
+          }),
+        }),
+      );
     });
   });
 
@@ -1223,13 +1260,13 @@
 
   describe('setStatusRefreshInterval', () => {
     it('sets refresh interval if different', () => {
       smartTransactionsController.setStatusRefreshInterval(100);
       expect(smartTransactionsController.config.interval).toBe(100);
     });
 
     it('does not set refresh interval if they are the same', () => {
       const configureSpy = jest.spyOn(smartTransactionsController, 'configure');
       smartTransactionsController.setStatusRefreshInterval(DEFAULT_INTERVAL);
       expect(configureSpy).toHaveBeenCalledTimes(0);
     });
   });

src/SmartTransactionsController.ts

@@ -227,9 +227,9 @@
       isSmartTransactionPending,
     );
     if (!this.timeoutHandle && pendingTransactions?.length > 0) {
       this.poll();
     } else if (this.timeoutHandle && pendingTransactions?.length === 0) {
       this.stop();
     }
   }
 
@@ -287,7 +287,7 @@
       return;
     }
     this.timeoutHandle = setInterval(() => {
       safelyExecute(async () => this.updateSmartTransactions());
     }, this.config.interval);
     await safelyExecute(async () => this.updateSmartTransactions());
   }
@@ -323,9 +323,10 @@
     this.trackMetaMetricsEvent({
       event: MetaMetricsEventName.StxStatusUpdated,
       category: MetaMetricsEventCategory.Transactions,
-      properties: getSmartTransactionMetricsProperties(smartTransaction),
-      sensitiveProperties:
-        getSmartTransactionMetricsSensitiveProperties(smartTransaction),
+      properties: getSmartTransactionMetricsProperties(updatedSmartTransaction),
+      sensitiveProperties: getSmartTransactionMetricsSensitiveProperties(
+        updatedSmartTransaction,
+      ),
     });
   }
 
@@ -354,7 +355,7 @@
       ethQuery = new EthQuery(networkClient.provider);
     }
 
     this.#createOrUpdateSmartTransaction(smartTransaction, {
       chainId,
       ethQuery,
     });
@@ -427,6 +428,10 @@
       throw new Error(ETH_QUERY_ERROR_MSG);
     }
 
+    if (isNewSmartTransaction) {
+      await this.#addMetaMetricsPropsToNewSmartTransaction(smartTransaction);
+    }
+
     this.trackStxStatusChange(
       smartTransaction,
       isNewSmartTransaction
@@ -435,7 +440,6 @@
     );
 
     if (isNewSmartTransaction) {
-      await this.#addMetaMetricsPropsToNewSmartTransaction(smartTransaction);
       // add smart transaction
       const cancelledNonceIndex = currentSmartTransactions?.findIndex(
         (stx: SmartTransaction) =>
@@ -507,7 +511,7 @@
       .map((smartTransaction) => smartTransaction.uuid);
 
     if (transactionsToUpdate.length > 0) {
       this.fetchSmartTransactionsStatus(transactionsToUpdate, {
         networkClientId,
       });
     }
@@ -671,7 +675,7 @@
     nonceLock.releaseLock();
     return {
       ...transaction,
       nonce: `0x${nonce.toString(16)}`,
     };
   }