Skip to content

Commit 89f5be5

Browse files
committed
BackupOriginalModule no return value and overwrite
1 parent d6e008f commit 89f5be5

File tree

2 files changed

+8
-4
lines changed

2 files changed

+8
-4
lines changed

src/coverlet.core/Helpers/InstrumentationHelper.cs

+2-3
Original file line numberDiff line numberDiff line change
@@ -78,11 +78,10 @@ public static bool HasPdb(string module)
7878
}
7979
}
8080

81-
public static string BackupOriginalModule(string module, string identifier)
81+
public static void BackupOriginalModule(string module, string identifier)
8282
{
8383
var backupPath = GetBackupPath(module, identifier);
84-
File.Copy(module, backupPath);
85-
return backupPath;
84+
File.Copy(module, backupPath, true);
8685
}
8786

8887
public static void RestoreOriginalModule(string module, string identifier)

test/coverlet.core.tests/Helpers/InstrumentationHelperTests.cs

+6-1
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,12 @@ public void TestBackupOriginalModule()
2828
string module = typeof(InstrumentationHelperTests).Assembly.Location;
2929
string identifier = Guid.NewGuid().ToString();
3030

31-
var backupPath = InstrumentationHelper.BackupOriginalModule(module, identifier);
31+
InstrumentationHelper.BackupOriginalModule(module, identifier);
32+
33+
var backupPath = Path.Combine(
34+
Path.GetTempPath(),
35+
Path.GetFileNameWithoutExtension(module) + "_" + identifier + ".dll"
36+
);
3237

3338
Assert.True(File.Exists(backupPath));
3439
}

0 commit comments

Comments
 (0)