Add support for code lens enable/disable options to roslyn LSP

package.json

@@ -1131,11 +1131,6 @@
             "default": true,
             "description": "Suppress 'hidden' diagnostics (such as 'unnecessary using directives') from appearing in the editor or the Problems pane."
           },
-          "csharp.referencesCodeLens.enabled": {
-            "type": "boolean",
-            "default": true,
-            "description": "Specifies whether the references CodeLens should be shown."
-          },
           "csharp.referencesCodeLens.filteredSymbols": {
             "type": "array",
             "items": {
@@ -1144,11 +1139,6 @@
             "default": [],
             "description": "Array of custom symbol names for which CodeLens should be disabled."
           },
-          "csharp.testsCodeLens.enabled": {
-            "type": "boolean",
-            "default": true,
-            "description": "Specifies whether the run and debug test CodeLens should be shown."
-          },
           "csharp.maxProjectFileCountForDiagnosticAnalysis": {
             "type": "number",
             "default": 1000,
@@ -1396,6 +1386,16 @@
             "description": "Generation behavior of properties when implement interface or abstract class.",
             "order": 10
           },
+          "dotnet.codeLens.enableReferencesCodeLens": {
+            "type": "boolean",
+            "default": true,
+            "description": "Specifies whether the references CodeLens should be shown."
+          },
+          "dotnet.codeLens.enableTestsCodeLens": {
+            "type": "boolean",
+            "default": true,
+            "description": "Specifies whether the run and debug test CodeLens should be shown."
+          },
           "dotnet.completion.showCompletionItemsFromUnimportedNamespaces": {
             "type": "boolean",
             "default": true,

src/shared/migrateOptions.ts

@@ -57,6 +57,14 @@ export const migrateOptions = [
         omnisharpOption: 'omnisharp.enableImportCompletion',
         roslynOption: 'dotnet.completion.showCompletionItemsFromUnimportedNamespaces',
     },
+    {
+        omnisharpOption: 'csharp.referencesCodeLens.enabled',
+        roslynOption: 'dotnet.codeLens.enableReferencesCodeLens',
+    },
+    {
+        omnisharpOption: 'csharp.testsCodeLens.enabled',
+        roslynOption: 'dotnet.codeLens.enableTestsCodeLens',
+    },
 ];
 
 export async function MigrateOptions(vscode: vscode): Promise<void> {

src/shared/options.ts

@@ -164,8 +164,18 @@ export class Options {
         const dotNetCliPaths = Options.readOption<string[]>(config, 'omnisharp.dotNetCliPaths', []);
 
         const useFormatting = Options.readOption<boolean>(config, 'csharp.format.enable', true);
-        const showReferencesCodeLens = Options.readOption<boolean>(config, 'csharp.referencesCodeLens.enabled', true);
-        const showTestsCodeLens = Options.readOption<boolean>(config, 'csharp.testsCodeLens.enabled', true);
+        const showReferencesCodeLens = Options.readOption<boolean>(
+            config,
+            'dotnet.codeLens.enableReferencesCodeLens',
+            true,
+            'csharp.referencesCodeLens.enabled'
+        );
+        const showTestsCodeLens = Options.readOption<boolean>(
+            config,
+            'dotnet.codeLens.enableTestsCodeLens',
+            true,
+            'csharp.testsCodeLens.enabled'
+        );
         const filteredSymbolsCodeLens = Options.readOption<string[]>(
             config,
             'csharp.referencesCodeLens.filteredSymbols',

test/integrationTests/codeLensProvider.integration.test.ts

@@ -28,9 +28,9 @@ suite(`CodeLensProvider: ${testAssetWorkspace.description}`, function () {
         const filePath = path.join(projectDirectory, fileName);
         fileUri = vscode.Uri.file(filePath);
 
-        const csharpConfig = vscode.workspace.getConfiguration('csharp');
-        await csharpConfig.update('referencesCodeLens.enabled', true);
-        await csharpConfig.update('testsCodeLens.enabled', true);
+        const csharpConfig = vscode.workspace.getConfiguration('dotnet');
+        await csharpConfig.update('codeLens.enableReferencesCodeLens', true);
+        await csharpConfig.update('codeLens.enableTestsCodeLens', true);
 
         await vscode.commands.executeCommand('vscode.open', fileUri);
 
@@ -90,10 +90,10 @@ suite(`CodeLensProvider options: ${testAssetWorkspace.description}`, function ()
     });
 
     /* Skip this test until we are able to understand the cause of flakiness */
-    test.skip("Returns no references code lenses when 'csharp.referencesCodeLens.enabled' option is set to false", async function () {
-        const csharpConfig = vscode.workspace.getConfiguration('csharp');
-        await csharpConfig.update('referencesCodeLens.enabled', false);
-        await csharpConfig.update('testsCodeLens.enabled', true);
+    test.skip("Returns no references code lenses when 'dotnet.codeLens.enableReferencesCodeLens' option is set to false", async function () {
+        const csharpConfig = vscode.workspace.getConfiguration('dotnet');
+        await csharpConfig.update('codeLens.enableReferencesCodeLens', false);
+        await csharpConfig.update('codeLens.enableTestsCodeLens', true);
 
         const codeLenses = await GetCodeLenses(fileUri, 100);
         expect(codeLenses.length).to.equal(4);
@@ -111,10 +111,10 @@ suite(`CodeLensProvider options: ${testAssetWorkspace.description}`, function ()
         }
     });
 
-    test("Returns no test code lenses when 'csharp.testsCodeLens.enabled' option is set to false", async function () {
-        const csharpConfig = vscode.workspace.getConfiguration('csharp');
-        await csharpConfig.update('referencesCodeLens.enabled', true);
-        await csharpConfig.update('testsCodeLens.enabled', false);
+    test("Returns no test code lenses when 'dotnet.testsCodeLens.enabled' option is set to false", async function () {
+        const csharpConfig = vscode.workspace.getConfiguration('dotnet');
+        await csharpConfig.update('codeLens.enableReferencesCodeLens', true);
+        await csharpConfig.update('codeLens.enableTestsCodeLens', false);
 
         const codeLenses = await GetCodeLenses(fileUri, 100);
         expect(codeLenses.length).to.equal(2);
@@ -127,10 +127,10 @@ suite(`CodeLensProvider options: ${testAssetWorkspace.description}`, function ()
         }
     });
 
-    test("Returns no code lenses when 'csharp.referencesCodeLens.enabled' and 'csharp.testsCodeLens.enabled' options are set to false", async function () {
-        const csharpConfig = vscode.workspace.getConfiguration('csharp');
-        await csharpConfig.update('referencesCodeLens.enabled', false);
-        await csharpConfig.update('testsCodeLens.enabled', false);
+    test("Returns no code lenses when 'dotnet.codeLens.enableReferencesCodeLens' and 'dotnet.codeLens.enableTestsCodeLens' options are set to false", async function () {
+        const csharpConfig = vscode.workspace.getConfiguration('dotnet');
+        await csharpConfig.update('codeLens.enableReferencesCodeLens', false);
+        await csharpConfig.update('codeLens.enableTestsCodeLens', false);
 
         const codeLenses = await GetCodeLenses(fileUri, 100);
         expect(codeLenses.length).to.equal(0);

test/unitTests/lspToolHost/configurationMiddleware.test.ts

@@ -209,6 +209,26 @@ const testData = [
         vsCodeConfiguration: null,
         declareInPackageJson: false,
     },
+    {
+        serverOption: 'csharp|code_lens.dotnet_enable_references_code_lens',
+        vsCodeConfiguration: 'dotnet.codeLens.enableReferencesCodeLens',
+        declareInPackageJson: true,
+    },
+    {
+        serverOption: 'mystery_language|code_lens.dotnet_enable_references_code_lens',
+        vsCodeConfiguration: null,
+        declareInPackageJson: false,
+    },
+    {
+        serverOption: 'csharp|code_lens.dotnet_enable_tests_code_lens',
+        vsCodeConfiguration: 'dotnet.codeLens.enableTestsCodeLens',
+        declareInPackageJson: true,
+    },
+    {
+        serverOption: 'mystery_language|code_lens.dotnet_enable_tests_code_lens',
+        vsCodeConfiguration: null,
+        declareInPackageJson: false,
+    },
 ];
 
 suite('Server option name to vscode configuration name test', () => {