test: remove client wrapping for debugging

Author: Hweinstock

packages/core/src/shared/clients/clientWrapper.ts

@@ -7,11 +7,11 @@ import globals from '../extensionGlobals'
 import { AwsClient, AwsClientConstructor, AwsCommand, AwsCommandConstructor } from '../awsClientBuilderV3'
 import { PaginationConfiguration, Paginator } from '@aws-sdk/types'
 import { AsyncCollection, toCollection } from '../utilities/asyncCollection'
-import { hasKey, isDefined } from '../utilities/tsUtils'
-import { PerfLog } from '../logger/perfLogger'
-import { truncateProps } from '../utilities/textUtilities'
-import { getLogger } from '../logger/logger'
-import { ToolkitError } from '../errors'
+import { isDefined } from '../utilities/tsUtils'
+// import { PerfLog } from '../logger/perfLogger'
+// import { truncateProps } from '../utilities/textUtilities'
+// import { getLogger } from '../logger/logger'
+// import { ToolkitError } from '../errors'
 
 type SDKPaginator<C, CommandInput extends object, CommandOutput extends object> = (
     config: Omit<PaginationConfiguration, 'client'> & { client: C },
@@ -39,25 +39,26 @@ export abstract class ClientWrapper<C extends AwsClient> implements vscode.Dispo
         CommandOptions extends CommandInput,
         Command extends AwsCommand<CommandInput, CommandOutput>,
     >(command: AwsCommandConstructor<CommandInput, Command>, commandOptions: CommandOptions): Promise<CommandOutput> {
-        const action = 'API Request'
-        const perflog = new PerfLog(action)
-        return await this.getClient()
-            .send(new command(commandOptions))
-            .catch((e) => {
-                const errWithoutStack = { ...e, name: e.name, message: e.message }
-                delete errWithoutStack['stack']
-                const timecost = perflog.elapsed().toFixed(1)
-                getLogger().error(
-                    `${action} failed (time: %dms) \nparams: %O\nerror: %O`,
-                    timecost,
-                    truncateProps(commandOptions, 20, ['nextToken']),
-                    errWithoutStack
-                )
-                throw new ToolkitError(`${action}: ${errWithoutStack.message}`, {
-                    code: extractCode(errWithoutStack),
-                    cause: errWithoutStack,
-                })
-            })
+        // const action = 'API Request'
+        // const perflog = new PerfLog(action)
+        return await this.getClient().send(new command(commandOptions))
+        // return await this.getClient()
+        //     .send(new command(commandOptions))
+        //     .catch((e) => {
+        //         const errWithoutStack = { ...e, name: e.name, message: e.message }
+        //         delete errWithoutStack['stack']
+        //         const timecost = perflog.elapsed().toFixed(1)
+        //         getLogger().error(
+        //             `${action} failed (time: %dms) \nparams: %O\nerror: %O`,
+        //             timecost,
+        //             truncateProps(commandOptions, 20, ['nextToken']),
+        //             errWithoutStack
+        //         )
+        //         throw new ToolkitError(`${action}: ${errWithoutStack.message}`, {
+        //             code: extractCode(errWithoutStack),
+        //             cause: errWithoutStack,
+        //         })
+        //     })
     }
 
     protected makePaginatedRequest<CommandInput extends object, CommandOutput extends object, Output extends object>(
@@ -88,10 +89,10 @@ export abstract class ClientWrapper<C extends AwsClient> implements vscode.Dispo
     }
 }
 
-function extractCode(e: Error): string {
-    return hasKey(e, 'code') && typeof e['code'] === 'string'
-        ? e.code
-        : hasKey(e, 'Code') && typeof e['Code'] === 'string'
-          ? e.Code
-          : e.name
-}
+// function extractCode(e: Error): string {
+//     return hasKey(e, 'code') && typeof e['code'] === 'string'
+//         ? e.code
+//         : hasKey(e, 'Code') && typeof e['Code'] === 'string'
+//           ? e.Code
+//           : e.name
+// }

packages/core/src/testE2E/codecatalyst/client.test.ts

@@ -88,7 +88,7 @@ let projectName: CodeCatalystProject['name']
  *     integ tests, but using the ssh hostname that we get from
  *     {@link prepareDevEnvConnection}.
  */
-describe('Test how this codebase uses the CodeCatalyst API', function () {
+describe.only('Test how this codebase uses the CodeCatalyst API', function () {
     let client: CodeCatalystClient
     let commands: CodeCatalystCommands
     let webviewClient: CodeCatalystCreateWebview