mirror of
https://github.com/microsoft/agent-framework.git
synced 2026-06-16 21:04:09 +08:00
628bb1af48
* Update Foundry Responses as ChatClientAgent * Migrate obsolete AzureAI integration tests to versioned agent pattern Replace obsolete CreateAIAgentAsync/GetAIAgentAsync calls with Agents.CreateAgentVersionAsync() + AsAIAgent(AgentVersion) in all AzureAI integration tests. - Rename AIProjectClient* test files to FoundryVersionedAgent* - Register AIFunction tools in PromptAgentDefinition.Tools for server-side visibility via AsOpenAIResponseTool() - Skip structured output tests (AzureAIProjectChatClient clears ResponseFormat for versioned agents) - Remove all [Obsolete] attributes and #pragma warning disable CS0618 * Merge FoundryMemory package into AzureAI under Memory/ folder Move all FoundryMemory source, unit tests, and integration tests into the Microsoft.Agents.AI.AzureAI package. Change namespace from Microsoft.Agents.AI.FoundryMemory to Microsoft.Agents.AI.AzureAI. - Add [Experimental] to FoundryMemoryProviderOptions and Scope - Rename internal AIProjectClientExtensions to MemoryStoreExtensions - Update AzureAI .csproj with Compliance.Abstractions, Redaction - Remove FoundryMemory from solution and release filter - Update sample to reference AzureAI instead of FoundryMemory - Delete old Microsoft.Agents.AI.FoundryMemory project and tests * Add EnsureMemoryStoreCreatedAsync and memory existence checks to integration tests - Ensure memory store is created before testing memory operations - Add AZURE_AI_EMBEDDING_DEPLOYMENT_NAME config setting - Assert memories exist in store via SearchMemoriesAsync before cleanup - Verify scope isolation with direct memory store queries * Fix and rename AzureAI unit tests for RAPI vs Versioned clarity - Rename AsAIAgentAsync_* to AsAIAgent_* (drop Async from method group) - Add _Rapi_ prefix to non-versioned (Responses API) tests - Add _Versioned_ prefix to versioned agent tests where needed - Fix RAPI tests: assert GetService<AIProjectClient>() is null - Fix Versioned tests: assert IsType<FoundryAgent> and GetService<AIProjectClient>() returns the client instance - Fix UserAgent header tests: proper HTTP handler routing - Fix ChatClient_UsesDefaultConversationIdAsync test setup - All 153 unit tests pass with 0 failures * Rename Microsoft.Agents.AI.AzureAI to Microsoft.Agents.AI.Foundry Rename the project, namespace, folder, and all references from Microsoft.Agents.AI.AzureAI to Microsoft.Agents.AI.Foundry. Also rename Workflows.Declarative.AzureAI to .Foundry. - Rename src, unit test, integration test, and workflow folders - Update namespaces in all source and test .cs files - Update ProjectReferences in ~47 sample and test .csproj files - Update solution files (.slnx, .slnf) - Update sample using statements - Update READMEs, SKILL.md, ADRs in docs/ - Disable package validation baseline for renamed packages - Fix UTF-8 BOM encoding on all affected .cs files - AzureAI.Persistent left completely unchanged * Fix format: remove ImplicitUsings, add explicit usings, fix BOM encoding - Remove ImplicitUsings=enable from Foundry csproj to resolve IDE0005 on shared ReplacingRedactor.cs - Add explicit System usings to all source files that relied on them - Sort usings alphabetically per editorconfig rules - Fix UTF-8 BOM on 12 sample Program.cs files - Rename Azure AI Foundry Agents to Microsoft Foundry Agents in docs
170 lines
7.0 KiB
C#
170 lines
7.0 KiB
C#
// Copyright (c) Microsoft. All rights reserved.
|
|
|
|
using System;
|
|
using System.Reflection;
|
|
using Azure.AI.Extensions.OpenAI;
|
|
using Microsoft.Extensions.AI;
|
|
using OpenAI.Responses;
|
|
|
|
namespace Microsoft.Agents.AI.Foundry.UnitTests;
|
|
|
|
/// <summary>
|
|
/// Unit tests for the <see cref="ProjectResponsesClientExtensions"/> class.
|
|
/// </summary>
|
|
public sealed class ProjectResponsesClientExtensionsTests
|
|
{
|
|
private static ProjectResponsesClient CreateTestClient()
|
|
{
|
|
return new ProjectResponsesClient(new FakeAuthenticationTokenProvider());
|
|
}
|
|
|
|
/// <summary>
|
|
/// Verify that AsIChatClientWithStoredOutputDisabled throws ArgumentNullException when client is null.
|
|
/// </summary>
|
|
[Fact]
|
|
public void AsIChatClientWithStoredOutputDisabled_WithNullClient_ThrowsArgumentNullException()
|
|
{
|
|
// Act & Assert
|
|
var exception = Assert.Throws<ArgumentNullException>(() =>
|
|
((ProjectResponsesClient)null!).AsIChatClientWithStoredOutputDisabled());
|
|
|
|
Assert.Equal("responseClient", exception.ParamName);
|
|
}
|
|
|
|
/// <summary>
|
|
/// Verify that AsIChatClientWithStoredOutputDisabled wraps the original ProjectResponsesClient,
|
|
/// which remains accessible via the service chain.
|
|
/// </summary>
|
|
[Fact]
|
|
public void AsIChatClientWithStoredOutputDisabled_InnerResponsesClientIsAccessible()
|
|
{
|
|
// Arrange
|
|
var responseClient = CreateTestClient();
|
|
|
|
// Act
|
|
var chatClient = responseClient.AsIChatClientWithStoredOutputDisabled();
|
|
|
|
// Assert - the inner ProjectResponsesClient should be accessible via GetService
|
|
var innerClient = chatClient.GetService<ResponsesClient>();
|
|
Assert.NotNull(innerClient);
|
|
Assert.Same(responseClient, innerClient);
|
|
}
|
|
|
|
/// <summary>
|
|
/// Verify that AsIChatClientWithStoredOutputDisabled with includeReasoningEncryptedContent false
|
|
/// wraps the original ProjectResponsesClient, which remains accessible via the service chain.
|
|
/// </summary>
|
|
[Fact]
|
|
public void AsIChatClientWithStoredOutputDisabled_WithIncludeReasoningFalse_InnerResponsesClientIsAccessible()
|
|
{
|
|
// Arrange
|
|
var responseClient = CreateTestClient();
|
|
|
|
// Act
|
|
var chatClient = responseClient.AsIChatClientWithStoredOutputDisabled(includeReasoningEncryptedContent: false);
|
|
|
|
// Assert - the inner ProjectResponsesClient should be accessible via GetService
|
|
var innerClient = chatClient.GetService<ResponsesClient>();
|
|
Assert.NotNull(innerClient);
|
|
Assert.Same(responseClient, innerClient);
|
|
}
|
|
|
|
/// <summary>
|
|
/// Verify that AsIChatClientWithStoredOutputDisabled with default parameter (includeReasoningEncryptedContent = true)
|
|
/// configures StoredOutputEnabled to false and includes ReasoningEncryptedContent in IncludedProperties.
|
|
/// </summary>
|
|
[Fact]
|
|
public void AsIChatClientWithStoredOutputDisabled_Default_ConfiguresStoredOutputDisabledWithReasoningEncryptedContent()
|
|
{
|
|
// Arrange
|
|
var responseClient = CreateTestClient();
|
|
|
|
// Act
|
|
var chatClient = responseClient.AsIChatClientWithStoredOutputDisabled();
|
|
|
|
// Assert
|
|
var createResponseOptions = GetCreateResponseOptionsFromPipeline(chatClient);
|
|
Assert.NotNull(createResponseOptions);
|
|
Assert.False(createResponseOptions.StoredOutputEnabled);
|
|
Assert.Contains(IncludedResponseProperty.ReasoningEncryptedContent, createResponseOptions.IncludedProperties);
|
|
}
|
|
|
|
/// <summary>
|
|
/// Verify that AsIChatClientWithStoredOutputDisabled with includeReasoningEncryptedContent explicitly set to true
|
|
/// configures StoredOutputEnabled to false and includes ReasoningEncryptedContent in IncludedProperties.
|
|
/// </summary>
|
|
[Fact]
|
|
public void AsIChatClientWithStoredOutputDisabled_WithIncludeReasoningTrue_ConfiguresStoredOutputDisabledWithReasoningEncryptedContent()
|
|
{
|
|
// Arrange
|
|
var responseClient = CreateTestClient();
|
|
|
|
// Act
|
|
var chatClient = responseClient.AsIChatClientWithStoredOutputDisabled(includeReasoningEncryptedContent: true);
|
|
|
|
// Assert
|
|
var createResponseOptions = GetCreateResponseOptionsFromPipeline(chatClient);
|
|
Assert.NotNull(createResponseOptions);
|
|
Assert.False(createResponseOptions.StoredOutputEnabled);
|
|
Assert.Contains(IncludedResponseProperty.ReasoningEncryptedContent, createResponseOptions.IncludedProperties);
|
|
}
|
|
|
|
/// <summary>
|
|
/// Verify that AsIChatClientWithStoredOutputDisabled with includeReasoningEncryptedContent set to false
|
|
/// configures StoredOutputEnabled to false and does not include ReasoningEncryptedContent in IncludedProperties.
|
|
/// </summary>
|
|
[Fact]
|
|
public void AsIChatClientWithStoredOutputDisabled_WithIncludeReasoningFalse_ConfiguresStoredOutputDisabledWithoutReasoningEncryptedContent()
|
|
{
|
|
// Arrange
|
|
var responseClient = CreateTestClient();
|
|
|
|
// Act
|
|
var chatClient = responseClient.AsIChatClientWithStoredOutputDisabled(includeReasoningEncryptedContent: false);
|
|
|
|
// Assert
|
|
var createResponseOptions = GetCreateResponseOptionsFromPipeline(chatClient);
|
|
Assert.NotNull(createResponseOptions);
|
|
Assert.False(createResponseOptions.StoredOutputEnabled);
|
|
Assert.DoesNotContain(IncludedResponseProperty.ReasoningEncryptedContent, createResponseOptions.IncludedProperties);
|
|
}
|
|
|
|
/// <summary>
|
|
/// Verify that AsIChatClientWithStoredOutputDisabled works with an optional deployment name.
|
|
/// </summary>
|
|
[Fact]
|
|
public void AsIChatClientWithStoredOutputDisabled_WithDeploymentName_ConfiguresStoredOutputDisabled()
|
|
{
|
|
// Arrange
|
|
var responseClient = CreateTestClient();
|
|
|
|
// Act
|
|
var chatClient = responseClient.AsIChatClientWithStoredOutputDisabled(deploymentName: "my-deployment");
|
|
|
|
// Assert
|
|
var createResponseOptions = GetCreateResponseOptionsFromPipeline(chatClient);
|
|
Assert.NotNull(createResponseOptions);
|
|
Assert.False(createResponseOptions.StoredOutputEnabled);
|
|
Assert.Contains(IncludedResponseProperty.ReasoningEncryptedContent, createResponseOptions.IncludedProperties);
|
|
}
|
|
|
|
/// <summary>
|
|
/// Extracts the <see cref="CreateResponseOptions"/> produced by the ConfigureOptions pipeline
|
|
/// by using reflection to access the configure action and invoking it on a test <see cref="ChatOptions"/>.
|
|
/// </summary>
|
|
private static CreateResponseOptions? GetCreateResponseOptionsFromPipeline(IChatClient chatClient)
|
|
{
|
|
var configureField = chatClient.GetType().GetField("_configureOptions", BindingFlags.NonPublic | BindingFlags.Instance);
|
|
Assert.NotNull(configureField);
|
|
|
|
var configureAction = configureField.GetValue(chatClient) as Action<ChatOptions>;
|
|
Assert.NotNull(configureAction);
|
|
|
|
var options = new ChatOptions();
|
|
configureAction(options);
|
|
|
|
Assert.NotNull(options.RawRepresentationFactory);
|
|
return options.RawRepresentationFactory(chatClient) as CreateResponseOptions;
|
|
}
|
|
}
|