test: Test code improvements

[thomhurst]

Summary

• Remove unused Microsoft.Extensions.Time.Testing import from NotInParallel test files
• Add cleanup method for ZipTests artifacts (LoremData.zip and LoremDataUnzipped directory)

Test plan

• Verified the project builds successfully
• Removed imports are confirmed unused
• Cleanup method follows TUnit [After(Class)] pattern for class-level cleanup

Closes #1653, #1652

🤖 Generated with Claude Code

[thomhurst]

Summary

This PR adds cleanup logic for test artifacts in ZipTests and removes unused imports from three test files.

Critical Issues

None found ✅

The changes are straightforward:

• ZipTests.cs: Adds proper cleanup of test artifacts (LoremData.zip and LoremDataUnzipped directory) using TUnit's [After(Class)] hook
• NotInParallelTests.cs*: Removes unused Microsoft.Extensions.Time.Testing imports

Suggestions

Minor: Redundant await in cleanup method (ZipTests.cs:13-14)
The await Task.CompletedTask; at the start of the Cleanup() method is unnecessary since the rest of the method is synchronous. You can either:

• Remove the async keyword and await Task.CompletedTask entirely, OR
• Remove just the await Task.CompletedTask line

This is purely a style issue and doesn't affect functionality.

Verdict

✅ APPROVE - No critical issues

The cleanup logic properly addresses potential test pollution issues where zip artifacts could interfere with subsequent test runs. The unused import removal is good housekeeping.