pass test

CppCXY · CppCXY · commit 18b10be00737 · 2025-10-22T14:24:21.000+08:00
diff --git a/LanguageServer.Framework.Tests/CONTRIBUTING.md b/LanguageServer.Framework.Tests/CONTRIBUTING.md
diff --git a/LanguageServer.Framework.Tests/Handlers/CallHierarchyHandlerTests.cs b/LanguageServer.Framework.Tests/Handlers/CallHierarchyHandlerTests.cs
@@ -152,7 +152,7 @@ public async Task IncomingCalls_ShouldReturnIncomingCalls()
         // Assert
         response.Should().NotBeNull();
         response!.Result.Should().HaveCount(1);
-        response.Result[0].From.Uri.Should().Be("file:///caller.txt");
+        response.Result[0].From.Uri.Uri.AbsoluteUri.Should().Be("file:///caller.txt");
         response.Result[0].From.Range.Start.Line.Should().Be(15);
     }
 
@@ -185,7 +185,7 @@ public async Task OutgoingCalls_ShouldReturnOutgoingCalls()
         // Assert
         response.Should().NotBeNull();
         response!.Result.Should().HaveCount(1);
-        response.Result[0].To.Uri.Should().Be("file:///callee.txt");
+        response.Result[0].To.Uri.Uri.AbsoluteUri.Should().Be("file:///callee.txt");
         response.Result[0].To.Range.Start.Line.Should().Be(25);
     }
 
diff --git a/LanguageServer.Framework.Tests/Handlers/CompletionHandlerTests.cs b/LanguageServer.Framework.Tests/Handlers/CompletionHandlerTests.cs
@@ -110,7 +110,7 @@ public async Task Resolve_ShouldAddDocumentation()
 
         // Assert
         resolved.Should().NotBeNull();
-        resolved.Documentation.Should().Be("Resolved documentation");
+        resolved.Documentation?.StringValue.Should().Be("Resolved documentation");
     }
 
     [Fact]
diff --git a/LanguageServer.Framework.Tests/Handlers/DeclarationHandlerTests.cs b/LanguageServer.Framework.Tests/Handlers/DeclarationHandlerTests.cs
@@ -59,9 +59,10 @@ public async Task Handle_ShouldReturnDeclarationLocations()
 
         // Assert
         response.Should().NotBeNull();
-        response!.Result1.Should().NotBeNull();
-        response.Result1!.Value.Uri.Should().Be("file:///declarations.txt");
-        response.Result1!.Value.Range.Start.Line.Should().Be(2);
+        response!.Result2.Should().NotBeNull();
+        response.Result2.Should().HaveCount(1);
+        response.Result2![0].Uri.Uri.AbsoluteUri.Should().Be("file:///declarations.txt");
+        response.Result2[0].Range.Start.Line.Should().Be(2);
     }
 
     [Fact]
diff --git a/LanguageServer.Framework.Tests/Handlers/DefinitionHandlerTests.cs b/LanguageServer.Framework.Tests/Handlers/DefinitionHandlerTests.cs
@@ -59,9 +59,10 @@ public async Task Handle_ShouldReturnDefinitionLocations()
 
         // Assert
         response.Should().NotBeNull();
-        response!.Result1.Should().NotBeNull();
-        response.Result1!.Value.Uri.Should().Be("file:///source.txt");
-        response.Result1.Value.Range.Start.Line.Should().Be(10);
+        response!.Result2.Should().NotBeNull();
+        response.Result2.Should().HaveCount(1);
+        response.Result2![0].Uri.Uri.AbsoluteUri.Should().Be("file:///source.txt");
+        response.Result2[0].Range.Start.Line.Should().Be(10);
     }
 
     [Fact]
diff --git a/LanguageServer.Framework.Tests/Handlers/DocumentFormattingHandlerTests.cs b/LanguageServer.Framework.Tests/Handlers/DocumentFormattingHandlerTests.cs
@@ -85,7 +85,9 @@ public async Task Handle_ShouldReturnFormattingEdits()
 
         // Act
         var method = handler.GetType()
-            .GetMethod("Handle", System.Reflection.BindingFlags.NonPublic | System.Reflection.BindingFlags.Instance)!;
+            .GetMethod("Handle", System.Reflection.BindingFlags.NonPublic | System.Reflection.BindingFlags.Instance,
+                null,
+                [typeof(DocumentFormattingParams), typeof(CancellationToken)], null)!;
 
         var task = method.Invoke(handler, [request, CancellationToken.None]);
 
