extension/src/goInstallTools: run updateImportantToolsStatus always

I noticed that the language bar status shows only "Analysis tools"
and nothing more when my environment is already setup and there is
no missing tool.

Even when all tools are installed, we need to run
updateImportantToolsStatus to set up the update command
and update the presentation. Add tests too.

Change-Id: I18a0a9b35f7651473fc7a7717303d5bd6f75901e
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/562400
Commit-Queue: Hyang-Ah Hana Kim <[email protected]>
TryBot-Result: kokoro <[email protected]>
Reviewed-by: Suzy Mueller <[email protected]>

Author: hyangah

extension/src/goInstallTools.ts

@@ -556,6 +556,9 @@ export async function maybeInstallImportantTools(
 			}
 			await installTools(missing, goVersion, { toolsManager: tm, skipRestartGopls: true });
 			// installTools will update ImportantToolsStatus.
+		} else {
+			// no essential tools to be installed.
+			await updateImportantToolsStatus(tm);
 		}
 	} catch (e) {
 		outputChannel.appendLine('install missing tools failed - ' + JSON.stringify(e));

extension/test/integration/install.test.ts

@@ -412,8 +412,14 @@ suite('maybeInstallImportantTools tests', () => {
 					: vscode.LanguageStatusSeverity.Information,
 				statusBarItem.text
 			);
-			for (const tool of wantMissingTools) {
-				assert(statusBarItem.detail!.includes(tool), statusBarItem.detail + ' does not contain ' + tool);
+			if (wantMissingTools && wantMissingTools.length > 0) {
+				assert.strictEqual(statusBarItem.command?.title, 'Install missing tools');
+				for (const tool of wantMissingTools) {
+					assert(statusBarItem.detail!.includes(tool), statusBarItem.detail + ' does not contain ' + tool);
+				}
+			} else {
+				assert.strictEqual(statusBarItem.command?.title, 'Update');
+				assert.strictEqual(statusBarItem.detail, 'no missing tools');
 			}
 		} catch (e) {
 			assert.fail(`maybeInstallImportantTools failed: ${e}`);