mirror of
https://github.com/microsoft/agent-framework.git
synced 2026-06-16 21:04:09 +08:00
3168eb4870
* .NET: [BREAKING] Add session statebag to use for state storage instead of inside providers (#3737) * Add a StateBag to AgentSession and pass Agent and AgentSession to AIContextProvider and ChatHistoryProviders * Convert all AIContextProviders to use the statebag * Update InMemoryChatHistoryProvider to use StateBag * Update Comsos and Workflow ChatHistoryProviders * Update 3rd party chat history storage sample. * Remove serialize method from providers * Replacing provider factories with properties * Remove Providers from Session and flatten state bag serialization * Update samples to use getservice on agent * Updated additional session types to serialize statebag * Fix regression * Address PR comments * Address PR comments. * Fix formatting * Fix unit tests * Remove InMemoryAgentSession since it is not required anymore. * Address PR comments * Convert sessions for A2AAgent, ChatClientAgent, CopilotStudioAgent and GithubCopilotAgent to use regular json serialization. * Fix durable agent session jso usgae * Add jso to InMemory and Workflow ChatHistoryProviders * Update InMemoryChatHistoryProvider to use an options class for it's many optional settings. * Apply suggestions from code review Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Address PR feedback * Fix verification bug. * Improve state bag thread safety * Address PR comments and fix unit tests * Address PR comments * Fix unit test --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Add a public StateKey property to providers (#3810) * .NET: [BREAKING] Update providers in such a way that they can participate in a pipeline (#3846) * Make providers pipeline capable * Fix unit tests * Move source stamping to providers from base class * Also update samples. * Address PR comments * Rename AsAgentRequestMessageSourcedMessage to WithAgentRequestMessageSource * .NET: [BREAKING] Add consistent message filtering to all providers. (#3851) * Add consistent message filtering to all providers. * Remove old chat history filtering classes * Fix merge issues * Fix unit test * Enforce non-nullable property * Fix merging bug and make troubleshooting source info easier by adding tostring implementation * .NET: [BREAKING] Add support for multiple AIContextProviders on a ChatClientAgent (#3863) * Add support for multiple AIContextProviders on a ChatClientAgent * Address PR comments and fix tests * Address PR comments. * .NET: [BREAKING]Delay AIContext Materialization until the end of the pipeline is reached. (#3883) * Delay AIContext Materialization until the end of the pipeline is reached. * Address PR comments. * Address PR comments * Modify InvokedContext to be immutable (#3888) * .NET: Address Feedback on StateBag feature branch PR (#3910) * Address Feedback on statebag feature branch PR * Update dotnet/src/Microsoft.Agents.AI.DurableTask/CHANGELOG.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Address PR comments --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
286 lines
8.9 KiB
C#
286 lines
8.9 KiB
C#
// Copyright (c) Microsoft. All rights reserved.
|
|
|
|
using System;
|
|
using System.Collections.Generic;
|
|
using System.Linq;
|
|
using System.Threading;
|
|
using System.Threading.Tasks;
|
|
using Microsoft.Extensions.AI;
|
|
using Moq;
|
|
|
|
namespace Microsoft.Agents.AI.Abstractions.UnitTests;
|
|
|
|
/// <summary>
|
|
/// Contains tests for the <see cref="ChatHistoryProvider"/> class.
|
|
/// </summary>
|
|
public class ChatHistoryProviderTests
|
|
{
|
|
private static readonly AIAgent s_mockAgent = new Mock<AIAgent>().Object;
|
|
private static readonly AgentSession s_mockSession = new Mock<AgentSession>().Object;
|
|
|
|
#region GetService Method Tests
|
|
|
|
[Fact]
|
|
public void GetService_RequestingExactProviderType_ReturnsProvider()
|
|
{
|
|
var provider = new TestChatHistoryProvider();
|
|
var result = provider.GetService(typeof(TestChatHistoryProvider));
|
|
Assert.NotNull(result);
|
|
Assert.Same(provider, result);
|
|
}
|
|
|
|
[Fact]
|
|
public void GetService_RequestingBaseProviderType_ReturnsProvider()
|
|
{
|
|
var provider = new TestChatHistoryProvider();
|
|
var result = provider.GetService(typeof(ChatHistoryProvider));
|
|
Assert.NotNull(result);
|
|
Assert.Same(provider, result);
|
|
}
|
|
|
|
[Fact]
|
|
public void GetService_RequestingUnrelatedType_ReturnsNull()
|
|
{
|
|
var provider = new TestChatHistoryProvider();
|
|
var result = provider.GetService(typeof(string));
|
|
Assert.Null(result);
|
|
}
|
|
|
|
[Fact]
|
|
public void GetService_WithServiceKey_ReturnsNull()
|
|
{
|
|
var provider = new TestChatHistoryProvider();
|
|
var result = provider.GetService(typeof(TestChatHistoryProvider), "some-key");
|
|
Assert.Null(result);
|
|
}
|
|
|
|
[Fact]
|
|
public void GetService_WithNullServiceType_ThrowsArgumentNullException()
|
|
{
|
|
var provider = new TestChatHistoryProvider();
|
|
Assert.Throws<ArgumentNullException>(() => provider.GetService(null!));
|
|
}
|
|
|
|
[Fact]
|
|
public void GetService_Generic_ReturnsCorrectType()
|
|
{
|
|
var provider = new TestChatHistoryProvider();
|
|
var result = provider.GetService<TestChatHistoryProvider>();
|
|
Assert.NotNull(result);
|
|
Assert.Same(provider, result);
|
|
}
|
|
|
|
[Fact]
|
|
public void GetService_Generic_ReturnsNullForUnrelatedType()
|
|
{
|
|
var provider = new TestChatHistoryProvider();
|
|
var result = provider.GetService<string>();
|
|
Assert.Null(result);
|
|
}
|
|
|
|
#endregion
|
|
|
|
#region InvokingContext Tests
|
|
|
|
[Fact]
|
|
public void InvokingContext_Constructor_ThrowsForNullMessages()
|
|
{
|
|
// Arrange & Act & Assert
|
|
Assert.Throws<ArgumentNullException>(() => new ChatHistoryProvider.InvokingContext(s_mockAgent, s_mockSession, null!));
|
|
}
|
|
|
|
[Fact]
|
|
public void InvokingContext_RequestMessages_SetterThrowsForNull()
|
|
{
|
|
// Arrange
|
|
var messages = new List<ChatMessage> { new(ChatRole.User, "Hello") };
|
|
var context = new ChatHistoryProvider.InvokingContext(s_mockAgent, s_mockSession, messages);
|
|
|
|
// Act & Assert
|
|
Assert.Throws<ArgumentNullException>(() => context.RequestMessages = null!);
|
|
}
|
|
|
|
[Fact]
|
|
public void InvokingContext_RequestMessages_SetterRoundtrips()
|
|
{
|
|
// Arrange
|
|
var initialMessages = new List<ChatMessage> { new(ChatRole.User, "Hello") };
|
|
var newMessages = new List<ChatMessage> { new(ChatRole.User, "New message") };
|
|
var context = new ChatHistoryProvider.InvokingContext(s_mockAgent, s_mockSession, initialMessages);
|
|
|
|
// Act
|
|
context.RequestMessages = newMessages;
|
|
|
|
// Assert
|
|
Assert.Same(newMessages, context.RequestMessages);
|
|
}
|
|
|
|
[Fact]
|
|
public void InvokingContext_Agent_ReturnsConstructorValue()
|
|
{
|
|
// Arrange
|
|
var messages = new List<ChatMessage> { new(ChatRole.User, "Hello") };
|
|
|
|
// Act
|
|
var context = new ChatHistoryProvider.InvokingContext(s_mockAgent, s_mockSession, messages);
|
|
|
|
// Assert
|
|
Assert.Same(s_mockAgent, context.Agent);
|
|
}
|
|
|
|
[Fact]
|
|
public void InvokingContext_Session_ReturnsConstructorValue()
|
|
{
|
|
// Arrange
|
|
var messages = new List<ChatMessage> { new(ChatRole.User, "Hello") };
|
|
|
|
// Act
|
|
var context = new ChatHistoryProvider.InvokingContext(s_mockAgent, s_mockSession, messages);
|
|
|
|
// Assert
|
|
Assert.Same(s_mockSession, context.Session);
|
|
}
|
|
|
|
[Fact]
|
|
public void InvokingContext_Session_CanBeNull()
|
|
{
|
|
// Arrange
|
|
var messages = new List<ChatMessage> { new(ChatRole.User, "Hello") };
|
|
|
|
// Act
|
|
var context = new ChatHistoryProvider.InvokingContext(s_mockAgent, null, messages);
|
|
|
|
// Assert
|
|
Assert.Null(context.Session);
|
|
}
|
|
|
|
[Fact]
|
|
public void InvokingContext_Constructor_ThrowsForNullAgent()
|
|
{
|
|
// Arrange
|
|
var messages = new List<ChatMessage> { new(ChatRole.User, "Hello") };
|
|
|
|
// Act & Assert
|
|
Assert.Throws<ArgumentNullException>(() => new ChatHistoryProvider.InvokingContext(null!, s_mockSession, messages));
|
|
}
|
|
|
|
#endregion
|
|
|
|
#region InvokedContext Tests
|
|
|
|
[Fact]
|
|
public void InvokedContext_Constructor_ThrowsForNullRequestMessages()
|
|
{
|
|
// Arrange & Act & Assert
|
|
Assert.Throws<ArgumentNullException>(() => new ChatHistoryProvider.InvokedContext(s_mockAgent, s_mockSession, null!, []));
|
|
}
|
|
|
|
[Fact]
|
|
public void InvokedContext_ResponseMessages_Roundtrips()
|
|
{
|
|
// Arrange
|
|
var requestMessages = new List<ChatMessage> { new(ChatRole.User, "Hello") };
|
|
var responseMessages = new List<ChatMessage> { new(ChatRole.Assistant, "Response message") };
|
|
|
|
// Act
|
|
var context = new ChatHistoryProvider.InvokedContext(s_mockAgent, s_mockSession, requestMessages, responseMessages);
|
|
|
|
// Assert
|
|
Assert.Same(responseMessages, context.ResponseMessages);
|
|
}
|
|
|
|
[Fact]
|
|
public void InvokedContext_InvokeException_Roundtrips()
|
|
{
|
|
// Arrange
|
|
var requestMessages = new List<ChatMessage> { new(ChatRole.User, "Hello") };
|
|
var exception = new InvalidOperationException("Test exception");
|
|
|
|
// Act
|
|
var context = new ChatHistoryProvider.InvokedContext(s_mockAgent, s_mockSession, requestMessages, exception);
|
|
|
|
// Assert
|
|
Assert.Same(exception, context.InvokeException);
|
|
}
|
|
|
|
[Fact]
|
|
public void InvokedContext_Agent_ReturnsConstructorValue()
|
|
{
|
|
// Arrange
|
|
var requestMessages = new List<ChatMessage> { new(ChatRole.User, "Hello") };
|
|
|
|
// Act
|
|
var context = new ChatHistoryProvider.InvokedContext(s_mockAgent, s_mockSession, requestMessages, []);
|
|
|
|
// Assert
|
|
Assert.Same(s_mockAgent, context.Agent);
|
|
}
|
|
|
|
[Fact]
|
|
public void InvokedContext_Session_ReturnsConstructorValue()
|
|
{
|
|
// Arrange
|
|
var requestMessages = new List<ChatMessage> { new(ChatRole.User, "Hello") };
|
|
|
|
// Act
|
|
var context = new ChatHistoryProvider.InvokedContext(s_mockAgent, s_mockSession, requestMessages, []);
|
|
|
|
// Assert
|
|
Assert.Same(s_mockSession, context.Session);
|
|
}
|
|
|
|
[Fact]
|
|
public void InvokedContext_Session_CanBeNull()
|
|
{
|
|
// Arrange
|
|
var requestMessages = new List<ChatMessage> { new(ChatRole.User, "Hello") };
|
|
|
|
// Act
|
|
var context = new ChatHistoryProvider.InvokedContext(s_mockAgent, null, requestMessages, []);
|
|
|
|
// Assert
|
|
Assert.Null(context.Session);
|
|
}
|
|
|
|
[Fact]
|
|
public void InvokedContext_Constructor_ThrowsForNullAgent()
|
|
{
|
|
// Arrange
|
|
var requestMessages = new List<ChatMessage> { new(ChatRole.User, "Hello") };
|
|
|
|
// Act & Assert
|
|
Assert.Throws<ArgumentNullException>(() => new ChatHistoryProvider.InvokedContext(null!, s_mockSession, requestMessages, []));
|
|
}
|
|
|
|
[Fact]
|
|
public void InvokedContext_SuccessConstructor_ThrowsForNullResponseMessages()
|
|
{
|
|
// Arrange
|
|
var requestMessages = new List<ChatMessage> { new(ChatRole.User, "Hello") };
|
|
|
|
// Act & Assert
|
|
Assert.Throws<ArgumentNullException>(() => new ChatHistoryProvider.InvokedContext(s_mockAgent, s_mockSession, requestMessages, (IEnumerable<ChatMessage>)null!));
|
|
}
|
|
|
|
[Fact]
|
|
public void InvokedContext_FailureConstructor_ThrowsForNullException()
|
|
{
|
|
// Arrange
|
|
var requestMessages = new List<ChatMessage> { new(ChatRole.User, "Hello") };
|
|
|
|
// Act & Assert
|
|
Assert.Throws<ArgumentNullException>(() => new ChatHistoryProvider.InvokedContext(s_mockAgent, s_mockSession, requestMessages, (Exception)null!));
|
|
}
|
|
|
|
#endregion
|
|
|
|
private sealed class TestChatHistoryProvider : ChatHistoryProvider
|
|
{
|
|
protected override ValueTask<IEnumerable<ChatMessage>> InvokingCoreAsync(InvokingContext context, CancellationToken cancellationToken = default)
|
|
=> new(new ChatMessage[] { new(ChatRole.User, "Test Message") }.Concat(context.RequestMessages));
|
|
|
|
protected override ValueTask InvokedCoreAsync(InvokedContext context, CancellationToken cancellationToken = default)
|
|
=> default;
|
|
}
|
|
}
|