Move delegation tests into dedicated DelegatingMcpServer*Tests.cs files

Co-authored-by: jeffhandley <1031940+jeffhandley@users.noreply.github.com>

Author: Copilot

tests/ModelContextProtocol.Tests/Server/DelegatingMcpServerPromptTests.cs

@@ -0,0 +1,29 @@
+using ModelContextProtocol.Server;
+using System.Reflection;
+
+namespace ModelContextProtocol.Tests.Server;
+
+public class DelegatingMcpServerPromptTests
+{
+    [Fact]
+    public void OverridesAllVirtualAndAbstractMembers()
+    {
+        MethodInfo[] baseMethods = typeof(McpServerPrompt).GetMethods(BindingFlags.Public | BindingFlags.Instance)
+            .Where(m => (m.IsVirtual || m.IsAbstract) && m.DeclaringType == typeof(McpServerPrompt))
+            .ToArray();
+
+        Assert.NotEmpty(baseMethods);
+
+        foreach (MethodInfo baseMethod in baseMethods)
+        {
+            MethodInfo? overriddenMethod = typeof(DelegatingMcpServerPrompt).GetMethod(
+                baseMethod.Name,
+                BindingFlags.Public | BindingFlags.Instance,
+                baseMethod.GetParameters().Select(p => p.ParameterType).ToArray());
+
+            Assert.True(
+                overriddenMethod is not null && overriddenMethod.DeclaringType == typeof(DelegatingMcpServerPrompt),
+                $"DelegatingMcpServerPrompt does not override {baseMethod.Name} from McpServerPrompt.");
+        }
+    }
+}

tests/ModelContextProtocol.Tests/Server/DelegatingMcpServerResourceTests.cs

@@ -0,0 +1,29 @@
+using ModelContextProtocol.Server;
+using System.Reflection;
+
+namespace ModelContextProtocol.Tests.Server;
+
+public class DelegatingMcpServerResourceTests
+{
+    [Fact]
+    public void OverridesAllVirtualAndAbstractMembers()
+    {
+        MethodInfo[] baseMethods = typeof(McpServerResource).GetMethods(BindingFlags.Public | BindingFlags.Instance)
+            .Where(m => (m.IsVirtual || m.IsAbstract) && m.DeclaringType == typeof(McpServerResource))
+            .ToArray();
+
+        Assert.NotEmpty(baseMethods);
+
+        foreach (MethodInfo baseMethod in baseMethods)
+        {
+            MethodInfo? overriddenMethod = typeof(DelegatingMcpServerResource).GetMethod(
+                baseMethod.Name,
+                BindingFlags.Public | BindingFlags.Instance,
+                baseMethod.GetParameters().Select(p => p.ParameterType).ToArray());
+
+            Assert.True(
+                overriddenMethod is not null && overriddenMethod.DeclaringType == typeof(DelegatingMcpServerResource),
+                $"DelegatingMcpServerResource does not override {baseMethod.Name} from McpServerResource.");
+        }
+    }
+}

tests/ModelContextProtocol.Tests/Server/DelegatingMcpServerToolTests.cs

@@ -0,0 +1,29 @@
+using ModelContextProtocol.Server;
+using System.Reflection;
+
+namespace ModelContextProtocol.Tests.Server;
+
+public class DelegatingMcpServerToolTests
+{
+    [Fact]
+    public void OverridesAllVirtualAndAbstractMembers()
+    {
+        MethodInfo[] baseMethods = typeof(McpServerTool).GetMethods(BindingFlags.Public | BindingFlags.Instance)
+            .Where(m => (m.IsVirtual || m.IsAbstract) && m.DeclaringType == typeof(McpServerTool))
+            .ToArray();
+
+        Assert.NotEmpty(baseMethods);
+
+        foreach (MethodInfo baseMethod in baseMethods)
+        {
+            MethodInfo? overriddenMethod = typeof(DelegatingMcpServerTool).GetMethod(
+                baseMethod.Name,
+                BindingFlags.Public | BindingFlags.Instance,
+                baseMethod.GetParameters().Select(p => p.ParameterType).ToArray());
+
+            Assert.True(
+                overriddenMethod is not null && overriddenMethod.DeclaringType == typeof(DelegatingMcpServerTool),
+                $"DelegatingMcpServerTool does not override {baseMethod.Name} from McpServerTool.");
+        }
+    }
+}

tests/ModelContextProtocol.Tests/Server/McpServerPromptTests.cs

