test/integration: skip tests broken on windows

Added affectedByIssue832 to clearly indicate the tests
that need to be fixed.

For #832
For #1788

Change-Id: Ied7dc9726b33f092ae93a1f11952f97fd1433337
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/418356
Reviewed-by: Jamal Carvalho <[email protected]>
TryBot-Result: kokoro <[email protected]>
Run-TryBot: Hyang-Ah Hana Kim <[email protected]>

Author: hyangah

test/integration/extension.test.ts

@@ -46,6 +46,7 @@ import {
 import cp = require('child_process');
 import os = require('os');
 import { MockExtensionContext } from '../mocks/MockContext';
+import { affectedByIssue832 } from './testutils';
 
 const testAll = (isModuleMode: boolean) => {
 	const dummyCancellationSource = new vscode.CancellationTokenSource();
@@ -741,7 +742,10 @@ It returns the number of bytes written and any write error encountered.
 		assert.equal(interfaces[0].name, 'circle');
 	});
 
-	test('Test listPackages', async () => {
+	test('Test listPackages', async function () {
+		if (affectedByIssue832()) {
+			this.skip(); // timeout on windows
+		}
 		const uri = vscode.Uri.file(path.join(fixturePath, 'baseTest', 'test.go'));
 		const document = await vscode.workspace.openTextDocument(uri);
 		await vscode.window.showTextDocument(document);
@@ -793,7 +797,10 @@ It returns the number of bytes written and any write error encountered.
 		});
 	});
 
