[release] src/goTest: avoid a name conflicting with the proposed quickpick api

src/goTest used an extended quickpick type that adds a custom property
'kind' to specify the profile type. Unfortunately, that conflicts with
the proposed API added during vscode 1.63 dev cycle and breaks the
extension's profile feature.

Rename ours to 'profilekind'.

And reenabled the profile feature tests disabled due to 1858.
(but these tests don't exercise the affected code path - quickpick
UI interaction)

Fixes #2048

Updates #1858

Change-Id: Iedd1c5f32cdbc8438e203f8989f28bfcaf1c04b4
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/382334
Trust: Hyang-Ah Hana Kim <[email protected]>
Reviewed-by: Ethan Reesor <[email protected]>
Reviewed-by: Peter Weinberger <[email protected]>
(cherry picked from commit 037e8a3)
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/383219
Reviewed-by: Robert Findley <[email protected]>
Run-TryBot: Hyang-Ah Hana Kim <[email protected]>
TryBot-Result: kokoro <[email protected]>

Author: hyangah

src/goTest/profile.ts

@@ -76,16 +76,16 @@ export class GoTestProfiler {
 	}
 
 	async configure(): Promise<ProfilingOptions | undefined> {
-		const { kind } = await vscode.window.showQuickPick(
-			Kind.all.map((x) => ({ label: x.label, kind: x })),
+		const { profilekind } = await vscode.window.showQuickPick(
+			Kind.all.map((x) => ({ label: x.label, profilekind: x })),
 			{
 				title: 'Profile'
 			}
 		);
-		if (!kind) return;
+		if (!profilekind) return;
 
 		return {
-			kind: kind.id
+			kind: profilekind.id
 		};
 	}
 

test/integration/goTest.run.test.ts

@@ -13,7 +13,7 @@ suite('Go Test Runner', () => {
 
 	let testExplorer: GoTestExplorer;
 
-	suite.skip('Profile', () => {
+	suite('Profile', () => {
 		const sandbox = sinon.createSandbox();
 		const ctx = MockExtensionContext.new();