Skip to content

fix: DirectoryMock.EnumerateDirectories does not throw exceptions until enumerated #857

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
vbreuss merged 4 commits into from
Sep 11, 2025
Merged

fix: DirectoryMock.EnumerateDirectories does not throw exceptions until enumerated #857

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
6c40f5a
Initial plan
Copilot Sep 11, 2025
90357af
Fix DirectoryMock.EnumerateDirectories immediate exception throwing
Copilot Sep 11, 2025
eb0bb1a
Fix statistics tests for immediate exception behavior
Copilot Sep 11, 2025
41908c2
Fix copilot errors
vbreuss Sep 11, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Fix copilot errors
  • Loading branch information
@vbreuss
vbreuss committed Sep 11, 2025
commit 41908c2b0a725ee9bffe8700deb72dc141902f92
9 changes: 4 additions & 5 deletions Source/Testably.Abstractions.Testing/Storage/InMemoryStorage.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -193,20 +193,19 @@ public IEnumerable<IStorageLocation> EnumerateLocations(
string searchPattern = EnumerationOptionsHelper.DefaultSearchPattern,
EnumerationOptions? enumerationOptions = null)
{
// Perform immediate validation and throw exceptions if necessary
ValidateExpression(searchPattern);
if (!_containers.TryGetValue(location, out IStorageContainer? parentContainer))
{
throw ExceptionFactory.DirectoryNotFound(location.FullPath);
}

// Return the actual enumeration implementation
return EnumerateLocationsImpl(location, type, requestParentAccess, searchPattern, enumerationOptions, parentContainer);
return EnumerateLocationsImpl(location, type, requestParentAccess, searchPattern,
enumerationOptions, parentContainer);
}

