chore: add test for getAddTransactionRequest

MetaMask · Sep 12, 2024 · b896b4b · b896b4b
1 parent b881549
commit b896b4b
Showing 1 changed file with 70 additions and 0 deletions.
diff --git a/app/scripts/metamask-controller.test.js b/app/scripts/metamask-controller.test.js
@@ -523,6 +523,76 @@ describe('MetaMaskController', () => {
       });
     });
 
+    describe('#getAddTransactionRequest', () => {
+      it('formats the transaction for submission', () => {
+        const transactionParams = { from: '0xa', to: '0xb' };
+        const transactionOptions = { foo: true };
+        const result = metamaskController.getAddTransactionRequest({
+          transactionParams,
+          transactionOptions,
+        });
+        expect(result).toEqual(
+          expect.objectContaining({
+            internalAccounts:
+              metamaskController.accountsController.listAccounts(),
+            dappRequest: undefined,
+            networkClientId:
+              metamaskController.networkController.state
+                .selectedNetworkClientId,
+            selectedAccount:
+              metamaskController.accountsController.getAccountByAddress(
+                transactionParams.from,
+              ),
+            transactionController: expect.objectContaining({}),
+            transactionOptions,
+            transactionParams,
+            userOperationController: expect.objectContaining({}),
+            chainId: '0x1',
+            ppomController: expect.objectContaining({}),
+            securityAlertsEnabled:
+              metamaskController.preferencesController.store.getState()
+                ?.securityAlertsEnabled,
+            updateSecurityAlertResponse: expect.any,
+          }),
+        );
+      });
+      it('passes through any additional params to the object', () => {
+        const transactionParams = { from: '0xa', to: '0xb' };
+        const transactionOptions = { foo: true };
+        const result = metamaskController.getAddTransactionRequest({
+          transactionParams,
+          transactionOptions,
+          test: '123',
+        });
+
+        expect(result).toEqual(
+          expect.objectContaining({
+            internalAccounts:
+              metamaskController.accountsController.listAccounts(),
+            dappRequest: undefined,
+            networkClientId:
+              metamaskController.networkController.state
+                .selectedNetworkClientId,
+            selectedAccount:
+              metamaskController.accountsController.getAccountByAddress(
+                transactionParams.from,
+              ),
+            transactionController: expect.objectContaining({}),
+            transactionOptions,
+            transactionParams,
+            userOperationController: expect.objectContaining({}),
+            chainId: '0x1',
+            ppomController: expect.objectContaining({}),
+            securityAlertsEnabled:
+              metamaskController.preferencesController.store.getState()
+                ?.securityAlertsEnabled,
+            updateSecurityAlertResponse: expect.any,
+            test: '123',
+          }),
+        );
+      });
+    });
+
     describe('submitPassword', () => {
       it('removes any identities that do not correspond to known accounts.', async () => {
         const fakeAddress = '0xbad0';