Skip to content

Commit 294623d

Browse files
thomhurstclaude
andauthored
test: Test code improvements (#1655)
* chore: Add .worktrees/ to .gitignore for parallel development πŸ€– Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * chore: Remove unused FakeTimeProvider imports (#1653) Remove the unused `Microsoft.Extensions.Time.Testing` import from three NotInParallel test files that no longer use FakeTimeProvider. πŸ€– Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * test: Add cleanup for ZipTests artifacts (#1652) Add [After(Class)] cleanup method to ZipTests to delete LoremData.zip and LoremDataUnzipped directory after tests complete, preventing test artifacts from accumulating in the working directory. πŸ€– Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
1 parent 693c58f commit 294623d

4 files changed

Lines changed: 19 additions & 3 deletions

File tree

β€Žtest/ModularPipelines.UnitTests/Helpers/ZipTests.csβ€Ž

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,25 @@ namespace ModularPipelines.UnitTests.Helpers;
88

99
public class ZipTests : TestBase
1010
{
11+
[After(Class)]
12+
public static async Task Cleanup()
13+
{
14+
await Task.CompletedTask;
15+
16+
var zipFile = Path.Combine(TestContext.WorkingDirectory, "LoremData.zip");
17+
var unzippedDir = Path.Combine(TestContext.WorkingDirectory, "LoremDataUnzipped");
18+
19+
if (System.IO.File.Exists(zipFile))
20+
{
21+
System.IO.File.Delete(zipFile);
22+
}
23+
24+
if (Directory.Exists(unzippedDir))
25+
{
26+
Directory.Delete(unzippedDir, recursive: true);
27+
}
28+
}
29+
1130
private class ZipModule : Module<string>
1231
{
1332
public override async Task<string?> ExecuteAsync(IModuleContext context, CancellationToken cancellationToken)

β€Žtest/ModularPipelines.UnitTests/NotInParallelTests.csβ€Ž

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
using System.Collections.Concurrent;
2-
using Microsoft.Extensions.Time.Testing;
32
using ModularPipelines.Attributes;
43
using ModularPipelines.Context;
54
using ModularPipelines.Modules;

β€Žtest/ModularPipelines.UnitTests/NotInParallelTestsWithConstraintKeys.csβ€Ž

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
using System.Collections.Concurrent;
2-
using Microsoft.Extensions.Time.Testing;
32
using ModularPipelines.Context;
43
using ModularPipelines.Modules;
54
using ModularPipelines.TestHelpers;

β€Žtest/ModularPipelines.UnitTests/NotInParallelTestsWithMultipleConstraintKeys.csβ€Ž

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
using System.Collections.Concurrent;
2-
using Microsoft.Extensions.Time.Testing;
32
using ModularPipelines.Context;
43
using ModularPipelines.Modules;
54
using ModularPipelines.TestHelpers;

0 commit comments

Comments
Β (0)