Fix 115 xUnit test warnings in MEAI.Abstractions.Tests (#7369)

- Fix duplicate test case IDs caused by unused gapBeginningEnd loop variable
- Suppress non-serializable data warnings with DisableDiscoveryEnumeration

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: jozkee

test/Libraries/Microsoft.Extensions.AI.Abstractions.Tests/ChatCompletion/ChatResponseUpdateExtensionsTests.cs

@@ -611,7 +611,7 @@ public static IEnumerable<object[]> ToChatResponse_Coalescing_VariousSequenceAnd
                     {
                         foreach (bool gapBeginningEnd in new[] { false, true })
                         {
-                            yield return new object[] { useAsync, numSequences, sequenceLength, gapLength, false };
+                            yield return new object[] { useAsync, numSequences, sequenceLength, gapLength, gapBeginningEnd };
                         }
                     }
                 }

test/Libraries/Microsoft.Extensions.AI.Abstractions.Tests/Contents/ToolApprovalRequestContentTests.cs

@@ -28,7 +28,7 @@ public void Constructor_InvalidArguments_Throws()
     };
 
     [Theory]
-    [MemberData(nameof(ToolCallContentInstances))]
+    [MemberData(nameof(ToolCallContentInstances), DisableDiscoveryEnumeration = true)]
     public void Constructor_Roundtrips(ToolCallContent toolCall)
     {
         string id = "req-1";
@@ -39,7 +39,7 @@ public void Constructor_Roundtrips(ToolCallContent toolCall)
     }
 
     [Theory]
-    [MemberData(nameof(ToolCallContentInstances))]
+    [MemberData(nameof(ToolCallContentInstances), DisableDiscoveryEnumeration = true)]
     public void CreateResponse_ReturnsExpectedResponse(ToolCallContent toolCall)
     {
         string id = "req-1";
@@ -76,7 +76,7 @@ public void CreateResponse_WithReason_ReturnsExpectedResponse(bool approved, str
     }
 
     [Theory]
-    [MemberData(nameof(ToolCallContentInstances))]
+    [MemberData(nameof(ToolCallContentInstances), DisableDiscoveryEnumeration = true)]
     public void Serialization_Roundtrips(ToolCallContent toolCall)
     {
         var content = new ToolApprovalRequestContent("request123", toolCall);

test/Libraries/Microsoft.Extensions.AI.Abstractions.Tests/Contents/ToolApprovalResponseContentTests.cs

@@ -28,7 +28,7 @@ public void Constructor_InvalidArguments_Throws()
     };
 
     [Theory]
-    [MemberData(nameof(ToolCallContentInstances))]
+    [MemberData(nameof(ToolCallContentInstances), DisableDiscoveryEnumeration = true)]
     public void Constructor_Roundtrips(ToolCallContent toolCall)
     {
         ToolApprovalResponseContent content = new("req-1", true, toolCall);
@@ -45,7 +45,7 @@ public void Constructor_Roundtrips(ToolCallContent toolCall)
     }
 
     [Theory]
-    [MemberData(nameof(ToolCallContentInstances))]
+    [MemberData(nameof(ToolCallContentInstances), DisableDiscoveryEnumeration = true)]
     public void Serialization_Roundtrips(ToolCallContent toolCall)
     {
         var content = new ToolApprovalResponseContent("request123", true, toolCall)

test/Libraries/Microsoft.Extensions.AI.Abstractions.Tests/SpeechToText/SpeechToTextResponseUpdateExtensionsTests.cs

@@ -24,7 +24,7 @@ public static IEnumerable<object[]> ToSpeechToTextResponse_Coalescing_VariousSeq
                     {
                         foreach (bool gapBeginningEnd in new[] { false, true })
                         {
-                            yield return new object[] { useAsync, numSequences, sequenceLength, gapLength, false };
+                            yield return new object[] { useAsync, numSequences, sequenceLength, gapLength, gapBeginningEnd };
                         }
                     }
                 }

test/Libraries/Microsoft.Extensions.AI.Abstractions.Tests/Utilities/AIJsonUtilitiesTests.cs

@@ -588,7 +588,7 @@ public static void CreateJsonSchema_CanBeBoolean()
     }
 
     [Theory]
-    [MemberData(nameof(TestTypes.GetTestDataUsingAllValues), MemberType = typeof(TestTypes))]
+    [MemberData(nameof(TestTypes.GetTestDataUsingAllValues), MemberType = typeof(TestTypes), DisableDiscoveryEnumeration = true)]
     public static void CreateJsonSchema_ValidateWithTestData(ITestData testData)
     {
         // Stress tests the schema generation method using types from the JsonSchemaExporter test battery.
@@ -1730,7 +1730,7 @@ public static void TransformJsonSchema_MoveDefaultKeywordToDescription()
     }
 
     [Theory]
-    [MemberData(nameof(TestTypes.GetTestDataUsingAllValues), MemberType = typeof(TestTypes))]
+    [MemberData(nameof(TestTypes.GetTestDataUsingAllValues), MemberType = typeof(TestTypes), DisableDiscoveryEnumeration = true)]
     public static void TransformJsonSchema_ValidateWithTestData(ITestData testData)
     {
         // Stress tests the schema generation method using types from the JsonSchemaExporter test battery.