* Initial plan * Initial plan for SEP-973 implementation Co-authored-by: MackinnonBuck <10456961+MackinnonBuck@users.noreply.github.com> * Implement SEP-973: Add Icon class and icon support to Implementation, Resource, Tool, and Prompt Co-authored-by: MackinnonBuck <10456961+MackinnonBuck@users.noreply.github.com> * Complete SEP-973 implementation with documentation and fix property consistency Co-authored-by: MackinnonBuck <10456961+MackinnonBuck@users.noreply.github.com> * Address code review feedback: improve tests, remove docs, split test files Co-authored-by: MackinnonBuck <10456961+MackinnonBuck@users.noreply.github.com> * Fix tests to use S.T.J. source generator * Styling fixes * Simplify Icons property documentation as suggested in code review Co-authored-by: MackinnonBuck <10456961+MackinnonBuck@users.noreply.github.com> * Rename Icon.Src property to Icon.Source for more .NET-y naming Co-authored-by: MackinnonBuck <10456961+MackinnonBuck@users.noreply.github.com> * Add icon support to McpServerTool infrastructure and attributes Co-authored-by: MackinnonBuck <10456961+MackinnonBuck@users.noreply.github.com> * Cleanup * Refactor icon tests to use Delegate overload instead of MethodInfo pattern Co-authored-by: MackinnonBuck <10456961+MackinnonBuck@users.noreply.github.com> * Change Icon.Sizes property from string to IList<string> per spec update Co-authored-by: jozkee <16040868+jozkee@users.noreply.github.com> * Add icon support to Prompts and Resources (CreateOptions and Attributes) Co-authored-by: MackinnonBuck <10456961+MackinnonBuck@users.noreply.github.com> * Add client-server integration tests for Icons and WebsiteUrl Co-authored-by: MackinnonBuck <10456961+MackinnonBuck@users.noreply.github.com> * Fix nullability warnings * Revert "Add client-server integration tests for Icons and WebsiteUrl" This reverts commit f37de15569e54e86a0709121c9bdf80c5045511c. * More nullability fixes * Add more tests * Address code review feedback: add Theme property, fix docs, improve test patterns Co-authored-by: jozkee <16040868+jozkee@users.noreply.github.com> * Update Icon tests to exercise Theme property Co-authored-by: jozkee <16040868+jozkee@users.noreply.github.com> * Add Theme property assertions to client-server integration tests Co-authored-by: jozkee <16040868+jozkee@users.noreply.github.com> * Fix nullability --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: MackinnonBuck <10456961+MackinnonBuck@users.noreply.github.com> Co-authored-by: Mackinnon Buck <mackinnon.buck@gmail.com> Co-authored-by: jozkee <16040868+jozkee@users.noreply.github.com> Co-authored-by: David Cantú <dacantu@microsoft.com>
94 lines
3.4 KiB
C#
94 lines
3.4 KiB
C#
using ModelContextProtocol.Protocol;
|
|
using System.Text.Json;
|
|
|
|
namespace ModelContextProtocol.Tests.Protocol;
|
|
|
|
public static class ToolTests
|
|
{
|
|
[Fact]
|
|
public static void Tool_SerializationRoundTrip_PreservesAllProperties()
|
|
{
|
|
// Arrange
|
|
var original = new Tool
|
|
{
|
|
Name = "get_weather",
|
|
Title = "Get Weather",
|
|
Description = "Get current weather information",
|
|
Icons =
|
|
[
|
|
new() { Source = "https://example.com/weather.png", MimeType = "image/png", Sizes = new List<string> { "48x48" } }
|
|
],
|
|
Annotations = new ToolAnnotations
|
|
{
|
|
Title = "Weather Tool",
|
|
ReadOnlyHint = true
|
|
}
|
|
};
|
|
|
|
// Act - Serialize to JSON
|
|
string json = JsonSerializer.Serialize(original, McpJsonUtilities.DefaultOptions);
|
|
|
|
// Act - Deserialize back from JSON
|
|
var deserialized = JsonSerializer.Deserialize<Tool>(json, McpJsonUtilities.DefaultOptions);
|
|
|
|
// Assert
|
|
Assert.NotNull(deserialized);
|
|
Assert.Equal(original.Name, deserialized.Name);
|
|
Assert.Equal(original.Title, deserialized.Title);
|
|
Assert.Equal(original.Description, deserialized.Description);
|
|
Assert.NotNull(deserialized.Icons);
|
|
Assert.Equal(original.Icons.Count, deserialized.Icons.Count);
|
|
Assert.Equal(original.Icons[0].Source, deserialized.Icons[0].Source);
|
|
Assert.Equal(original.Icons[0].MimeType, deserialized.Icons[0].MimeType);
|
|
Assert.Equal(original.Icons[0].Sizes, deserialized.Icons[0].Sizes);
|
|
Assert.NotNull(deserialized.Annotations);
|
|
Assert.Equal(original.Annotations.Title, deserialized.Annotations.Title);
|
|
Assert.Equal(original.Annotations.ReadOnlyHint, deserialized.Annotations.ReadOnlyHint);
|
|
}
|
|
|
|
[Fact]
|
|
public static void Tool_SerializationRoundTrip_WithMinimalProperties()
|
|
{
|
|
// Arrange
|
|
var original = new Tool
|
|
{
|
|
Name = "calculate"
|
|
};
|
|
|
|
// Act - Serialize to JSON
|
|
string json = JsonSerializer.Serialize(original, McpJsonUtilities.DefaultOptions);
|
|
|
|
// Act - Deserialize back from JSON
|
|
var deserialized = JsonSerializer.Deserialize<Tool>(json, McpJsonUtilities.DefaultOptions);
|
|
|
|
// Assert
|
|
Assert.NotNull(deserialized);
|
|
Assert.Equal(original.Name, deserialized.Name);
|
|
Assert.Equal(original.Title, deserialized.Title);
|
|
Assert.Equal(original.Description, deserialized.Description);
|
|
Assert.Equal(original.Icons, deserialized.Icons);
|
|
Assert.Equal(original.Annotations, deserialized.Annotations);
|
|
}
|
|
|
|
[Fact]
|
|
public static void Tool_HasCorrectJsonPropertyNames()
|
|
{
|
|
var tool = new Tool
|
|
{
|
|
Name = "test_tool",
|
|
Title = "Test Tool",
|
|
Description = "A test tool",
|
|
Icons = [new() { Source = "https://example.com/icon.png" }],
|
|
Annotations = new ToolAnnotations { Title = "Annotation Title" }
|
|
};
|
|
|
|
string json = JsonSerializer.Serialize(tool, McpJsonUtilities.DefaultOptions);
|
|
|
|
Assert.Contains("\"name\":", json);
|
|
Assert.Contains("\"title\":", json);
|
|
Assert.Contains("\"description\":", json);
|
|
Assert.Contains("\"icons\":", json);
|
|
Assert.Contains("\"annotations\":", json);
|
|
Assert.Contains("\"inputSchema\":", json);
|
|
}
|
|
}
|