Skip to content

Commit 2c20567

Browse files
committed
refactor: improve resource disposal logic in test classes
1 parent f02d160 commit 2c20567

2 files changed

Lines changed: 18 additions & 8 deletions

File tree

CodeLineCounter.Tests/FileUtilsTests.cs

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -99,11 +99,16 @@ protected virtual void Dispose(bool disposing)
9999
if (!_disposed)
100100
{
101101
// Ensure the file is closed before attempting to delete it
102-
Task.Delay(100).Wait();
103-
if (disposing && Directory.Exists(_testDirectory))
102+
if (disposing)
104103
{
105-
// Dispose managed resources
106-
Directory.Delete(_testDirectory, true);
104+
105+
Task.Delay(100).Wait();
106+
107+
if (Directory.Exists(_testDirectory))
108+
{
109+
// Dispose managed resources
110+
Directory.Delete(_testDirectory, true);
111+
}
107112
}
108113

109114
// Dispose unmanaged resources (if any)

CodeLineCounter.Tests/SolutionAnalyzerTests.cs

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -270,11 +270,16 @@ protected virtual void Dispose(bool disposing)
270270
{
271271
if (!_disposed)
272272
{
273-
if (disposing && Directory.Exists(_testDirectory))
273+
if (disposing)
274274
{
275-
// Dispose managed resources
276-
File.Delete(_testSolutionPath);
277-
Directory.Delete(_testDirectory, true);
275+
Task.Delay(100).Wait();
276+
277+
if (Directory.Exists(_testDirectory))
278+
{
279+
// Dispose managed resources
280+
File.Delete(_testSolutionPath);
281+
Directory.Delete(_testDirectory, true);
282+
}
278283
}
279284

280285
// Dispose unmanaged resources (if any)

0 commit comments

Comments
 (0)