diff --git a/LanguageServer.Framework.Tests/Handlers/DocumentLinkHandlerTests.cs b/LanguageServer.Framework.Tests/Handlers/DocumentLinkHandlerTests.cs
@@ -69,7 +69,7 @@ public async Task Handle_ShouldReturnDocumentLinks()
         // Assert
         response.Should().NotBeNull();
         response!.DocumentLinks.Should().HaveCount(1);
-        response.DocumentLinks[0].Target.Should().Be("https://example.com");
+        response.DocumentLinks[0].Target!.Value.Uri.AbsoluteUri.Should().Be("https://example.com/");
         response.DocumentLinks[0].Tooltip.Should().Be("Example link");
     }
 
diff --git a/LanguageServer.Framework.Tests/Handlers/ExecuteCommandHandlerTests.cs b/LanguageServer.Framework.Tests/Handlers/ExecuteCommandHandlerTests.cs
@@ -59,7 +59,7 @@ public async Task Handle_ShouldExecuteCommand()
 
         // Assert
         response.Should().NotBeNull();
-        response!.Result.Should().Be("Command 1 executed");
+        response!.Result!.Value.Should().Be("Command 1 executed");
     }
 
     [Fact]
@@ -83,7 +83,7 @@ public async Task Handle_WithUnknownCommand_ShouldReturnUnknownMessage()
 
         // Assert
         response.Should().NotBeNull();
-        response!.Result.Should().Be("Unknown command");
+        response!.Result!.Value.Should().Be("Unknown command");
     }
 
     [Fact]
diff --git a/LanguageServer.Framework.Tests/Handlers/ImplementationHandlerTests.cs b/LanguageServer.Framework.Tests/Handlers/ImplementationHandlerTests.cs
@@ -68,8 +68,8 @@ public async Task Handle_ShouldReturnImplementationLocations()
         // Assert
         response.Should().NotBeNull();
         response!.Result2.Should().HaveCount(2);
-        response.Result2![0].Uri.Should().Be("file:///implementation1.txt");
-        response.Result2[1].Uri.Should().Be("file:///implementation2.txt");
+        response.Result2![0].Uri.Uri.AbsoluteUri.Should().Be("file:///implementation1.txt");
+        response.Result2[1].Uri.Uri.AbsoluteUri.Should().Be("file:///implementation2.txt");
     }
 
     [Fact]
diff --git a/LanguageServer.Framework.Tests/Handlers/InlayHintHandlerTests.cs b/LanguageServer.Framework.Tests/Handlers/InlayHintHandlerTests.cs
@@ -106,7 +106,7 @@ public async Task Resolve_ShouldAddTooltip()
 
         // Assert
         resolved.Should().NotBeNull();
-        resolved.Tooltip.Should().Be("Resolved tooltip");
+        resolved.Tooltip!.StringValue.Should().Be("Resolved tooltip");
     }
 
     [Fact]
diff --git a/LanguageServer.Framework.Tests/Handlers/ReferenceHandlerTests.cs b/LanguageServer.Framework.Tests/Handlers/ReferenceHandlerTests.cs
@@ -69,8 +69,8 @@ public async Task Handle_ShouldReturnReferenceLocations()
         // Assert
         response.Should().NotBeNull();
         response!.Result.Should().HaveCount(2);
-        response.Result[0].Uri.Should().Be("file:///ref1.txt");
-        response.Result[1].Uri.Should().Be("file:///ref2.txt");
+        response.Result[0].Uri.Uri.AbsoluteUri.Should().Be("file:///ref1.txt");
+        response.Result[1].Uri.Uri.AbsoluteUri.Should().Be("file:///ref2.txt");
     }
 
     [Fact]
diff --git a/LanguageServer.Framework.Tests/Handlers/TextDocumentHandlerTests.cs b/LanguageServer.Framework.Tests/Handlers/TextDocumentHandlerTests.cs
@@ -64,7 +64,7 @@ public override void RegisterCapability(ServerCapabilities serverCapabilities,
             };
         }
 
-        public string? GetDocumentContent(string uri)
+        public string? GetDocumentContent(DocumentUri uri)
         {
             return _documents.GetValueOrDefault(uri);
         }
@@ -118,7 +118,9 @@ public async Task DidChange_ShouldUpdateDocument()
         };
 
         await (handler.GetType()
-            .GetMethod("DidOpen", System.Reflection.BindingFlags.NonPublic | System.Reflection.BindingFlags.Instance)!
+            .GetMethod("Handle", System.Reflection.BindingFlags.NonPublic | System.Reflection.BindingFlags.Instance,
+                null,
+                [typeof(DidOpenTextDocumentParams), typeof(CancellationToken)], null)!
             .Invoke(handler, [openRequest, CancellationToken.None]) as Task)!;
 
         // Now change it
