Fix, skip or update tooling tests

Skipped folding range test tracked by #10859 Skipped rename test tracked by #10856 Updated semantic tokens test tracked by #10857 Update folding range tests tracked by #10860 Update spell checking test tracked by #10863 Skipped formatting tests tracked by #10864 Skipped auto insert tests tracked by #10865
dotnet · Sep 10, 2024 · f929020 · f929020
1 parent 116950b
commit f929020
Show file tree

Hide file tree

Showing 9 changed files with 21 additions and 23 deletions.
diff --git a/...t.VisualStudio.LanguageServices.Razor.Test/Cohost/CohostDocumentFormattingEndpointTest.cs b/...t.VisualStudio.LanguageServices.Razor.Test/Cohost/CohostDocumentFormattingEndpointTest.cs
@@ -26,7 +26,7 @@ public class CohostDocumentFormattingEndpointTest(ITestOutputHelper testOutputHe
     // and provide regression prevention. The tests here are not exhaustive, but they validate the the cohost endpoints
     // call into the formatting engine at least, and handles C#, Html and Razor formatting changes correctly.
 
-    [Fact]
+    [Fact(Skip = "https://github.com/dotnet/razor/issues/10864")]
     public Task Formatting()
         => VerifyDocumentFormattingAsync(
             input: """

diff --git a/...t.VisualStudio.LanguageServices.Razor.Test/Cohost/CohostDocumentSpellCheckEndpointTest.cs b/...t.VisualStudio.LanguageServices.Razor.Test/Cohost/CohostDocumentSpellCheckEndpointTest.cs
@@ -19,7 +19,7 @@ public class CohostDocumentSpellCheckEndpointTest(ITestOutputHelper testOutputHe
     public async Task Handle()
     {
         var input = """
-            @page [|"this is csharp"|]
+            @page [|"/this is csharp"|]
 
             <div>[|
 
@@ -37,7 +37,7 @@ Eat more chickin.
                 @([|"unless they contain csharp"|])
             </style>
 
-            @{ var [|x|] = [|"csharp"|];
+            @{ var [|x|] = [|"csharp"|]; }
 
             @*[| Eat more chickin. |]*@
 
@@ -52,10 +52,10 @@ Eat more chickin.
             }
             """;
 
-        await VerifySemanticTokensAsync(input);
+        await VerifySpellCheckRangesAsync(input);
     }
 
-    private async Task VerifySemanticTokensAsync(TestCode input)
+    private async Task VerifySpellCheckRangesAsync(TestCode input)
     {
         var document = CreateProjectAndRazorDocument(input.Text);
         var sourceText = await document.GetTextAsync(DisposalToken);

diff --git a/...crosoft.VisualStudio.LanguageServices.Razor.Test/Cohost/CohostFoldingRangeEndpointTest.cs b/...crosoft.VisualStudio.LanguageServices.Razor.Test/Cohost/CohostFoldingRangeEndpointTest.cs
@@ -27,15 +27,15 @@ public Task IfStatements()
                 <div>
                   Hello World
                 </div>
-              }|]
-            </div>
+              }
+            |]</div>
 
             @if (true) {[|
               <div>
                 Hello World
               </div>
-            }|]
-
+            }
+            |]
             @if (true) {[|
             }|]
             """);
@@ -67,11 +67,11 @@ Hello World
                     Goodbye World
                 </div>
                 }|]
-              }|]
-            </div>
+              }
+            |]</div>
             """);
 
-    [Fact]
+    [Fact(Skip = "https://github.com/dotnet/razor/issues/10859")]
     public Task Usings()
         => VerifyFoldingRangesAsync("""
             @using System[|

diff --git a/...crosoft.VisualStudio.LanguageServices.Razor.Test/Cohost/CohostOnAutoInsertEndpointTest.cs b/...crosoft.VisualStudio.LanguageServices.Razor.Test/Cohost/CohostOnAutoInsertEndpointTest.cs
@@ -84,7 +84,7 @@ The end.
             delegatedResponseText: "\"$0\"");
     }
 
-    [Fact]
+    [Fact(Skip = "https://github.com/dotnet/razor/issues/10865")]
     public async Task CSharp_OnForwardSlash()
     {
         await VerifyOnAutoInsertAsync(
@@ -120,7 +120,7 @@ void TestMethod() {}
             formatOnType: false);
     }
 
