VS Code doesn't call shutdown when exiting the standard client (#2472)

Signed-off-by: Snjezana Peco <snjezana.peco@redhat.com>

Author: snjeza

src/extension.ts

@@ -427,7 +427,9 @@ export function activate(context: ExtensionContext): Promise<ExtensionAPI> {
 
 			apiManager.getApiInstance().onDidServerModeChange((event: ServerMode) => {
 				if (event === ServerMode.STANDARD) {
-					syntaxClient.stop();
+					syntaxClient.getClient().stop().then(client => {
+						syntaxClient.stop();
+					});
 					fileEventHandler.setServerStatus(true);
 					runtimeStatusBarProvider.initialize(context);
 				}
@@ -633,9 +635,16 @@ export function getJavaConfig(javaHome: string) {
 	return javaConfig;
 }
 
-export function deactivate(): void {
-	standardClient.stop();
-	syntaxClient.stop();
+export function deactivate(): Promise<void> {
+	return getActiveLanguageClient().then(client => {
+		standardClient.stop();
+		syntaxClient.stop();
+		if (!client) {
+			return undefined;
+		} else {
+			return client.stop();
+		}
+	});
 }
 
 export async function getActiveLanguageClient(): Promise<LanguageClient | undefined> {

src/standardLanguageClient.ts

@@ -520,7 +520,6 @@ export class StandardLanguageClient {
 
 	public stop() {
 		if (this.languageClient) {
-			this.languageClient.stop();
 			this.status = ClientStatus.Stopping;
 		}
 	}

src/syntaxLanguageClient.ts

@@ -92,7 +92,6 @@ export class SyntaxLanguageClient {
 	public stop() {
 		this.status = ClientStatus.Stopping;
 		if (this.languageClient) {
-			this.languageClient.stop();
 			this.languageClient = null;
 		}
 	}