mirror of
https://github.com/microsoft/agent-framework.git
synced 2026-06-16 21:04:09 +08:00
626b418622
* .NET: Add a TODO AIContextProvider (#5233) * Add a TODO AIContextProvider * Add unit tests * Address PR comments * Address PR comments * Fix test after removing one tool * .NET: Add a ModeProvider for managing agent modes (#5247) * Add a ModeProvider for managing agent modes * Fix typo * Fix typo * Fix typo * Address PR comments * .NET: Add sample to show how to build a harness (#5268) * Add sample to show how to build a harness * Improve sample * Sample max output tokens and model * Fix encoding * Fix model name in readme * Address PR comments * .NET: Add context window size compaction strategy for harness (#5304) * Add context window size compaction strategy for harness * Apply suggestions from code review Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Address PR comments --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * .NET: Add a file memory provider (#5315) * Add a file memory provider * Address PR comments * Fix review comments. * Add additional unit tests * Addressing PR comments. * .NET: Harness: Improve prompts and add FileSystem store (#5365) * Harness: Improve prompts and add FileSystem store * Address PR comments * .NET: Harness: Improve path validation (#5404) * Harness: Improve path validation * Address PR comments * .NET: Add always approve helpers, improve sample and fix bug (#5451) * Add always approve helpers, improve sample and fix bug * Address PR comments * .NET: Make Todo, Mode and FileMemory providers more configurable (#5477) * Make Todo, Mode and FileMemory providers more configurable * Address PR comments. * .NET: Add subagents provider and sample (#5518) * Add subagents provider and sample * Addressing PR comments. * .NET: Harness filememory index plus instructions consistency (#5540) * Add FileMemoryProvider index and improve instruction consistency * Address PR comments. * Address PR comments * Address PR comments. * Apply suggestion from @rogerbarreto Co-authored-by: Roger Barreto <19890735+rogerbarreto@users.noreply.github.com> --------- Co-authored-by: Roger Barreto <19890735+rogerbarreto@users.noreply.github.com> * .NET: Refactor harness console to be more extensible and easy to understand with better UX (#5573) * Refactor harness console to be more extensible and easy to understand with better UX. * Fix formatting issues. * Allow multiple clarifications in one response * Address PR comments * .NET: Add FileAccessProvdider and concurrency fix for FileMemoryProvider (#5583) * Add FileAccessProvdider and concurrency fix for FileMemoryProvider * Address PR comments --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Roger Barreto <19890735+rogerbarreto@users.noreply.github.com>
175 lines
4.8 KiB
C#
175 lines
4.8 KiB
C#
// Copyright (c) Microsoft. All rights reserved.
|
|
|
|
using System;
|
|
using Microsoft.Extensions.FileSystemGlobbing;
|
|
|
|
namespace Microsoft.Agents.AI.UnitTests.Harness.FileMemory;
|
|
|
|
public class StorePathsTests
|
|
{
|
|
#region NormalizeRelativePath — valid paths
|
|
|
|
[Theory]
|
|
[InlineData("file.md", "file.md")]
|
|
[InlineData("folder/file.md", "folder/file.md")]
|
|
[InlineData("a/b/c.txt", "a/b/c.txt")]
|
|
public void NormalizeRelativePath_ValidPath_ReturnsNormalized(string input, string expected)
|
|
{
|
|
// Act
|
|
string result = StorePaths.NormalizeRelativePath(input);
|
|
|
|
// Assert
|
|
Assert.Equal(expected, result);
|
|
}
|
|
|
|
[Theory]
|
|
[InlineData("folder\\file.md", "folder/file.md")]
|
|
[InlineData("a\\b\\c.txt", "a/b/c.txt")]
|
|
public void NormalizeRelativePath_Backslashes_NormalizesToForwardSlash(string input, string expected)
|
|
{
|
|
// Act
|
|
string result = StorePaths.NormalizeRelativePath(input);
|
|
|
|
// Assert
|
|
Assert.Equal(expected, result);
|
|
}
|
|
|
|
[Theory]
|
|
[InlineData("folder//file.md", "folder/file.md")]
|
|
[InlineData("a///b////c.txt", "a/b/c.txt")]
|
|
public void NormalizeRelativePath_ConsecutiveSeparators_Collapsed(string input, string expected)
|
|
{
|
|
// Act
|
|
string result = StorePaths.NormalizeRelativePath(input);
|
|
|
|
// Assert
|
|
Assert.Equal(expected, result);
|
|
}
|
|
|
|
[Fact]
|
|
public void NormalizeRelativePath_TrailingSlash_Trimmed()
|
|
{
|
|
// Act
|
|
string result = StorePaths.NormalizeRelativePath("file.md/");
|
|
|
|
// Assert
|
|
Assert.Equal("file.md", result);
|
|
}
|
|
|
|
[Theory]
|
|
[InlineData("/file.md")]
|
|
[InlineData("/folder/file.md/")]
|
|
public void NormalizeRelativePath_LeadingSlash_Throws(string input)
|
|
{
|
|
// Act & Assert — leading slash is treated as a rooted path.
|
|
Assert.Throws<ArgumentException>(() => StorePaths.NormalizeRelativePath(input));
|
|
}
|
|
|
|
#endregion
|
|
|
|
#region NormalizeRelativePath — rejected paths
|
|
|
|
[Theory]
|
|
[InlineData("../file.md")]
|
|
[InlineData("folder/../file.md")]
|
|
[InlineData("./file.md")]
|
|
[InlineData("folder/./file.md")]
|
|
public void NormalizeRelativePath_TraversalSegments_Throws(string input)
|
|
{
|
|
// Act & Assert
|
|
Assert.Throws<ArgumentException>(() => StorePaths.NormalizeRelativePath(input));
|
|
}
|
|
|
|
[Theory]
|
|
[InlineData("C:\\file.md")]
|
|
[InlineData("C:/file.md")]
|
|
[InlineData("D:file.md")]
|
|
public void NormalizeRelativePath_DriveRoot_Throws(string input)
|
|
{
|
|
// Act & Assert
|
|
Assert.Throws<ArgumentException>(() => StorePaths.NormalizeRelativePath(input));
|
|
}
|
|
|
|
[Fact]
|
|
public void NormalizeRelativePath_EmptyFile_Throws()
|
|
{
|
|
// Act & Assert
|
|
Assert.Throws<ArgumentException>(() => StorePaths.NormalizeRelativePath(""));
|
|
}
|
|
|
|
[Fact]
|
|
public void NormalizeRelativePath_WhitespaceOnlyFile_Throws()
|
|
{
|
|
// Act & Assert — whitespace-only paths are rejected as invalid file names.
|
|
Assert.Throws<ArgumentException>(() => StorePaths.NormalizeRelativePath(" "));
|
|
}
|
|
|
|
#endregion
|
|
|
|
#region NormalizeRelativePath — directory mode
|
|
|
|
[Fact]
|
|
public void NormalizeRelativePath_EmptyDirectory_ReturnsEmpty()
|
|
{
|
|
// Act
|
|
string result = StorePaths.NormalizeRelativePath("", isDirectory: true);
|
|
|
|
// Assert
|
|
Assert.Equal("", result);
|
|
}
|
|
|
|
[Theory]
|
|
[InlineData("folder", "folder")]
|
|
[InlineData("a/b", "a/b")]
|
|
[InlineData("a\\b/", "a/b")]
|
|
public void NormalizeRelativePath_DirectoryMode_NormalizesPath(string input, string expected)
|
|
{
|
|
// Act
|
|
string result = StorePaths.NormalizeRelativePath(input, isDirectory: true);
|
|
|
|
// Assert
|
|
Assert.Equal(expected, result);
|
|
}
|
|
|
|
[Fact]
|
|
public void NormalizeRelativePath_DirectoryTraversal_Throws()
|
|
{
|
|
// Act & Assert
|
|
Assert.Throws<ArgumentException>(() => StorePaths.NormalizeRelativePath("../folder", isDirectory: true));
|
|
}
|
|
|
|
#endregion
|
|
|
|
#region CreateGlobMatcher and MatchesGlob
|
|
|
|
[Theory]
|
|
[InlineData("*.md", "notes.md", true)]
|
|
[InlineData("*.md", "notes.txt", false)]
|
|
[InlineData("research*", "research_results.md", true)]
|
|
[InlineData("research*", "notes.md", false)]
|
|
[InlineData("*.md", "NOTES.MD", true)] // case-insensitive
|
|
public void MatchesGlob_WithMatcher_MatchesCorrectly(string pattern, string fileName, bool expected)
|
|
{
|
|
// Arrange
|
|
Matcher matcher = StorePaths.CreateGlobMatcher(pattern);
|
|
|
|
// Act
|
|
bool result = StorePaths.MatchesGlob(fileName, matcher);
|
|
|
|
// Assert
|
|
Assert.Equal(expected, result);
|
|
}
|
|
|
|
[Fact]
|
|
public void MatchesGlob_NullMatcher_ReturnsTrue()
|
|
{
|
|
// Act
|
|
bool result = StorePaths.MatchesGlob("anything.txt", null);
|
|
|
|
// Assert
|
|
Assert.True(result);
|
|
}
|
|
|
|
#endregion
|
|
}
|