@@ -547,26 +547,4 @@ public void SupportsPromptWithoutIcons()
 
         Assert.Null(prompt.ProtocolPrompt.Icons);
     }
-
-    [Fact]
-    public void DelegatingMcpServerPrompt_OverridesAllVirtualAndAbstractMembers()
-    {
-        MethodInfo[] baseMethods = typeof(McpServerPrompt).GetMethods(BindingFlags.Public | BindingFlags.Instance)
-            .Where(m => (m.IsVirtual || m.IsAbstract) && m.DeclaringType == typeof(McpServerPrompt))
-            .ToArray();
-
-        Assert.NotEmpty(baseMethods);
-
-        foreach (MethodInfo baseMethod in baseMethods)
-        {
-            MethodInfo? overriddenMethod = typeof(DelegatingMcpServerPrompt).GetMethod(
-                baseMethod.Name,
-                BindingFlags.Public | BindingFlags.Instance,
-                baseMethod.GetParameters().Select(p => p.ParameterType).ToArray());
-
-            Assert.True(
-                overriddenMethod is not null && overriddenMethod.DeclaringType == typeof(DelegatingMcpServerPrompt),
-                $"DelegatingMcpServerPrompt does not override {baseMethod.Name} from McpServerPrompt.");
-        }
-    }
 }

tests/ModelContextProtocol.Tests/Server/McpServerResourceTests.cs

@@ -734,28 +734,6 @@ public void SupportsResourceWithoutIcons()
         Assert.Null(resource.ProtocolResourceTemplate.Icons);
     }
 
-    [Fact]
-    public void DelegatingMcpServerResource_OverridesAllVirtualAndAbstractMembers()
-    {
-        MethodInfo[] baseMethods = typeof(McpServerResource).GetMethods(BindingFlags.Public | BindingFlags.Instance)
-            .Where(m => (m.IsVirtual || m.IsAbstract) && m.DeclaringType == typeof(McpServerResource))
-            .ToArray();
-
-        Assert.NotEmpty(baseMethods);
-
-        foreach (MethodInfo baseMethod in baseMethods)
-        {
-            MethodInfo? overriddenMethod = typeof(DelegatingMcpServerResource).GetMethod(
-                baseMethod.Name,
-                BindingFlags.Public | BindingFlags.Instance,
-                baseMethod.GetParameters().Select(p => p.ParameterType).ToArray());
-
-            Assert.True(
-                overriddenMethod is not null && overriddenMethod.DeclaringType == typeof(DelegatingMcpServerResource),
-                $"DelegatingMcpServerResource does not override {baseMethod.Name} from McpServerResource.");
-        }
-    }
-
     [JsonSourceGenerationOptions(PropertyNamingPolicy = JsonKnownNamingPolicy.CamelCase)]
     [JsonSerializable(typeof(DisposableResourceType))]
     [JsonSerializable(typeof(List<AIContent>))]

tests/ModelContextProtocol.Tests/Server/McpServerToolTests.cs

@@ -918,28 +918,6 @@ private static string SyncTool()
     [Description("Tool without return description.")]
     private static string ToolWithoutReturnDescription() => "result";
 
-    [Fact]
-    public void DelegatingMcpServerTool_OverridesAllVirtualAndAbstractMembers()
-    {
-        MethodInfo[] baseMethods = typeof(McpServerTool).GetMethods(BindingFlags.Public | BindingFlags.Instance)
-            .Where(m => (m.IsVirtual || m.IsAbstract) && m.DeclaringType == typeof(McpServerTool))
-            .ToArray();
-
-        Assert.NotEmpty(baseMethods);
-
-        foreach (MethodInfo baseMethod in baseMethods)
-        {
-            MethodInfo? overriddenMethod = typeof(DelegatingMcpServerTool).GetMethod(
-                baseMethod.Name,
-                BindingFlags.Public | BindingFlags.Instance,
-                baseMethod.GetParameters().Select(p => p.ParameterType).ToArray());
-
-            Assert.True(
-                overriddenMethod is not null && overriddenMethod.DeclaringType == typeof(DelegatingMcpServerTool),
-                $"DelegatingMcpServerTool does not override {baseMethod.Name} from McpServerTool.");
-        }
-    }
-
     [JsonSourceGenerationOptions(PropertyNamingPolicy = JsonKnownNamingPolicy.CamelCase)]
     [JsonSerializable(typeof(JsonNode))]
     [JsonSerializable(typeof(DisposableToolType))]