Fix CI failures by resetting InvocationRequired in test helper methods

The tests reuse FunctionCallContent objects across multiple test iterations. When we set InvocationRequired=false during processing, subsequent test iterations see them as already processed. Fixed by resetting InvocationRequired=true at the beginning of InvokeAndAssertAsync and InvokeAndAssertStreamingAsync helper methods.

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

Author: Copilot

test/Libraries/Microsoft.Extensions.AI.Tests/ChatCompletion/FunctionInvokingChatClientTests.cs

@@ -136,6 +136,7 @@ public async Task SupportsToolsProvidedByAdditionalTools(bool provideOptions)
 
         await InvokeAndAssertAsync(options, plan, configurePipeline: configure);
 
+        ResetPlanFunctionCallStates(plan);
         await InvokeAndAssertStreamingAsync(options, plan, configurePipeline: configure);
     }
 
@@ -169,6 +170,7 @@ public async Task PrefersToolsProvidedByChatOptions()
 
         await InvokeAndAssertAsync(options, plan, configurePipeline: configure);
 
+        ResetPlanFunctionCallStates(plan);
         await InvokeAndAssertStreamingAsync(options, plan, configurePipeline: configure);
     }
 
@@ -219,6 +221,7 @@ public async Task SupportsMultipleFunctionCallsPerRequestAsync(bool concurrentIn
 
         await InvokeAndAssertAsync(options, plan, configurePipeline: configure);
 
+        ResetPlanFunctionCallStates(plan);
         await InvokeAndAssertStreamingAsync(options, plan, configurePipeline: configure);
     }
 
@@ -267,6 +270,7 @@ public async Task ParallelFunctionCallsMayBeInvokedConcurrentlyAsync()
 
         await InvokeAndAssertAsync(options, plan, configurePipeline: configure);
 
+        ResetPlanFunctionCallStates(plan);
         await InvokeAndAssertStreamingAsync(options, plan, configurePipeline: configure);
     }
 
@@ -308,6 +312,7 @@ public async Task ConcurrentInvocationOfParallelCallsDisabledByDefaultAsync()
 
         await InvokeAndAssertAsync(options, plan);
 
+        ResetPlanFunctionCallStates(plan);
         await InvokeAndAssertStreamingAsync(options, plan);
     }
 
@@ -351,6 +356,7 @@ public async Task FunctionInvokerDelegateOverridesHandlingAsync()
 
         await InvokeAndAssertAsync(options, plan, configurePipeline: configure);
 
+        ResetPlanFunctionCallStates(plan);
         await InvokeAndAssertStreamingAsync(options, plan, configurePipeline: configure);
     }
 
@@ -562,6 +568,7 @@ public async Task KeepsFunctionCallingContent()
 #pragma warning disable SA1005, S125
         Validate(await InvokeAndAssertAsync(options, plan));
 
+        ResetPlanFunctionCallStates(plan);
         Validate(await InvokeAndAssertStreamingAsync(options, plan));
 
         static void Validate(List<ChatMessage> finalChat)
@@ -597,6 +604,7 @@ public async Task ExceptionDetailsOnlyReportedWhenRequestedAsync(bool detailedEr
 
         await InvokeAndAssertAsync(options, plan, configurePipeline: configure);
 
+        ResetPlanFunctionCallStates(plan);
         await InvokeAndAssertStreamingAsync(options, plan, configurePipeline: configure);
     }
 
@@ -1077,6 +1085,7 @@ public async Task FunctionInvocations_InvokedOnOriginalSynchronizationContext()
             .UseFunctionInvocation(configure: c => { c.AllowConcurrentInvocation = true; c.IncludeDetailedErrors = true; });
 
         await InvokeAndAssertAsync(options, plan, configurePipeline: configurePipeline);
+        ResetPlanFunctionCallStates(plan);
         await InvokeAndAssertStreamingAsync(options, plan, configurePipeline: configurePipeline);
     }
 
@@ -1110,6 +1119,7 @@ public async Task TerminateOnUnknownCalls_ControlsBehaviorForUnknownFunctions(bo
             ];
 
             await InvokeAndAssertAsync(options, planForContinue, configurePipeline: configure);
+            ResetPlanFunctionCallStates(planForContinue);
             await InvokeAndAssertStreamingAsync(options, planForContinue, configurePipeline: configure);
         }
         else
@@ -1621,6 +1631,9 @@ private static async Task<List<ChatMessage>> InvokeAndAssertAsync(
     {
         Assert.NotEmpty(plan);
 
+        // Reset InvocationRequired for all FunctionCallContent in the plan to allow reuse
+        ResetPlanFunctionCallStates(plan);
+
         configurePipeline ??= static b => b.UseFunctionInvocation();
 
         using CancellationTokenSource cts = new();
@@ -1691,6 +1704,9 @@ private static async Task<List<ChatMessage>> InvokeAndAssertStreamingAsync(
     {
         Assert.NotEmpty(plan);
 
+        // Reset InvocationRequired for all FunctionCallContent in the plan to allow reuse
+        ResetPlanFunctionCallStates(plan);
+
         configurePipeline ??= static b => b.UseFunctionInvocation();
 
         using CancellationTokenSource cts = new();
@@ -1731,4 +1747,22 @@ private static async IAsyncEnumerable<T> YieldAsync<T>(params IEnumerable<T> ite
             yield return item;
         }
     }
+
+    /// <summary>
+    /// Resets InvocationRequired to true for all FunctionCallContent in the plan.
+    /// This is needed when reusing a plan across multiple test invocations.
+    /// </summary>
+    private static void ResetPlanFunctionCallStates(List<ChatMessage> plan)
+    {
+        foreach (var message in plan)
+        {
+            foreach (var content in message.Contents)
+            {
+                if (content is FunctionCallContent fcc)
+                {
+                    fcc.InvocationRequired = true;
+                }
+            }
+        }
+    }
 }