Merge branch 'topic/als#1311' into 'master'

Add logging for didChangeConfiguration event

See merge request eng/ide/ada_language_server!1517

Author: AnthonyLeonardoGracio

integration/vscode/ada/src/ExtensionState.ts

@@ -8,6 +8,7 @@ import { initializeTesting } from './gnattest';
 import { GprTaskProvider } from './gprTaskProvider';
 import { TERMINAL_ENV_SETTING_NAME } from './helpers';
 import { registerTaskProviders } from './taskProviders';
+import { logger } from './extension';
 
 /**
  * This class encapsulates all state that should be maintained throughout the
@@ -125,10 +126,13 @@ export class ExtensionState {
     //  React to changes in configuration to recompute predefined tasks if the user
     //  changes scenario variables' values.
     public configChanged = (e: vscode.ConfigurationChangeEvent) => {
+        logger.info('didChangeConfiguration event received');
+
         if (
             e.affectsConfiguration('ada.scenarioVariables') ||
             e.affectsConfiguration('ada.projectFile')
         ) {
+            logger.info('project related settings have changed: clearing caches for tasks');
             this.clearALSCache();
             this.unregisterTaskProviders();
             this.registerTaskProviders();
@@ -138,6 +142,10 @@ export class ExtensionState {
         //  a popup to reload the VS Code window and thus restart the
         //  Ada extension.
         if (e.affectsConfiguration(TERMINAL_ENV_SETTING_NAME)) {
+            const new_value = vscode.workspace.getConfiguration().get(TERMINAL_ENV_SETTING_NAME);
+            logger.info(`${TERMINAL_ENV_SETTING_NAME} has changed: show reload popup`);
+            logger.info(`${TERMINAL_ENV_SETTING_NAME}: ${JSON.stringify(new_value, undefined, 2)}`);
+
             void this.showReloadWindowPopup();
         }
     };

integration/vscode/ada/src/extension.ts

@@ -140,6 +140,11 @@ async function activateExtension(context: vscode.ExtensionContext) {
     adaExtState = new ExtensionState(context);
     context.subscriptions.push(adaExtState);
 
+    // Subscribe to the didChangeConfiguration event
+    context.subscriptions.push(
+        vscode.workspace.onDidChangeConfiguration(adaExtState.configChanged)
+    );
+
     const alsMiddleware: Middleware = {
         executeCommand: alsCommandExecutor(adaExtState.adaClient),
     };
@@ -148,10 +153,6 @@ async function activateExtension(context: vscode.ExtensionContext) {
 
     await adaExtState.start();
 
-    context.subscriptions.push(
-        vscode.workspace.onDidChangeConfiguration(adaExtState.configChanged)
-    );
-
     /**
      * Register commands first so that commands such as displaying the extension
      * Output become available even if the language servers fail to start.