Run all Razor tests with cohosting on

davidwengier · davidwengier · commit 514b13878d8d · 2025-07-22T14:12:28.000+10:00
diff --git a/.vscode/launch.json b/.vscode/launch.json
@@ -92,6 +92,33 @@
             "preLaunchTask": "packageDev",
             "internalConsoleOptions": "openOnSessionStart"
         },
+        {
+            "name": "[Razor Cohost] Run Current File Integration Test",
+            "type": "extensionHost",
+            "request": "launch",
+            "runtimeExecutable": "${execPath}",
+            "args": [
+                // Launch VSCode using a specific profile to ensure that user settings are not used.
+                // This profile must be imported into vscode before running this launch configuration.
+                // The profile can be found under /test/csharp-test-profile.
+                "--profile",
+                "csharp-test-profile",
+                "${workspaceRoot}/test/razor/razorIntegrationTests/testAssets/RazorApp/.vscode/RazorApp.code-workspace",
+                "--extensionDevelopmentPath=${workspaceRoot}",
+                "--extensionTestsPath=${workspaceRoot}/out/test/razor/razorIntegrationTests",
+                "--log",
+                "ms-dotnettools.csharp:trace"
+            ],
+            "env": {
+                "CODE_EXTENSIONS_PATH": "${workspaceRoot}",
+                "TEST_FILE_FILTER": "${file}"
+            },
+            "sourceMaps": true,
+            "outFiles": ["${workspaceRoot}/dist/*.js", "${workspaceRoot}/out/test/**/*.js"],
+            "resolveSourceMapLocations": ["${workspaceFolder}/**", "!**/node_modules/**"],
+            "preLaunchTask": "packageDev",
+            "internalConsoleOptions": "openOnSessionStart"
+        },
         {
             "name": "[O#] Run Current File Integration Tests",
             "type": "extensionHost",
@@ -197,6 +224,15 @@
             "program": "${workspaceFolder}/node_modules/gulp/bin/gulp.js",
             "args": ["test:integration:razor"],
             "cwd": "${workspaceFolder}"
+        },
+        {
+            "type": "node",
+            "request": "launch",
+            "name": "Razor Cohost integration tests",
+            "preLaunchTask": "build",
+            "program": "${workspaceFolder}/node_modules/gulp/bin/gulp.js",
+            "args": ["test:integration:razor:cohost"],
+            "cwd": "${workspaceFolder}"
         }
     ],
     "inputs": [
diff --git a/azure-pipelines/test-matrix.yml b/azure-pipelines/test-matrix.yml
@@ -28,6 +28,9 @@ jobs:
       RazorTests:
         npmCommand: test:integration:razor
         isIntegration: true
+      RazorCohostTests:
+        npmCommand: test:integration:razor:cohost
+        isIntegration: true
   pool: ${{ parameters.pool }}
   ${{ if parameters.containerName }}:
     container: ${{ parameters.containerName }}
diff --git a/package.json b/package.json
@@ -73,6 +73,7 @@
     "test:unit": "npm run compileDev && gulp test:unit",
     "test:integration:csharp": "npm run package && gulp test:integration:csharp",
     "test:integration:razor": "npm run package && gulp test:integration:razor",
+    "test:integration:razor:cohost": "npm run package && gulp test:integration:razor:cohost",
     "test:integration:devkit": "npm run package && gulp test:integration:devkit",
     "profiling": "npm run package && gulp profiling",
     "test:artifacts": "npm run compileDev && gulp test:artifacts",
diff --git a/tasks/testTasks.ts b/tasks/testTasks.ts
@@ -85,13 +85,27 @@ function createIntegrationTestSubTasks() {
                 `Razor Test Integration ${projectName}`
             )
         );
+
+        gulp.task(`test:integration:razor:cohost:${projectName}`, async () =>
+            // Register each test again, but as a regular test, which will run with cohosting on
+            runIntegrationTest(
+                projectName,
+                path.join('razor', 'razorIntegrationTests'),
+                `Razor Test Integration ${projectName}`
+            )
+        );
     }
 
     gulp.task(
         'test:integration:razor',
         gulp.series(razorIntegrationTestProjects.map((projectName) => `test:integration:razor:${projectName}`))
     );
 
