Merge branch 'topic/vscode-testing' into 'master'

Fix VS Code extension on Windows

See merge request eng/ide/ada_language_server!1263

Author: eliericha

integration/vscode/ada/package-lock.json

@@ -196,6 +196,11 @@ function createClient(
         );
     }
 
+    if (process.platform == 'win32') {
+        // Add the extension for the file lookup further below
+        serverExecPath = `${serverExecPath}.exe`;
+    }
+
     // If the ALS environment variable is specified, use it as the path of the
     // server executable.
     if (process.env.ALS) {

integration/vscode/ada/src/extension.ts

@@ -242,8 +242,11 @@ function testSyntaxHighlighting(absFilePath: string, syntax: Syntaxes) {
         }
 
         const cmd = [
-            // Use npx to avoid sensitivity to PATH env var
-            'npx',
+            // Use npx to avoid sensitivity to PATH env var. On Windows, the
+            // Node installation provides a 'npx' executable file which is a
+            // Bash script which doesn't work on Windows. Instead on Windows,
+            // the 'npx.cmd' file should be used.
+            process.platform == 'win32' ? 'npx.cmd' : 'npx',
             'vscode-tmgrammar-snap',
             // We pass a non-existing language configuration, otherwise the tool
             // picks up the package.json file and always loads the grammar in