refactor: First batch of lint fixes

Author: rekmarks

packages/wallet/src/Wallet.test.ts

@@ -4,7 +4,7 @@ import {
   DistributionType,
   EnvironmentType,
 } from '@metamask/remote-feature-flag-controller';
-import { cleanAll, enableNetConnect } from 'nock';
+import { enableNetConnect } from 'nock';
 
 import { importSecretRecoveryPhrase, sendTransaction } from './utilities';
 import { Wallet } from './Wallet';
@@ -51,7 +51,6 @@ describe('Wallet', () => {
 
   afterEach(async () => {
     await wallet?.destroy();
-    cleanAll();
     enableNetConnect();
     jest.useRealTimers();
   });
@@ -82,6 +81,7 @@ describe('Wallet', () => {
       { networkClientId: 'sepolia' },
     );
 
+    // Advance timers by an arbitrary value to trigger downstream timer logic.
     const hash = await jest.advanceTimersByTimeAsync(60_000).then(() => result);
 
     expect(hash).toStrictEqual(expect.any(String));
@@ -96,7 +96,7 @@ describe('Wallet', () => {
         }),
       }),
     );
-  }, 30_000);
+  }, 10_000);
 
   it('exposes state', async () => {
     wallet = await setupWallet();

packages/wallet/src/initialization/types.ts

@@ -46,10 +46,10 @@ export function bindMessengerAction<
   ) => ExtractActionResponse<MessengerActions<Msgr>, ActionType>;
 }
 
-// Method syntax provides bivariant parameter checking, which is needed to
-// collect heterogeneous InitializationConfiguration values in a single array.
 export type InitializationConfiguration<Instance, InstanceMessenger> = {
   name: string;
+  // This is a method as opposed to function property in order to collect
+  // heterogeneous InitializationConfiguration values in a single array.
   init(args: InitFunctionArguments<Instance, InstanceMessenger>): {
     instance: Instance;
   };