/// <summary>
/// Internal implementation of location enumeration that uses yield return.
/// This method contains the actual enumeration logic and is only called after validation passes.
/// Internal implementation of location enumeration that uses yield return.
/// This method contains the actual enumeration logic and is only called after validation passes.
/// </summary>
private IEnumerable<IStorageLocation> EnumerateLocationsImpl(
IStorageLocation location,
Expand Down
8 changes: 8 additions & 0 deletions ...Testably.Abstractions.Testing.Tests/Statistics/FileSystem/DirectoryInfoStatisticsTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -142,6 +142,7 @@ await That(sut.Statistics.DirectoryInfo["foo"])
public async Task Method_EnumerateFiles_ShouldRegisterCall()
{
MockFileSystem sut = new();
sut.Initialize().WithSubdirectory("foo");

sut.DirectoryInfo.New("foo").EnumerateFiles();

Expand All @@ -155,6 +156,7 @@ await That(sut.Statistics.DirectoryInfo["foo"])
public async Task Method_EnumerateFiles_String_EnumerationOptions_ShouldRegisterCall()
{
MockFileSystem sut = new();
sut.Initialize().WithSubdirectory("foo");
string searchPattern = "foo";
EnumerationOptions enumerationOptions = new();

Expand All @@ -171,6 +173,7 @@ await That(sut.Statistics.DirectoryInfo["foo"])
public async Task Method_EnumerateFiles_String_SearchOption_ShouldRegisterCall()
{
MockFileSystem sut = new();
sut.Initialize().WithSubdirectory("foo");
string searchPattern = "foo";
SearchOption searchOption = SearchOption.AllDirectories;

Expand All @@ -186,6 +189,7 @@ await That(sut.Statistics.DirectoryInfo["foo"])
public async Task Method_EnumerateFiles_String_ShouldRegisterCall()
{
MockFileSystem sut = new();
sut.Initialize().WithSubdirectory("foo");
string searchPattern = "foo";

sut.DirectoryInfo.New("foo").EnumerateFiles(searchPattern);
Expand All @@ -200,6 +204,7 @@ await That(sut.Statistics.DirectoryInfo["foo"])
public async Task Method_EnumerateFileSystemInfos_ShouldRegisterCall()
{
MockFileSystem sut = new();
sut.Initialize().WithSubdirectory("foo");

sut.DirectoryInfo.New("foo").EnumerateFileSystemInfos();

Expand All @@ -213,6 +218,7 @@ await That(sut.Statistics.DirectoryInfo["foo"])
public async Task Method_EnumerateFileSystemInfos_String_EnumerationOptions_ShouldRegisterCall()
{
MockFileSystem sut = new();
sut.Initialize().WithSubdirectory("foo");
string searchPattern = "foo";
EnumerationOptions enumerationOptions = new();

Expand All @@ -229,6 +235,7 @@ await That(sut.Statistics.DirectoryInfo["foo"])
public async Task Method_EnumerateFileSystemInfos_String_SearchOption_ShouldRegisterCall()
{
MockFileSystem sut = new();
sut.Initialize().WithSubdirectory("foo");
string searchPattern = "foo";
SearchOption searchOption = SearchOption.AllDirectories;

Expand All @@ -244,6 +251,7 @@ await That(sut.Statistics.DirectoryInfo["foo"])
public async Task Method_EnumerateFileSystemInfos_String_ShouldRegisterCall()
{
MockFileSystem sut = new();
sut.Initialize().WithSubdirectory("foo");
string searchPattern = "foo";

sut.DirectoryInfo.New("foo").EnumerateFileSystemInfos(searchPattern);
Expand Down
8 changes: 8 additions & 0 deletions Tests/Testably.Abstractions.Testing.Tests/Statistics/FileSystem/DirectoryStatisticsTests.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -170,6 +170,7 @@ await That(sut.Statistics.Directory).OnlyContainsMethodCall(
public async Task Method_EnumerateFiles_String_ShouldRegisterCall()
{
MockFileSystem sut = new();
sut.Initialize().WithSubdirectory("foo");
string path = "foo";

sut.Directory.EnumerateFiles(path);
Expand All @@ -185,6 +186,7 @@ await That(sut.Statistics.Directory).OnlyContainsMethodCall(
public async Task Method_EnumerateFiles_String_String_EnumerationOptions_ShouldRegisterCall()
{
MockFileSystem sut = new();
sut.Initialize().WithSubdirectory("foo");
string path = "foo";
string searchPattern = "foo";
EnumerationOptions enumerationOptions = new();
Expand All @@ -201,6 +203,7 @@ await That(sut.Statistics.Directory).OnlyContainsMethodCall(nameof(IDirectory.En
public async Task Method_EnumerateFiles_String_String_SearchOption_ShouldRegisterCall()
{
MockFileSystem sut = new();
sut.Initialize().WithSubdirectory("foo");
string path = "foo";
string searchPattern = "foo";
SearchOption searchOption = SearchOption.AllDirectories;
Expand All @@ -217,6 +220,7 @@ await That(sut.Statistics.Directory).OnlyContainsMethodCall(
public async Task Method_EnumerateFiles_String_String_ShouldRegisterCall()
{
MockFileSystem sut = new();
sut.Initialize().WithSubdirectory("foo");
string path = "foo";
string searchPattern = "foo";

Expand All @@ -232,6 +236,7 @@ await That(sut.Statistics.Directory).OnlyContainsMethodCall(
public async Task Method_EnumerateFileSystemEntries_String_ShouldRegisterCall()
{
MockFileSystem sut = new();
sut.Initialize().WithSubdirectory("foo");
string path = "foo";

sut.Directory.EnumerateFileSystemEntries(path);
Expand All @@ -247,6 +252,7 @@ await That(sut.Statistics.Directory).OnlyContainsMethodCall(
public async Task Method_EnumerateFileSystemEntries_String_String_EnumerationOptions_ShouldRegisterCall()
{
MockFileSystem sut = new();
sut.Initialize().WithSubdirectory("foo");
string path = "foo";
string searchPattern = "foo";
EnumerationOptions enumerationOptions = new();
Expand All @@ -265,6 +271,7 @@ public async Task
Method_EnumerateFileSystemEntries_String_String_SearchOption_ShouldRegisterCall()
{
MockFileSystem sut = new();
sut.Initialize().WithSubdirectory("foo");
string path = "foo";
string searchPattern = "foo";
SearchOption searchOption = SearchOption.AllDirectories;
Expand All @@ -281,6 +288,7 @@ await That(sut.Statistics.Directory).OnlyContainsMethodCall(
public async Task Method_EnumerateFileSystemEntries_String_String_ShouldRegisterCall()
{
MockFileSystem sut = new();
sut.Initialize().WithSubdirectory("foo");
string path = "foo";
string searchPattern = "foo";

Expand Down
Loading