-	test('Workspace Symbols', () => {
+	test('Workspace Symbols', function () {
+		if (affectedByIssue832()) {
+			this.skip(); // frequent timeout on windows
+		}
 		const workspacePath = path.join(fixturePath, 'vendoring');
 		const configWithoutIgnoringFolders = Object.create(getGoConfig(), {
 			gotoSymbol: {
@@ -861,7 +868,11 @@ It returns the number of bytes written and any write error encountered.
 		return Promise.all([withIgnoringFolders, withoutIgnoringFolders, withIncludingGoroot, withoutIncludingGoroot]);
 	});
 
-	test('Test Completion', async () => {
+	test('Test Completion', async function () {
+		if (affectedByIssue832()) {
+			this.skip(); // timeout on windows
+		}
+
 		const printlnDoc = `Println formats using the default formats for its operands and writes to
 standard output. Spaces are always added between operands and a newline is
 appended. It returns the number of bytes written and any write error
@@ -911,7 +922,10 @@ encountered.
 		await Promise.all(promises);
 	});
 
-	test('Test Completion Snippets For Functions', async () => {
+	test('Test Completion Snippets For Functions', async function () {
+		if (affectedByIssue832()) {
+			this.skip(); // timeout on windows
+		}
 		const provider = new GoCompletionItemProvider();
 		const uri = vscode.Uri.file(path.join(fixturePath, 'completions', 'snippets.go'));
 		const baseConfig = getGoConfig();
@@ -1176,7 +1190,7 @@ encountered.
 	});
 
 	test('Test Completion on unimported packages', async function () {
-		if (isModuleMode) {
+		if (isModuleMode || affectedByIssue832()) {
 			this.skip();
 		}
 		// gocode-gomod does not handle unimported package completion.
@@ -1212,7 +1226,10 @@ encountered.
 		await Promise.all(promises);
 	});
 
-	test('Test Completion on unimported packages (multiple)', async () => {
+	test('Test Completion on unimported packages (multiple)', async function () {
+		if (affectedByIssue832()) {
+			this.skip();
+		}
 		const config = Object.create(getGoConfig(), {
 			gocodeFlags: { value: ['-builtin'] }
 		});

test/integration/goDebug.test.ts

@@ -32,6 +32,7 @@ import { killProcessTree, killProcess } from '../../src/utils/processUtils';
 import getPort = require('get-port');
 import util = require('util');
 import { TimestampedLogger } from '../../src/goLogging';
+import { affectedByIssue832 } from './testutils';
 
 // For debugging test and streaming the trace instead of buffering, set this.
 const PRINT_TO_CONSOLE = false;
@@ -1728,7 +1729,7 @@ const testAll = (ctx: Mocha.Context, isDlvDap: boolean, withConsole?: string) =>
 		// has a chance to clean up.
 		// BUG(https://github.com/golang/vscode-go/issues/1993)
 		test('should cleanup when stopped', async function () {
-			if (!isDlvDap) {
+			if (!isDlvDap || affectedByIssue832()) {
 				this.skip();
 			}
 			const PROGRAM = path.join(DATA_ROOT, 'loop');
@@ -1990,6 +1991,9 @@ const testAll = (ctx: Mocha.Context, isDlvDap: boolean, withConsole?: string) =>
 	});
 
 	suite('substitute path', () => {
+		if (affectedByIssue832()) {
+			return;
+		}
 		// TODO(suzmue): add unit tests for substitutePath.
 		let tmpDir: string;
 

test/integration/goDebugConfiguration.test.ts

@@ -14,6 +14,7 @@ import { rmdirRecursive } from '../../src/util';
 import goEnv = require('../../src/goEnv');
 import { MockCfg } from '../mocks/MockCfg';
 import { extensionId } from '../../src/const';
+import { affectedByIssue832 } from './testutils';
 
 suite('Debug Environment Variable Merge Test', () => {
 	const debugConfigProvider = new GoDebugConfigurationProvider();
@@ -630,6 +631,9 @@ suite('Debug Configuration With Invalid Program', () => {
 });
 
 suite('Debug Configuration Converts Relative Paths', () => {
+	if (affectedByIssue832()) {
+		return;
+	}
 	const debugConfigProvider = new GoDebugConfigurationProvider();
 
 	let workspaceDir = '';

test/integration/goExplorer.test.ts

@@ -13,6 +13,7 @@ import { getConfiguredTools } from '../../src/goTools';
 import { getGoVersion } from '../../src/util';
 import { resolveHomeDir } from '../../src/utils/pathUtils';
 import { MockExtensionContext } from '../mocks/MockContext';
+import { affectedByIssue832 } from './testutils';
 
 suite('GoExplorerProvider', () => {
 	const fixtureDir = path.join(__dirname, '../../../test/testdata/baseTest');
@@ -36,7 +37,10 @@ suite('GoExplorerProvider', () => {
 		assert.strictEqual(env.contextValue, 'go:explorer:envtree');
 	});
 
-	test('env tree items', async () => {
+	test('env tree items', async function () {
+		if (affectedByIssue832()) {
+			this.skip();
+		}
 		const [env] = await explorer.getChildren()!;
 		const [goenv, gomod] = (await explorer.getChildren(env)) as { key: string; value: string }[];
 		assert.strictEqual(goenv.key, 'GOENV');

test/integration/goTest.explore.test.ts

@@ -15,6 +15,7 @@ import * as config from '../../src/config';
 import { GoTestResolver } from '../../src/goTest/resolve';
 import * as testUtils from '../../src/testUtils';
 import { GoTest } from '../../src/goTest/utils';
+import { affectedByIssue832 } from './testutils';
 
 type Files = Record<string, string | { contents: string; language: string }>;
 
@@ -56,6 +57,9 @@ async function forceResolve(resolver: GoTestResolver, item?: TestItem) {
 
 suite('Go Test Explorer', () => {
 	suite('Document opened', () => {
+		if (affectedByIssue832()) {
+			return;
+		}
 		class DUT extends GoTestExplorer {
 			async _didOpen(doc: TextDocument) {
 				await this.didOpenTextDocument(doc);
@@ -113,6 +117,9 @@ suite('Go Test Explorer', () => {
 	});
 
 	suite('Document edited', async () => {
+		if (affectedByIssue832()) {
+			return;
+		}
 		class DUT extends GoTestExplorer {
 			async _didOpen(doc: TextDocument) {
 				await this.didOpenTextDocument(doc);
@@ -224,6 +231,9 @@ suite('Go Test Explorer', () => {
 	});
 
 	suite('settings', () => {
+		if (affectedByIssue832()) {
+			return;
+		}
 		const sandbox = sinon.createSandbox();
 
 		// eslint-disable-next-line @typescript-eslint/no-explicit-any

test/integration/goTest.resolve.test.ts

@@ -8,6 +8,7 @@ import { GoTestResolver } from '../../src/goTest/resolve';
 import { GoTest, GoTestKind } from '../../src/goTest/utils';
 import { MockTestController, MockTestWorkspace } from '../mocks/MockTest';
 import { getSymbols_Regex, populateModulePathCache } from './goTest.utils';
+import { affectedByIssue832 } from './testutils';
 
 type Files = Record<string, string | { contents: string; language: string }>;
 
@@ -25,6 +26,9 @@ function setup(folders: string[], files: Files) {
 }
 
 suite('Go Test Resolver', () => {
+	if (affectedByIssue832()) {
+		return;
+	}
 	interface TC extends TestCase {
 		item?: ([string, string, GoTestKind] | [string, string, GoTestKind, string])[];
 		expect: string[];

test/integration/testutils.ts

@@ -0,0 +1,3 @@
+export function affectedByIssue832(): boolean {
+	return process.platform === 'win32';
+}