mirror of
https://github.com/microsoft/agent-framework.git
synced 2026-06-16 21:04:09 +08:00
9f4c5f3faa
* Initial plan * Add comprehensive unit tests for EditTableV2Executor - Test AddItemOperation with record and scalar values - Test ClearItemsOperation - Test RemoveItemOperation - Test TakeLastItemOperation (with items and empty table) - Test TakeFirstItemOperation (with items and empty table) - Test error cases (null ItemsVariable, non-table variable) - Include ExecuteTestAsync and CreateModel helper methods - All 10 tests passing Co-authored-by: crickman <66376200+crickman@users.noreply.github.com> * Add comprehensive unit tests for EditTableV2Executor - complete with 100% coverage - Added 13 comprehensive tests covering all code paths - Test AddItemOperation with record and scalar values - Test ClearItemsOperation - Test RemoveItemOperation (including non-table value case) - Test TakeLastItemOperation (with items and empty table) - Test TakeFirstItemOperation (with items and empty table) - Test error cases (null ItemsVariable, non-table variable, null operation values) - Include ExecuteTestAsync and CreateModel helper methods - 100% line and branch coverage achieved Co-authored-by: crickman <66376200+crickman@users.noreply.github.com> * Update tests / refine product code * Checkpoint * Updated * Update dotnet/tests/Microsoft.Agents.AI.Workflows.Declarative.UnitTests/ObjectModel/SetTextVariableExecutorTest.cs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Address code review feedback - Fix typo: rename metadataExpresssion to metadataExpression - Fix test name in AddMessageWithMetadataAsync (was using wrong test name) - Fix test name in ClearGlobalScopeAsync (was using wrong test name) - Remove pre-population in SetTextVariableExecutorTest that made tests ineffective - Use explicit .Where() filter in SetMultipleVariablesExecutorTest foreach loop Co-authored-by: crickman <66376200+crickman@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: crickman <66376200+crickman@users.noreply.github.com> Co-authored-by: Chris Rickman <crickman@microsoft.com> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
203 lines
6.5 KiB
C#
203 lines
6.5 KiB
C#
// Copyright (c) Microsoft. All rights reserved.
|
|
|
|
using System.Threading.Tasks;
|
|
using Microsoft.Agents.AI.Workflows.Declarative.ObjectModel;
|
|
using Microsoft.Agents.ObjectModel;
|
|
using Microsoft.PowerFx.Types;
|
|
using Xunit.Abstractions;
|
|
|
|
namespace Microsoft.Agents.AI.Workflows.Declarative.UnitTests.ObjectModel;
|
|
|
|
/// <summary>
|
|
/// Tests for <see cref="SetVariableExecutor"/>.
|
|
/// </summary>
|
|
public sealed class SetVariableExecutorTest(ITestOutputHelper output) : WorkflowActionExecutorTest(output)
|
|
{
|
|
[Fact]
|
|
public void InvalidModel() =>
|
|
// Arrange, Act, Assert
|
|
Assert.Throws<DeclarativeModelException>(() => new SetVariableExecutor(new SetVariable(), this.State));
|
|
|
|
[Fact]
|
|
public async Task SetNumericValueAsync() =>
|
|
// Arrange, Act, Assert
|
|
await this.ExecuteTestAsync(
|
|
displayName: nameof(SetNumericValueAsync),
|
|
variableName: "TestVariable",
|
|
variableValue: new NumberDataValue(42),
|
|
expectedValue: FormulaValue.New(42));
|
|
|
|
[Fact]
|
|
public async Task SetStringValueAsync() =>
|
|
// Arrange, Act, Assert
|
|
await this.ExecuteTestAsync(
|
|
displayName: nameof(SetStringValueAsync),
|
|
variableName: "TestVariable",
|
|
variableValue: new StringDataValue("Text"),
|
|
expectedValue: FormulaValue.New("Text"));
|
|
|
|
[Fact]
|
|
public async Task SetBooleanValueAsync() =>
|
|
// Arrange, Act, Assert
|
|
await this.ExecuteTestAsync(
|
|
displayName: nameof(SetBooleanValueAsync),
|
|
variableName: "TestVariable",
|
|
variableValue: new BooleanDataValue(true),
|
|
expectedValue: FormulaValue.New(true));
|
|
|
|
[Fact]
|
|
public async Task SetBooleanExpressionAsync()
|
|
{
|
|
// Arrange
|
|
ValueExpression.Builder expressionBuilder = new(ValueExpression.Expression("true || false"));
|
|
|
|
// Act, Assert
|
|
await this.ExecuteTestAsync(
|
|
displayName: nameof(SetBooleanExpressionAsync),
|
|
variableName: "TestVariable",
|
|
valueExpression: expressionBuilder,
|
|
expectedValue: FormulaValue.New(true));
|
|
}
|
|
|
|
[Fact]
|
|
public async Task SetNumberExpressionAsync()
|
|
{
|
|
// Arrange
|
|
ValueExpression.Builder expressionBuilder = new(ValueExpression.Expression("9 - 3"));
|
|
|
|
// Act, Assert
|
|
await this.ExecuteTestAsync(
|
|
displayName: nameof(SetBooleanExpressionAsync),
|
|
variableName: "TestVariable",
|
|
valueExpression: expressionBuilder,
|
|
expectedValue: FormulaValue.New(6));
|
|
}
|
|
|
|
[Fact]
|
|
public async Task SetStringExpressionAsync()
|
|
{
|
|
// Arrange
|
|
ValueExpression.Builder expressionBuilder = new(ValueExpression.Expression(@"Concatenate(""A"", ""B"", ""C"")"));
|
|
|
|
// Act, Assert
|
|
await this.ExecuteTestAsync(
|
|
displayName: nameof(SetBooleanExpressionAsync),
|
|
variableName: "TestVariable",
|
|
valueExpression: expressionBuilder,
|
|
expectedValue: FormulaValue.New("ABC"));
|
|
}
|
|
|
|
[Fact]
|
|
public async Task SetBooleanVariableAsync()
|
|
{
|
|
// Arrange
|
|
this.State.Set("Source", FormulaValue.New(true));
|
|
|
|
ValueExpression.Builder expressionBuilder = new(ValueExpression.Variable(PropertyPath.TopicVariable("Source")));
|
|
|
|
// Act, Assert
|
|
await this.ExecuteTestAsync(
|
|
displayName: nameof(SetBooleanExpressionAsync),
|
|
variableName: "TestVariable",
|
|
valueExpression: expressionBuilder,
|
|
expectedValue: FormulaValue.New(true));
|
|
}
|
|
|
|
[Fact]
|
|
public async Task SetNumberVariableAsync()
|
|
{
|
|
// Arrange
|
|
this.State.Set("Source", FormulaValue.New(321));
|
|
|
|
ValueExpression.Builder expressionBuilder = new(ValueExpression.Variable(PropertyPath.TopicVariable("Source")));
|
|
|
|
// Act, Assert
|
|
await this.ExecuteTestAsync(
|
|
displayName: nameof(SetBooleanExpressionAsync),
|
|
variableName: "TestVariable",
|
|
valueExpression: expressionBuilder,
|
|
expectedValue: FormulaValue.New(321));
|
|
}
|
|
|
|
[Fact]
|
|
public async Task SetStringVariableAsync()
|
|
{
|
|
// Arrange
|
|
this.State.Set("Source", FormulaValue.New("Test"));
|
|
|
|
ValueExpression.Builder expressionBuilder = new(ValueExpression.Variable(PropertyPath.TopicVariable("Source")));
|
|
|
|
// Act, Assert
|
|
await this.ExecuteTestAsync(
|
|
displayName: nameof(SetBooleanExpressionAsync),
|
|
variableName: "TestVariable",
|
|
valueExpression: expressionBuilder,
|
|
expectedValue: FormulaValue.New("Test"));
|
|
}
|
|
|
|
[Fact]
|
|
public async Task UpdateExistingValueAsync()
|
|
{
|
|
// Arrange
|
|
this.State.Set("VarA", FormulaValue.New(33));
|
|
|
|
// Act, Assert
|
|
await this.ExecuteTestAsync(
|
|
displayName: nameof(UpdateExistingValueAsync),
|
|
variableName: "VarA",
|
|
variableValue: new NumberDataValue(42),
|
|
expectedValue: FormulaValue.New(42));
|
|
}
|
|
|
|
private Task ExecuteTestAsync(
|
|
string displayName,
|
|
string variableName,
|
|
DataValue variableValue,
|
|
FormulaValue expectedValue)
|
|
{
|
|
// Arrange
|
|
ValueExpression.Builder expressionBuilder = new(ValueExpression.Literal(variableValue));
|
|
|
|
// Act & Assert
|
|
return this.ExecuteTestAsync(displayName, variableName, expressionBuilder, expectedValue);
|
|
}
|
|
|
|
private async Task ExecuteTestAsync(
|
|
string displayName,
|
|
string variableName,
|
|
ValueExpression.Builder valueExpression,
|
|
FormulaValue expectedValue)
|
|
{
|
|
// Arrange
|
|
SetVariable model =
|
|
this.CreateModel(
|
|
displayName,
|
|
FormatVariablePath(variableName),
|
|
valueExpression);
|
|
|
|
this.State.Set(variableName, FormulaValue.New(33));
|
|
|
|
// Act
|
|
SetVariableExecutor action = new(model, this.State);
|
|
await this.ExecuteAsync(action);
|
|
|
|
// Assert
|
|
VerifyModel(model, action);
|
|
this.VerifyState(variableName, expectedValue);
|
|
}
|
|
|
|
private SetVariable CreateModel(string displayName, string variablePath, ValueExpression.Builder valueExpression)
|
|
{
|
|
SetVariable.Builder actionBuilder =
|
|
new()
|
|
{
|
|
Id = this.CreateActionId(),
|
|
DisplayName = this.FormatDisplayName(displayName),
|
|
Variable = PropertyPath.Create(variablePath),
|
|
Value = valueExpression,
|
|
};
|
|
|
|
return AssignParent<SetVariable>(actionBuilder);
|
|
}
|
|
}
|