src/goTools: explicitly specify staticcheck, gotests package paths

Remove disableModulesForWildcard. We will not use wildcard
import paths for tools installation.

Change-Id: Ibb91ef51df697d5320cb73462d501ef21f7ac9a0
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/300390
Trust: Hyang-Ah Hana Kim <hyangah@gmail.com>
Run-TryBot: Hyang-Ah Hana Kim <hyangah@gmail.com>
TryBot-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Suzy Mueller <suzmue@golang.org>

Author: hyangah

src/goInstallTools.ts

@@ -21,7 +21,6 @@ import { restartLanguageServer } from './goMain';
 import { addGoStatus, initGoStatusBar, outputChannel, removeGoStatus } from './goStatus';
 import {
 	containsTool,
-	disableModulesForWildcard,
 	getConfiguredTools,
 	getImportPath,
 	getImportPathWithVersion,
@@ -158,8 +157,7 @@ export async function installTools(
 
 	const toInstall: Promise<{ tool: Tool; reason: string }>[] = [];
 	for (const tool of missing) {
-		// Disable modules for tools which are installed with the "..." wildcard.
-		const modulesOffForTool = modulesOff || disableModulesForWildcard(tool, goVersion);
+		const modulesOffForTool = modulesOff;
 
 		const reason = installTool(tool, goVersion, envForTools, !modulesOffForTool);
 		toInstall.push(Promise.resolve({ tool, reason: await reason }));

src/goTools.ts

@@ -68,19 +68,6 @@ export function getImportPathWithVersion(tool: Tool, version: semver.SemVer, goV
 	}
 	return importPath;
 }
-/**
- * Returns boolean denoting if the import path for the given tool ends with `/...`
- * and if the version of Go supports installing wildcard paths in module mode.
- * @param tool  	Object of type `Tool` for the Go tool.
- * @param goVersion The current Go version.
- */
-export function disableModulesForWildcard(tool: Tool, goVersion: GoVersion): boolean {
-	const importPath = getImportPath(tool, goVersion);
-	const isWildcard = importPath.endsWith('...');
-
-	// Only Go >= 1.13 supports installing wildcards in module mode.
-	return isWildcard && goVersion.lt('1.13');
-}
 
 export function containsTool(tools: Tool[], tool: Tool): boolean {
 	return tools.indexOf(tool) > -1;
@@ -332,7 +319,7 @@ export const allToolsInformation: { [key: string]: Tool } = {
 	},
 	'gotests': {
 		name: 'gotests',
-		importPath: 'github.com/cweill/gotests/...',
+		importPath: 'github.com/cweill/gotests/gotests',
 		replacedByGopls: false,
 		isImportant: false,
 		description: 'Generate unit tests',
@@ -349,7 +336,7 @@ export const allToolsInformation: { [key: string]: Tool } = {
 	},
 	'staticcheck': {
 		name: 'staticcheck',
-		importPath: 'honnef.co/go/tools/...',
+		importPath: 'honnef.co/go/tools/cmd/staticcheck',
 		replacedByGopls: false,
 		isImportant: true,
 		description: 'Linter'