fix(sdk-api): type errors in old code

They are correctly detected with Typescript 5

Issue: BTC-1450

Author: OttoAllmendinger

modules/sdk-api/src/v1/wallet.ts

@@ -2173,7 +2173,7 @@ Wallet.prototype.consolidateUnspents = function (params, callback) {
     return consolidationTransactions;
   });
 
-  return runNextConsolidation(this, target)
+  return runNextConsolidation()
     .catch(function (err) {
       if (err.message === 'Done') {
         return;

modules/sdk-api/test/unit/v1/wallet.ts

@@ -23,8 +23,7 @@ const TestBitGo = {
   TEST_WALLET1_PASSCODE: 'iVWeATjqLS1jJShrPpETti0b',
 };
 const originalFetchConstants = BitGoAPI.prototype.fetchConstants;
-BitGoAPI.prototype.fetchConstants = function () {
-  // @ts-expect-error - no implicit this
+BitGoAPI.prototype.fetchConstants = function (this: any) {
   nock(this._baseUrl).get('/api/v1/client/constants').reply(200, { ttl: 3600, constants: {} });
 
   // force client constants reload

modules/sdk-test/src/bitgo/lib/test_bitgo.ts

@@ -582,8 +582,7 @@ export class TestBitGo {
       }
     };
 
-    BitGoAPI.prototype.fetchConstants = function () {
-      // @ts-expect-error - no implicit this
+    BitGoAPI.prototype.fetchConstants = function (this: any) {
       nock(this._baseUrl)
         .get('/api/v1/client/constants')
         .reply(200, {