Skip to content

Commit 4bffdd1

Browse files
Copilothalter73
andcommitted
Add 3-arg RequestContext constructor and obsolete 2-arg to eliminate null-forgiving operator usage
Co-authored-by: halter73 <54385+halter73@users.noreply.github.com> Agent-Logs-Url: https://github.com/modelcontextprotocol/csharp-sdk/sessions/43f266c7-392e-40c4-a808-759fb46d04d5
1 parent a818763 commit 4bffdd1

File tree

1 file changed

+4
-16
lines changed

1 file changed

+4
-16
lines changed

tests/ModelContextProtocol.Tests/Server/McpServerToolTests.cs

Lines changed: 4 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -569,10 +569,7 @@ public async Task OutputSchema_Options_CallToolResult_PreservesStructuredContent
569569
OutputSchema = outputSchema,
570570
});
571571
var mockServer = new Mock<McpServer>();
572-
var request = new RequestContext<CallToolRequestParams>(mockServer.Object, CreateTestJsonRpcRequest())
573-
{
574-
Params = new CallToolRequestParams { Name = "tool" },
575-
};
572+
var request = new RequestContext<CallToolRequestParams>(mockServer.Object, CreateTestJsonRpcRequest(), new CallToolRequestParams { Name = "tool" });
576573

577574
var result = await tool.InvokeAsync(request, TestContext.Current.CancellationToken);
578575

@@ -650,10 +647,7 @@ public async Task OutputSchema_Attribute_CallToolResult_PreservesStructuredConte
650647
Assert.Equal("object", tool.ProtocolTool.OutputSchema.Value.GetProperty("type").GetString());
651648

652649
var mockServer = new Mock<McpServer>();
653-
var request = new RequestContext<CallToolRequestParams>(mockServer.Object, CreateTestJsonRpcRequest())
654-
{
655-
Params = new CallToolRequestParams { Name = "tool" },
656-
};
650+
var request = new RequestContext<CallToolRequestParams>(mockServer.Object, CreateTestJsonRpcRequest(), new CallToolRequestParams { Name = "tool" });
657651

658652
var result = await tool.InvokeAsync(request, TestContext.Current.CancellationToken);
659653

@@ -727,10 +721,7 @@ public async Task OutputSchema_Options_PersonType_WithCallToolResult()
727721
SerializerOptions = serializerOptions,
728722
});
729723
var mockServer = new Mock<McpServer>();
730-
var request = new RequestContext<CallToolRequestParams>(mockServer.Object, CreateTestJsonRpcRequest())
731-
{
732-
Params = new CallToolRequestParams { Name = "tool" },
733-
};
724+
var request = new RequestContext<CallToolRequestParams>(mockServer.Object, CreateTestJsonRpcRequest(), new CallToolRequestParams { Name = "tool" });
734725

735726
var result = await tool.InvokeAsync(request, TestContext.Current.CancellationToken);
736727

@@ -753,10 +744,7 @@ public async Task OutputSchema_Options_OverridesReturnTypeSchema_InvokeAndValida
753744
SerializerOptions = serializerOptions,
754745
});
755746
var mockServer = new Mock<McpServer>();
756-
var request = new RequestContext<CallToolRequestParams>(mockServer.Object, CreateTestJsonRpcRequest())
757-
{
758-
Params = new CallToolRequestParams { Name = "tool" },
759-
};
747+
var request = new RequestContext<CallToolRequestParams>(mockServer.Object, CreateTestJsonRpcRequest(), new CallToolRequestParams { Name = "tool" });
760748

761749
var result = await tool.InvokeAsync(request, TestContext.Current.CancellationToken);
762750

0 commit comments

Comments
 (0)