Combine type qualification tests into a single Theory

Replace Generator_WithEnumFromDifferentNamespace_GeneratesFullyQualifiedTypeName
and Generator_WithUnqualifiedTypeInSource_GeneratesFullyQualifiedTypeName with
a single Generator_WithTypeFromDifferentNamespace_GeneratesFullyQualifiedTypeName
Theory that takes a bool for whether to fully-qualify types in source.

Co-authored-by: stephentoub <2642209+stephentoub@users.noreply.github.com>

Author: Copilot

tests/ModelContextProtocol.Analyzers.Tests/XmlToDescriptionGeneratorTests.cs

@@ -1719,75 +1719,24 @@ partial class TestTools
         AssertGeneratedSourceEquals(expected, result.GeneratedSources[0].SourceText.ToString());
     }
 
-    [Fact]
-    public void Generator_WithEnumFromDifferentNamespace_GeneratesFullyQualifiedTypeName()
+    [Theory]
+    [InlineData(true)]
+    [InlineData(false)]
+    public void Generator_WithTypeFromDifferentNamespace_GeneratesFullyQualifiedTypeName(bool useFullyQualifiedTypesInSource)
     {
-        // This test validates the fix for the issue where parameter types from different namespaces
-        // caused build failures because the generator was emitting unqualified type names.
-        var result = RunGenerator("""
-            using ModelContextProtocol.Server;
-            using System.ComponentModel;
-            using System.Threading.Tasks;
-
-            namespace MyApp.Actions
-            {
-                public enum MyAction
-                {
-                    One,
-                    Two
-                }
-            }
+        // This test validates that regardless of whether the source code uses fully qualified
+        // or unqualified type names, the generator always emits fully qualified type names
+        // with global:: prefix. This fixes the issue where parameter types from different
+        // namespaces caused build failures.
+        string usingDirective = useFullyQualifiedTypesInSource ? "" : "using MyApp.Actions;";
+        string returnType = useFullyQualifiedTypesInSource ? "System.Threading.Tasks.Task<string>" : "Task<string>";
+        string parameterType = useFullyQualifiedTypesInSource ? "MyApp.Actions.MyAction" : "MyAction";
 
-            namespace MyApp
-            {
-                [McpServerToolType]
-                public sealed partial class Tools
-                {
-                    /// <summary>Do a thing based on an action.</summary>
-                    /// <param name="action">The action to perform.</param>
-                    [McpServerTool]
-                    public async partial Task<string> DoThing(MyApp.Actions.MyAction action)
-                        => await Task.FromResult("ok");
-                }
-            }
-            """);
-
-        Assert.True(result.Success);
-        Assert.Single(result.GeneratedSources);
-
-        var expected = $$"""
-            // <auto-generated/>
-            // ModelContextProtocol.Analyzers {{typeof(XmlToDescriptionGenerator).Assembly.GetName().Version}}
-
-            #pragma warning disable
-
-            using System.ComponentModel;
-            using ModelContextProtocol.Server;
-
-            namespace MyApp
-            {
-                partial class Tools
-                {
-                    [Description("Do a thing based on an action.")]
-                    public partial global::System.Threading.Tasks.Task<string> DoThing([Description("The action to perform.")] global::MyApp.Actions.MyAction action);
-                }
-            }
-            """;
-
-        AssertGeneratedSourceEquals(expected, result.GeneratedSources[0].SourceText.ToString());
-    }
-
-    [Fact]
-    public void Generator_WithUnqualifiedTypeInSource_GeneratesFullyQualifiedTypeName()
-    {
-        // This test validates that even when the source code uses an unqualified type name
-        // (via a using directive), the generator still emits fully qualified type names.
-        // This is the exact scenario from the original issue.
-        var result = RunGenerator("""
+        var result = RunGenerator($$"""
             using ModelContextProtocol.Server;
             using System.ComponentModel;
             using System.Threading.Tasks;
-            using MyApp.Actions;
+            {{usingDirective}}
 
             namespace MyApp.Actions
             {
@@ -1806,7 +1755,7 @@ public sealed partial class Tools
                     /// <summary>Do a thing based on an action.</summary>
                     /// <param name="action">The action to perform.</param>
                     [McpServerTool]
-                    public async partial Task<string> DoThing(MyAction action)
+                    public async partial {{returnType}} DoThing({{parameterType}} action)
                         => await Task.FromResult("ok");
                 }
             }
@@ -1815,8 +1764,8 @@ public async partial Task<string> DoThing(MyAction action)
         Assert.True(result.Success);
         Assert.Single(result.GeneratedSources);
 
-        // Even though the source uses unqualified "MyAction", the generated code should use
-        // the fully qualified "global::MyApp.Actions.MyAction"
+        // Regardless of source qualification, generated code should always use
+        // fully qualified type names with global:: prefix
         var expected = $$"""
             // <auto-generated/>
             // ModelContextProtocol.Analyzers {{typeof(XmlToDescriptionGenerator).Assembly.GetName().Version}}