+    gulp.task(
+        'test:integration:razor:cohost',
+        gulp.series(razorIntegrationTestProjects.map((projectName) => `test:integration:razor:cohost:${projectName}`))
+    );
+
     gulp.task(
         'test:integration',
         gulp.series('test:integration:csharp', 'test:integration:devkit', 'test:integration:razor')
diff --git a/test/razor/razorIntegrationTests/completion.integration.test.ts b/test/razor/razorIntegrationTests/completion.integration.test.ts
@@ -5,11 +5,11 @@
 
 import * as path from 'path';
 import * as vscode from 'vscode';
-import { beforeAll, afterAll, test, expect, beforeEach } from '@jest/globals';
+import { beforeAll, afterAll, test, expect, beforeEach, describe } from '@jest/globals';
 import testAssetWorkspace from './testAssets/testAssetWorkspace';
 import * as integrationHelpers from '../../lsptoolshost/integrationTests/integrationHelpers';
 
-integrationHelpers.describeIfDevKit(`Razor Hover ${testAssetWorkspace.description}`, function () {
+describe(`Razor Hover ${testAssetWorkspace.description}`, function () {
     beforeAll(async function () {
         if (!integrationHelpers.isRazorWorkspace(vscode.workspace)) {
             return;
diff --git a/test/razor/razorIntegrationTests/formatting.integration.test.ts b/test/razor/razorIntegrationTests/formatting.integration.test.ts
@@ -5,11 +5,11 @@
 
 import * as path from 'path';
 import * as vscode from 'vscode';
-import { beforeAll, afterAll, test, expect, beforeEach } from '@jest/globals';
+import { beforeAll, afterAll, test, expect, beforeEach, describe } from '@jest/globals';
 import testAssetWorkspace from './testAssets/testAssetWorkspace';
 import * as integrationHelpers from '../../lsptoolshost/integrationTests/integrationHelpers';
 
-integrationHelpers.describeIfDevKit(`Razor Formatting ${testAssetWorkspace.description}`, function () {
+describe(`Razor Formatting ${testAssetWorkspace.description}`, function () {
     beforeAll(async function () {
         if (!integrationHelpers.isRazorWorkspace(vscode.workspace)) {
             return;
diff --git a/test/razor/razorIntegrationTests/hover.integration.test.ts b/test/razor/razorIntegrationTests/hover.integration.test.ts
@@ -5,11 +5,11 @@
 
 import * as path from 'path';
 import * as vscode from 'vscode';
-import { beforeAll, afterAll, test, expect, beforeEach } from '@jest/globals';
+import { beforeAll, afterAll, test, expect, beforeEach, describe } from '@jest/globals';
 import testAssetWorkspace from './testAssets/testAssetWorkspace';
 import * as integrationHelpers from '../../lsptoolshost/integrationTests/integrationHelpers';
 
-integrationHelpers.describeIfDevKit(`Razor Hover ${testAssetWorkspace.description}`, function () {
+describe(`Razor Hover ${testAssetWorkspace.description}`, function () {
     beforeAll(async function () {
         if (!integrationHelpers.isRazorWorkspace(vscode.workspace)) {
             return;
diff --git a/test/razor/razorIntegrationTests/reference.integration.test.ts b/test/razor/razorIntegrationTests/reference.integration.test.ts
@@ -5,11 +5,11 @@
 
 import * as path from 'path';
 import * as vscode from 'vscode';
-import { beforeAll, afterAll, test, expect, beforeEach } from '@jest/globals';
+import { beforeAll, afterAll, test, expect, beforeEach, describe } from '@jest/globals';
 import testAssetWorkspace from './testAssets/testAssetWorkspace';
 import * as integrationHelpers from '../../lsptoolshost/integrationTests/integrationHelpers';
 
-integrationHelpers.describeIfDevKit(`Razor References ${testAssetWorkspace.description}`, function () {
+describe(`Razor References ${testAssetWorkspace.description}`, function () {
     beforeAll(async function () {
         if (!integrationHelpers.isRazorWorkspace(vscode.workspace)) {
             return;
diff --git a/test/razor/razorIntegrationTests/rename.integration.test.ts b/test/razor/razorIntegrationTests/rename.integration.test.ts
@@ -5,11 +5,11 @@
 
 import * as path from 'path';
 import * as vscode from 'vscode';
-import { beforeAll, afterAll, test, expect, beforeEach } from '@jest/globals';
+import { beforeAll, afterAll, test, expect, beforeEach, describe } from '@jest/globals';
 import testAssetWorkspace from './testAssets/testAssetWorkspace';
 import * as integrationHelpers from '../../lsptoolshost/integrationTests/integrationHelpers';
 
-integrationHelpers.describeIfDevKit(`Razor Rename ${testAssetWorkspace.description}`, function () {
+describe(`Razor Rename ${testAssetWorkspace.description}`, function () {
     beforeAll(async function () {
         if (!integrationHelpers.isRazorWorkspace(vscode.workspace)) {
             return;
diff --git a/test/razor/razorIntegrationTests/testAssets/RazorApp/.vscode/RazorApp.code-workspace b/test/razor/razorIntegrationTests/testAssets/RazorApp/.vscode/RazorApp.code-workspace
@@ -9,6 +9,9 @@
         "dotnet.server.useOmnisharp": false,
         "omnisharp.enableLspDriver": false,
         "razor.server.trace": "Trace",
-        "dotnet.preferCSharpExtension": true
+        "dotnet.preferCSharpExtension": true,
+        "razor.languageServer.cohostingEnabled": true,
+        "razor.format.enable": true,
+        "html.format.enable": true
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -9,6 +9,9 @@`
`9`	`9`	`"dotnet.server.useOmnisharp": false,`
`10`	`10`	`"omnisharp.enableLspDriver": false,`
`11`	`11`	`"razor.server.trace": "Trace",`
`12`		`- "dotnet.preferCSharpExtension": true`
	`12`	`+ "dotnet.preferCSharpExtension": true,`
	`13`	`+ "razor.languageServer.cohostingEnabled": true,`
	`14`	`+ "razor.format.enable": true,`
	`15`	`+ "html.format.enable": true`
`13`	`16`	`}`
`14`	`17`	`}`