chore: make networkClientId mandatory field in assetsContractController

Author: sahar-fehri

eslint-warning-thresholds.json

@@ -21,7 +21,7 @@
     "jsdoc/check-tag-names": 5
   },
   "packages/assets-controllers/src/AssetsContractController.test.ts": {
-    "import-x/order": 3
+    "import-x/order": 2
   },
   "packages/assets-controllers/src/AssetsContractController.ts": {
     "jsdoc/check-tag-names": 2,

packages/assets-controllers/src/AccountTrackerController.test.ts

@@ -672,10 +672,10 @@ describe('AccountTrackerController', () => {
           mockedQuery
             .mockReturnValueOnce(Promise.resolve('0x10'))
             .mockReturnValueOnce(Promise.resolve('0x20'));
-          const result = await controller.syncBalanceWithAddresses([
-            ADDRESS_1,
-            ADDRESS_2,
-          ]);
+          const result = await controller.syncBalanceWithAddresses(
+            [ADDRESS_1, ADDRESS_2],
+            'mainnet',
+          );
           expect(result[ADDRESS_1].balance).toBe('0x10');
           expect(result[ADDRESS_2].balance).toBe('0x20');
         },
@@ -697,10 +697,10 @@ describe('AccountTrackerController', () => {
           mockedQuery
             .mockReturnValueOnce(Promise.resolve('0x10'))
             .mockReturnValueOnce(Promise.resolve('0x20'));
-          const result = await controller.syncBalanceWithAddresses([
-            ADDRESS_1,
-            ADDRESS_2,
-          ]);
+          const result = await controller.syncBalanceWithAddresses(
+            [ADDRESS_1, ADDRESS_2],
+            'mainnet',
+          );
           expect(result[ADDRESS_1].balance).toBe('0x10');
           expect(result[ADDRESS_2].balance).toBe('0x20');
           expect(result[ADDRESS_1].stakedBalance).toBe('0x1');

packages/assets-controllers/src/AccountTrackerController.ts

@@ -429,12 +429,12 @@ export class AccountTrackerController extends StaticIntervalPollingController<Ac
    * Sync accounts balances with some additional addresses.
    *
    * @param addresses - the additional addresses, may be hardware wallet addresses.
-   * @param networkClientId - Optional networkClientId to fetch a network client with.
+   * @param networkClientId - networkClientId to fetch a network client with.
    * @returns accounts - addresses with synced balance
    */
   async syncBalanceWithAddresses(
     addresses: string[],
-    networkClientId?: NetworkClientId,
+    networkClientId: NetworkClientId,
   ): Promise<
     Record<string, { balance: string; stakedBalance?: StakedBalance }>
   > {