Update Roslyn to 4.14.0-3.25156.1 (#8049)

Author: JoeRobich

CHANGELOG.md

@@ -9,6 +9,23 @@
   * Improve solution load performance (#11591) (PR: [#11591](https://github.com/dotnet/razor/pull/11591))
   * Make logging fall into the pit of success (#11581) (PR: [#11581](https://github.com/dotnet/razor/pull/11581))
 * Bump xamlTools to 17.14.35904.287 (PR: [#8042](https://github.com/dotnet/vscode-csharp/pull/8042))
+* Update Roslyn to 4.14.0-3.25156.1 (PR: [#8049](https://github.com/dotnet/vscode-csharp/pull/8049))
+  * Don't use an implicit-object if the lang version doesn't support it (PR: [#77437](https://github.com/dotnet/roslyn/pull/77437))
+  * Fix issue where we were changing semantics when converting to a collection expr. (PR: [#77417](https://github.com/dotnet/roslyn/pull/77417))
+  * Detect and emit more idiomatic null check patterns (PR: [#77412](https://github.com/dotnet/roslyn/pull/77412))
+  * Partial events and constructors: IDE (PR: [#77337](https://github.com/dotnet/roslyn/pull/77337))
+  * Simplify keyword recommenders. (PR: [#77396](https://github.com/dotnet/roslyn/pull/77396))
+  * Remove async/await (PR: [#77397](https://github.com/dotnet/roslyn/pull/77397))
+  * Preserve encoding during DocumentState updates (PR: [#77354](https://github.com/dotnet/roslyn/pull/77354))
+  * Don't realize the SourceText in SyntaxTree.OverlapsHiddenPosition if not needed (PR: [#77334](https://github.com/dotnet/roslyn/pull/77334))
+  * Handle ModuleCancellationTokenExpression in AbstractFlow visitor (PR: [#77310](https://github.com/dotnet/roslyn/pull/77310))
+  * PERF: Reduce the number of nodes walked during import completion commit. (PR: [#77305](https://github.com/dotnet/roslyn/pull/77305))
+  * Allow expression body refactorings on non empty selections (PR: [#76969](https://github.com/dotnet/roslyn/pull/76969))
+  * Partial events and constructors: public API (PR: [#77202](https://github.com/dotnet/roslyn/pull/77202))
+  * Refresh diagnostics when fading options change (PR: [#77322](https://github.com/dotnet/roslyn/pull/77322))
+  * Reduce allocations during completion in FilterToVisibleAndBrowsableSymbols (PR: [#77315](https://github.com/dotnet/roslyn/pull/77315))
+  * Change override completion to select text after updating the buffer. (PR: [#76983](https://github.com/dotnet/roslyn/pull/76983))
+  * Fix false positive 'Unnecessary assignment of a value' (PR: [#77297](https://github.com/dotnet/roslyn/pull/77297))
 
 # 2.68.x
 * Update Razor to 9.0.0-preview.25125.9 (PR: [#8027](https://github.com/dotnet/vscode-csharp/pull/8027))

package.json

@@ -37,7 +37,7 @@
     }
   },
   "defaults": {
-    "roslyn": "4.14.0-2.25120.5",
+    "roslyn": "4.14.0-3.25156.1",
     "omniSharp": "1.39.12",
     "razor": "9.0.0-preview.25156.2",
     "razorOmnisharp": "7.0.0-preview.23363.1",

test/lsptoolshost/integrationTests/codeactions.integration.test.ts

@@ -79,10 +79,12 @@ describe(`Code Actions Tests`, () => {
     test('Add accessibility modifiers applied', async () => {
         const actions = await getCodeActions(new vscode.Range(6, 16, 6, 19), 10);
 
-        expect(actions[0].title).toBe('Add accessibility modifiers');
-        expect(actions[0].edit).toBeDefined();
+        const action = actions.find((a) => a.title === 'Add accessibility modifiers');
 
-        await vscode.workspace.applyEdit(actions[0].edit!);
+        expect(action).toBeDefined();
+        expect(action!.edit).toBeDefined();
+
+        await vscode.workspace.applyEdit(action!.edit!);
 
         await expectText(vscode.window.activeTextEditor!.document, [
             'using System;',