Convert new integration tests to jest and add to CI

Author: dibarbet

Diff for: .vscode/launch.json

@@ -138,7 +138,6 @@
             ],
             "env": {
                 "CODE_EXTENSIONS_PATH": "${workspaceRoot}",
-                "OSVC_SUITE": "slnWithCsproj",
             },
             "sourceMaps": true,
             "outFiles": [

Diff for: azure-pipelines.yml

@@ -38,26 +38,28 @@ stages:
     strategy:
       matrix:
         linux:
-          demandsName: ImageOverride -equals Build.Ubuntu.2204.Amd64.Open
+          demandsName: 1es-ubuntu-2004-open
         windows:
-          demandsName: ImageOverride -equals 1es-windows-2022-open
+          demandsName: 1es-windows-2022-open
     pool:
       name: NetCore-Public
-      demands: $(demandsName)
+      demands: ImageOverride -equals $(demandsName)
     steps:
     - template: azure-pipelines/test.yml
 
-- stage: Test_Omnisharp
-  displayName: Test Omnisharp
+- stage: Test_OmniSharp
+  displayName: Test OmniSharp
   dependsOn: []
   jobs:
   - job: Test
     strategy:
       matrix:
         linux:
-          demandsName: ImageOverride -equals Build.Ubuntu.2204.Amd64.Open
+          demandsName: 1es-ubuntu-2004-open
+        windows:
+          demandsName: 1es-windows-2022-open
     pool:
       name: NetCore-Public
-      demands: $(demandsName)
+      demands: ImageOverride -equals $(demandsName)
     steps:
     - template: azure-pipelines/test-omnisharp.yml

Diff for: azure-pipelines/test-omnisharp.yml

@@ -7,20 +7,16 @@ steps:
 
 - template: prereqs.yml
 
-- pwsh: |
-    if ($IsLinux) {
-      Write-Host "Activating screen emulation"
-      /usr/bin/Xvfb :99 -screen 0 1024x768x24 > /dev/null 2>&1 &
-      $env:DISPLAY=':99.0'
-      Write-Host "Now running tests"
-    }
+- template: test-prereqs.yml
 
-    npm run omnisharptest
+- script: npm run omnisharptest
   displayName: 🧪 Run unit and integration tests
+  env:
+    DISPLAY: :99.0
 
 - task: PublishPipelineArtifact@1
   condition: failed()
   displayName: 'Upload integration test logs'
   inputs:
     targetPath: '$(Build.SourcesDirectory)/.vscode-test/user-data/logs'
-    artifactName: 'VSCode Test Logs ($(Agent.JobName)-$(System.JobAttempt))'
+    artifactName: 'VSCode Test OmniSharp Logs ($(Agent.JobName)-$(System.JobAttempt))'

Diff for: azure-pipelines/test-prereqs.yml

@@ -0,0 +1,9 @@
+steps:
+
+# We need to download the roslyn language server for the current platform to run integration tests against it.
+- script: gulp installDependencies
+  displayName: 'Install Roslyn Language Server'
+
+- script: /usr/bin/Xvfb :99 -screen 0 1024x768x24 > /dev/null 2>&1 &
+  displayName: 'Activate screen emulation'
+  condition: eq(variables['Agent.OS'], 'Linux')

Diff for: azure-pipelines/test.yml

@@ -7,16 +7,12 @@ steps:
 
 - template: prereqs.yml
 
-- pwsh: |
-    if ($IsLinux) {
-      Write-Host "Activating screen emulation"
-      /usr/bin/Xvfb :99 -screen 0 1024x768x24 > /dev/null 2>&1 &
-      $env:DISPLAY=':99.0'
-      Write-Host "Now running tests"
-    }
+- template: test-prereqs.yml
 
-    npm run test
+- script: npm run test
   displayName: 🧪 Run unit and integration tests
+  env:
+    DISPLAY: :99.0
 
 - task: PublishPipelineArtifact@1
   condition: failed()

Diff for: baseJestConfig.ts

@@ -0,0 +1,15 @@
+/*---------------------------------------------------------------------------------------------
+ *  Copyright (c) Microsoft Corporation. All rights reserved.
+ *  Licensed under the MIT License. See License.txt in the project root for license information.
+ *--------------------------------------------------------------------------------------------*/
+import type { Config } from 'jest';
+
+/**
+ * Defines a base project config that we can re-use across the project specific jest configs.
+ * We do this because jest generally does not inherit project configuration settings.
+ */
+export const baseProjectConfig: Config = {
+    preset: 'ts-jest',
+    testEnvironment: 'node',
+    transformIgnorePatterns: ['/dist/.+\\.js'],
+};

Diff for: jest.config.ts

@@ -5,17 +5,11 @@
 import type { Config } from 'jest';
 
 const config: Config = {
-    verbose: true,
-    preset: 'ts-jest',
-    testEnvironment: 'node',
-    transformIgnorePatterns: ['/dist/.+\\.js'],
-    // We need to explicity ignore the out directory for modules - otherwise we'll get duplicate vscode module,
-    // the TS version from the __mocks__ directory and the compiled js version from the out directory.
-    modulePathIgnorePatterns: ['out'],
-    // Specify jest to only run tests in jest folders.
-    // We also have to include the __mocks__ folder.  That folder must be next to node_modules so we can't move it,
-    // but if we specify roots, jest won't automatically pick it up.  So we have to specify it here.
-    roots: ['<rootDir>/test/unitTests', '<rootDir>/omnisharptest/omnisharpJestTests', '<rootDir>/__mocks__'],
+    projects: [
+        '<rootDir>/test/unitTests/jest.config.ts',
+        '<rootDir>/test/integrationTests/jest.config.ts',
+        '<rootDir>/omnisharptest/omnisharpJestTests/jest.config.ts',
+    ],
 };
 
 export default config;

Diff for: omnisharptest/omnisharpJestTests/jest.config.ts

@@ -0,0 +1,25 @@
+/*---------------------------------------------------------------------------------------------
+ *  Copyright (c) Microsoft Corporation. All rights reserved.
+ *  Licensed under the MIT License. See License.txt in the project root for license information.
+ *--------------------------------------------------------------------------------------------*/
+import type { Config } from 'jest';
+import { baseProjectConfig } from '../../baseJestConfig';
+
+export const jestOmniSharpUnitTestProjectName = 'OmniSharp Unit Tests';
+
+/**
+ * Defines a jest project configuration for O# jest tests.
+ */
+const omnisharpConfig: Config = {
+    ...baseProjectConfig,
+    displayName: jestOmniSharpUnitTestProjectName,
+    // We need to explicity ignore the out directory for modules - otherwise we'll get duplicate vscode module,
+    // the TS version from the __mocks__ directory and the compiled js version from the out directory.
+    modulePathIgnorePatterns: ['out'],
+    // Specify jest to only run tests in jest folders.
+    // We also have to include the __mocks__ folder.  That folder must be next to node_modules so we can't move it,
+    // but if we specify roots, jest won't automatically pick it up.  So we have to specify it here.
+    roots: ['<rootDir>', '<rootDir>/../../__mocks__'],
+};
+
+export default omnisharpConfig;