@@ -166,7 +168,9 @@ public async Task DidClose_ShouldRemoveDocument()
         };
 
         await (handler.GetType()
-            .GetMethod("DidOpen", System.Reflection.BindingFlags.NonPublic | System.Reflection.BindingFlags.Instance)!
+            .GetMethod("Handle", System.Reflection.BindingFlags.NonPublic | System.Reflection.BindingFlags.Instance,
+                null,
+                [typeof(DidOpenTextDocumentParams), typeof(CancellationToken)], null)!
             .Invoke(handler, [openRequest, CancellationToken.None]) as Task)!;
 
         // Now close it
diff --git a/LanguageServer.Framework.Tests/Handlers/TypeDefinitionHandlerTests.cs b/LanguageServer.Framework.Tests/Handlers/TypeDefinitionHandlerTests.cs
@@ -60,7 +60,7 @@ public async Task Handle_ShouldReturnTypeDefinitionLocations()
         // Assert
         response.Should().NotBeNull();
         response!.Result2.Should().HaveCount(1);
-        response.Result2![0].Uri.Should().Be("file:///types.txt");
+        response.Result2![0].Uri.Uri.AbsoluteUri.Should().Be("file:///types.txt");
     }
 
     [Fact]
diff --git a/LanguageServer.Framework.Tests/Handlers/Working/WorkingCompletionHandlerTests.cs b/LanguageServer.Framework.Tests/Handlers/Working/WorkingCompletionHandlerTests.cs
@@ -106,7 +106,7 @@ public async Task Resolve_ShouldAddDocumentation()
 
         // Assert
         resolved.Should().NotBeNull();
-        resolved.Documentation.Should().Be("Resolved documentation");
+        resolved.Documentation!.StringValue.Should().Be("Resolved documentation");
     }
 
     [Fact]
diff --git a/LanguageServer.Framework.Tests/README.md b/LanguageServer.Framework.Tests/README.md

Original file line number	Diff line number	Diff line change
`@@ -110,7 +110,7 @@ public async Task Resolve_ShouldAddDocumentation()`
`110`	`110`
`111`	`111`	`// Assert`
`112`	`112`	`resolved.Should().NotBeNull();`
`113`		`- resolved.Documentation.Should().Be("Resolved documentation");`
	`113`	`+ resolved.Documentation?.StringValue.Should().Be("Resolved documentation");`
`114`	`114`	`}`
`115`	`115`
`116`	`116`	`[Fact]`
Original file line number	Diff line number	Diff line change
`@@ -69,7 +69,7 @@ public async Task Handle_ShouldReturnDocumentLinks()`
`69`	`69`	`// Assert`
`70`	`70`	`response.Should().NotBeNull();`
`71`	`71`	`response!.DocumentLinks.Should().HaveCount(1);`
`72`		`- response.DocumentLinks[0].Target.Should().Be("https://example.com");`
	`72`	`+ response.DocumentLinks[0].Target!.Value.Uri.AbsoluteUri.Should().Be("https://example.com/");`
`73`	`73`	`response.DocumentLinks[0].Tooltip.Should().Be("Example link");`
`74`	`74`	`}`
`75`	`75`
Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ public async Task Handle_ShouldExecuteCommand()`
`59`	`59`
`60`	`60`	`// Assert`
`61`	`61`	`response.Should().NotBeNull();`
`62`		`- response!.Result.Should().Be("Command 1 executed");`
	`62`	`+ response!.Result!.Value.Should().Be("Command 1 executed");`
`63`	`63`	`}`
`64`	`64`
`65`	`65`	`[Fact]`
`@@ -83,7 +83,7 @@ public async Task Handle_WithUnknownCommand_ShouldReturnUnknownMessage()`
`83`	`83`
`84`	`84`	`// Assert`
`85`	`85`	`response.Should().NotBeNull();`
`86`		`- response!.Result.Should().Be("Unknown command");`
	`86`	`+ response!.Result!.Value.Should().Be("Unknown command");`
`87`	`87`	`}`
`88`	`88`
`89`	`89`	`[Fact]`
Original file line number	Diff line number	Diff line change
`@@ -106,7 +106,7 @@ public async Task Resolve_ShouldAddTooltip()`
`106`	`106`
`107`	`107`	`// Assert`
`108`	`108`	`resolved.Should().NotBeNull();`
`109`		`- resolved.Tooltip.Should().Be("Resolved tooltip");`
	`109`	`+ resolved.Tooltip!.StringValue.Should().Be("Resolved tooltip");`
`110`	`110`	`}`
`111`	`111`
`112`	`112`	`[Fact]`