-    [Fact]
+    [Fact(Skip = "https://github.com/dotnet/razor/issues/10865")]
     public async Task CSharp_OnEnter()
     {
         await VerifyOnAutoInsertAsync(
@@ -141,7 +141,7 @@ void TestMethod()
             triggerCharacter: "\n");
     }
 
-    [Fact]
+    [Fact(Skip = "https://github.com/dotnet/razor/issues/10865")]
     public async Task CSharp_OnEnter_TwoSpaceIndent()
     {
         await VerifyOnAutoInsertAsync(
@@ -163,7 +163,7 @@ void TestMethod()
             tabSize: 2);
     }
 
-    [Fact]
+    [Fact(Skip = "https://github.com/dotnet/razor/issues/10865")]
     public async Task CSharp_OnEnter_UseTabs()
     {
         const char tab = '\t';

diff --git a/...oft.VisualStudio.LanguageServices.Razor.Test/Cohost/CohostOnTypeFormattingEndpointTest.cs b/...oft.VisualStudio.LanguageServices.Razor.Test/Cohost/CohostOnTypeFormattingEndpointTest.cs
@@ -56,7 +56,7 @@ await VerifyOnTypeFormattingAsync(
             triggerCharacter: '\n');
     }
 
-    [Fact]
+    [Fact(Skip = "https://github.com/dotnet/razor/issues/10864")]
     public async Task CSharp()
     {
         await VerifyOnTypeFormattingAsync(

diff --git a/...soft.VisualStudio.LanguageServices.Razor.Test/Cohost/CohostRangeFormattingEndpointTest.cs b/...soft.VisualStudio.LanguageServices.Razor.Test/Cohost/CohostRangeFormattingEndpointTest.cs
@@ -22,7 +22,7 @@ namespace Microsoft.VisualStudio.Razor.LanguageClient.Cohost;
 [UseExportProvider]
 public class CohostRangeFormattingEndpointTest(ITestOutputHelper testOutputHelper) : CohostEndpointTestBase(testOutputHelper)
 {
-    [Fact]
+    [Fact(Skip = "https://github.com/dotnet/razor/issues/10864")]
     public Task RangeFormatting()
         => VerifyRangeFormattingAsync(
             input: """

diff --git a/...est/Microsoft.VisualStudio.LanguageServices.Razor.Test/Cohost/CohostRenameEndpointTest.cs b/...est/Microsoft.VisualStudio.LanguageServices.Razor.Test/Cohost/CohostRenameEndpointTest.cs
@@ -20,7 +20,7 @@ namespace Microsoft.VisualStudio.Razor.LanguageClient.Cohost;
 
 public class CohostRenameEndpointTest(ITestOutputHelper testOutputHelper) : CohostEndpointTestBase(testOutputHelper)
 {
-    [Fact]
+    [Fact(Skip = "https://github.com/dotnet/razor/issues/10856")]
     public Task CSharp_Method()
         => VerifyRenamesAsync(
             input: """

diff --git a/...st/Microsoft.VisualStudio.LanguageServices.Razor.Test/TestFiles/SemanticTokens/Legacy.txt b/...st/Microsoft.VisualStudio.LanguageServices.Razor.Test/TestFiles/SemanticTokens/Legacy.txt
@@ -42,8 +42,7 @@ Line Δ, Char Δ, Length, Type, Modifier(s), Text
 1 0 1 razorTransition [] [}]
 2 0 1 razorTransition [] [@]
 0 1 7 razorDirective [] [section]
-0 8 9 local name [] [MySection]
-0 10 1 razorTransition [] [{]
+0 18 1 razorTransition [] [{]
 1 4 1 markupTagDelimiter [] [<]
 0 1 3 markupElement [] [div]
 0 3 1 markupTagDelimiter [] [>]

diff --git a/...ualStudio.LanguageServices.Razor.Test/TestFiles/SemanticTokens/Legacy_with_background.txt b/...ualStudio.LanguageServices.Razor.Test/TestFiles/SemanticTokens/Legacy_with_background.txt
@@ -48,8 +48,7 @@ Line Δ, Char Δ, Length, Type, Modifier(s), Text
 1 0 1 razorTransition [] [}]
 2 0 1 razorTransition [] [@]
 0 1 7 razorDirective [] [section]
-0 8 9 local name [razorCode] [MySection]
-0 10 1 razorTransition [] [{]
+0 18 1 razorTransition [] [{]
 1 4 1 markupTagDelimiter [] [<]
 0 1 3 markupElement [] [div]
 0 3 1 markupTagDelimiter [] [>]