Skip to content

Commit ed9195d

Browse files
Merge pull request #498 from tonerdo/revert-415-FileExists
Revert "Using File.Exists for HitsFile creation/update validation"
2 parents e3d1a6c + 320763d commit ed9195d

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

src/coverlet.core/Instrumentation/ModuleTrackerTemplate.cs

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,8 @@ public static void UnloadModule(object sender, EventArgs e)
8282
if (!createdNew)
8383
mutex.WaitOne();
8484

85-
if (!File.Exists(HitsFilePath))
85+
bool failedToCreateNewHitsFile = false;
86+
try
8687
{
8788
using (var fs = new FileStream(HitsFilePath, FileMode.CreateNew))
8889
using (var bw = new BinaryWriter(fs))
@@ -94,7 +95,12 @@ public static void UnloadModule(object sender, EventArgs e)
9495
}
9596
}
9697
}
97-
else
98+
catch
99+
{
100+
failedToCreateNewHitsFile = true;
101+
}
102+
103+
if (failedToCreateNewHitsFile)
98104
{
99105
// Update the number of hits by adding value on disk with the ones on memory.
100106
// This path should be triggered only in the case of multiple AppDomain unloads.

0 commit comments

Comments
 (0)