Update Msbuild.cs fix name

Bertk · web-flow · commit 9c341dc0aade · 2025-06-06T09:28:49.000+02:00
diff --git a/test/coverlet.integration.tests/Msbuild.cs b/test/coverlet.integration.tests/Msbuild.cs
@@ -14,7 +14,7 @@ public class Msbuild : BaseTest
     private readonly string _buildConfiguration;
     private readonly string _buildTargetFramework;
     private readonly ITestOutputHelper _output;
-    private static readonly string _expectedResult = "| coverletsamplelib.integration.template | 50 %  | 100 %   | 50 %    |";
+    private static readonly string s_expectedResult = "| coverletsamplelib.integration.template | 50 %  | 100 %   | 50 %    |";
 
     public Msbuild(ITestOutputHelper output)
     {
@@ -46,7 +46,7 @@ public void TestMsbuild()
       }
       Assert.Equal(0, result);
       Assert.Contains("Passed!", standardOutput, StringComparison.Ordinal);
-      Assert.Contains($"{_expectedResult}", standardOutput, StringComparison.Ordinal);
+      Assert.Contains($"{s_expectedResult}", standardOutput, StringComparison.Ordinal);
       string coverageFileName = $"coverage.json";
       Assert.True(File.Exists(Path.Combine(clonedTemplateProject.ProjectRootPath, coverageFileName)));
       AssertCoverage(clonedTemplateProject, coverageFileName);
@@ -67,7 +67,7 @@ public void TestMsbuild_NoCoverletOutput()
       }
       Assert.Equal(0, result);
       Assert.Contains("Passed!", standardOutput, StringComparison.Ordinal);
-      Assert.Contains($"{_expectedResult}", standardOutput, StringComparison.Ordinal);
+      Assert.Contains($"{s_expectedResult}", standardOutput, StringComparison.Ordinal);
       string coverageFileName = $"coverage.json";
       Assert.True(File.Exists(Path.Combine(clonedTemplateProject.ProjectRootPath, coverageFileName)));
       AssertCoverage(clonedTemplateProject, coverageFileName);
@@ -88,7 +88,7 @@ public void TestMsbuild_CoverletOutput_Folder_FileNameWithoutExtension()
       }
       Assert.Equal(0, result);
       Assert.Contains("Passed!", standardOutput, StringComparison.Ordinal);
-      Assert.Contains($"{_expectedResult}", standardOutput, StringComparison.Ordinal);
+      Assert.Contains($"{s_expectedResult}", standardOutput, StringComparison.Ordinal);
       string coverageFileName = $"file.json";
       Assert.True(File.Exists(Path.Combine(clonedTemplateProject.ProjectRootPath, coverageFileName)));
       AssertCoverage(clonedTemplateProject, coverageFileName);
@@ -100,7 +100,7 @@ public void TestMsbuild_CoverletOutput_Folder_FileNameExtension()
       using ClonedTemplateProject clonedTemplateProject = PrepareTemplateProject();
       Assert.Equal(0, DotnetCli($"test -c {_buildConfiguration} -f {_buildTargetFramework} \"{clonedTemplateProject.ProjectRootPath}\" /p:CollectCoverage=true /p:Include=\"[{ClonedTemplateProject.AssemblyName}]*DeepThought\" /p:IncludeTestAssembly=true /p:CoverletOutput=\"{clonedTemplateProject.ProjectRootPath}\"\\file.ext", out string standardOutput, out string standardError));
       Assert.Contains("Passed!", standardOutput, StringComparison.Ordinal);
-      Assert.Contains($"{_expectedResult}", standardOutput, StringComparison.Ordinal);
+      Assert.Contains($"{s_expectedResult}", standardOutput, StringComparison.Ordinal);
       string coverageFileName = $"file.ext";
       Assert.True(File.Exists(Path.Combine(clonedTemplateProject.ProjectRootPath, coverageFileName)));
       AssertCoverage(clonedTemplateProject, coverageFileName);
@@ -124,7 +124,7 @@ public void TestMsbuild_CoverletOutput_Folder_FileNameExtension_SpecifyFramework
         _output.WriteLine(standardOutput);
       }
       Assert.Contains("Passed!", standardOutput, StringComparison.Ordinal);
-      Assert.Contains($"{_expectedResult}", standardOutput, StringComparison.Ordinal);
+      Assert.Contains($"{s_expectedResult}", standardOutput, StringComparison.Ordinal);
       Assert.True(File.Exists(Path.Combine(clonedTemplateProject.ProjectRootPath, "file.ext")));
       AssertCoverage(clonedTemplateProject, "file.ext");
     }
@@ -143,7 +143,7 @@ public void TestMsbuild_CoverletOutput_Folder_FileNameWithDoubleExtension()
         _output.WriteLine(standardOutput);
       }
       Assert.Contains("Passed!", standardOutput, StringComparison.Ordinal);
-      Assert.Contains($"{_expectedResult}", standardOutput, StringComparison.Ordinal);
+      Assert.Contains($"{s_expectedResult}", standardOutput, StringComparison.Ordinal);
       string coverageFileName = $"file.ext1.ext2";
       Assert.True(File.Exists(Path.Combine(clonedTemplateProject.ProjectRootPath, coverageFileName)));
       AssertCoverage(clonedTemplateProject, coverageFileName);
@@ -165,7 +165,7 @@ public void Test_MultipleTargetFrameworkReport_NoCoverletOutput()
         _output.WriteLine(standardOutput);
       }
       Assert.Contains("Passed!", standardOutput, StringComparison.Ordinal);
-      Assert.Contains($"{_expectedResult}", standardOutput, StringComparison.Ordinal);
+      Assert.Contains($"{s_expectedResult}", standardOutput, StringComparison.Ordinal);
 
       foreach (string targetFramework in targetFrameworks)
       {
@@ -192,7 +192,7 @@ public void Test_MultipleTargetFrameworkReport_CoverletOutput_Folder()
       }
       Assert.Equal(0, result);
       Assert.Contains("Passed!", standardOutput, StringComparison.Ordinal);
-      Assert.Contains($"{_expectedResult}", standardOutput, StringComparison.Ordinal);
+      Assert.Contains($"{s_expectedResult}", standardOutput, StringComparison.Ordinal);
 
       foreach (string targetFramework in targetFrameworks)
       {
@@ -219,7 +219,7 @@ public void Test_MultipleTargetFrameworkReport_CoverletOutput_Folder_FileNameWit
         _output.WriteLine(standardOutput);
       }
       Assert.Contains("Passed!", standardOutput, StringComparison.Ordinal);
-      Assert.Contains($"{_expectedResult}", standardOutput, StringComparison.Ordinal);
+      Assert.Contains($"{s_expectedResult}", standardOutput, StringComparison.Ordinal);
 
       foreach (string targetFramework in targetFrameworks)
       {
@@ -249,7 +249,7 @@ public void Test_MultipleTargetFrameworkReport_CoverletOutput_Folder_FileNameWit
         _output.WriteLine(standardOutput);
       }
       Assert.Contains("Passed!", standardOutput, StringComparison.Ordinal);
-      Assert.Contains($"{_expectedResult}", standardOutput, StringComparison.Ordinal);
+      Assert.Contains($"{s_expectedResult}", standardOutput, StringComparison.Ordinal);
 
       foreach (string targetFramework in targetFrameworks)
       {
@@ -282,7 +282,7 @@ public void Test_MultipleTargetFrameworkReport_CoverletOutput_Folder_FileNameWit
         _output.WriteLine(standardOutput);
       }
       Assert.Contains("Passed!", standardOutput, StringComparison.Ordinal);
-      Assert.Contains($"{_expectedResult}", standardOutput, StringComparison.Ordinal);
+      Assert.Contains($"{s_expectedResult}", standardOutput, StringComparison.Ordinal);
 
       foreach (string targetFramework in targetFrameworks)
       {
@@ -308,7 +308,7 @@ public void Test_MultipleTargetFrameworkReport_CoverletOutput_Folder_FileNameWit
         _output.WriteLine(standardOutput);
       }
       Assert.Contains("Passed!", standardOutput, StringComparison.Ordinal);
-      Assert.Contains($"{_expectedResult}", standardOutput, StringComparison.Ordinal);
+      Assert.Contains($"{s_expectedResult}", standardOutput, StringComparison.Ordinal);
 
       foreach (string targetFramework in targetFrameworks)
       {

Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@ public class Msbuild : BaseTest`
`14`	`14`	`private readonly string _buildConfiguration;`
`15`	`15`	`private readonly string _buildTargetFramework;`
`16`	`16`	`private readonly ITestOutputHelper _output;`
`17`		`- private static readonly string _expectedResult = "\| coverletsamplelib.integration.template \| 50 % \| 100 % \| 50 % \|";`
	`17`	`+ private static readonly string s_expectedResult = "\| coverletsamplelib.integration.template \| 50 % \| 100 % \| 50 % \|";`
`18`	`18`
`19`	`19`	`public Msbuild(ITestOutputHelper output)`
`20`	`20`	`{`
`@@ -46,7 +46,7 @@ public void TestMsbuild()`
`46`	`46`	`}`
`47`	`47`	`Assert.Equal(0, result);`
`48`	`48`	`Assert.Contains("Passed!", standardOutput, StringComparison.Ordinal);`
`49`		`- Assert.Contains($"{_expectedResult}", standardOutput, StringComparison.Ordinal);`
	`49`	`+ Assert.Contains($"{s_expectedResult}", standardOutput, StringComparison.Ordinal);`
`50`	`50`	`string coverageFileName = $"coverage.json";`
`51`	`51`	`Assert.True(File.Exists(Path.Combine(clonedTemplateProject.ProjectRootPath, coverageFileName)));`
`52`	`52`	`AssertCoverage(clonedTemplateProject, coverageFileName);`
`@@ -67,7 +67,7 @@ public void TestMsbuild_NoCoverletOutput()`
`67`	`67`	`}`
`68`	`68`	`Assert.Equal(0, result);`
`69`	`69`	`Assert.Contains("Passed!", standardOutput, StringComparison.Ordinal);`
`70`		`- Assert.Contains($"{_expectedResult}", standardOutput, StringComparison.Ordinal);`
	`70`	`+ Assert.Contains($"{s_expectedResult}", standardOutput, StringComparison.Ordinal);`
`71`	`71`	`string coverageFileName = $"coverage.json";`
`72`	`72`	`Assert.True(File.Exists(Path.Combine(clonedTemplateProject.ProjectRootPath, coverageFileName)));`
`73`	`73`	`AssertCoverage(clonedTemplateProject, coverageFileName);`
`@@ -88,7 +88,7 @@ public void TestMsbuild_CoverletOutput_Folder_FileNameWithoutExtension()`
`88`	`88`	`}`
`89`	`89`	`Assert.Equal(0, result);`
`90`	`90`	`Assert.Contains("Passed!", standardOutput, StringComparison.Ordinal);`
`91`		`- Assert.Contains($"{_expectedResult}", standardOutput, StringComparison.Ordinal);`
	`91`	`+ Assert.Contains($"{s_expectedResult}", standardOutput, StringComparison.Ordinal);`
`92`	`92`	`string coverageFileName = $"file.json";`
`93`	`93`	`Assert.True(File.Exists(Path.Combine(clonedTemplateProject.ProjectRootPath, coverageFileName)));`
`94`	`94`	`AssertCoverage(clonedTemplateProject, coverageFileName);`
`@@ -100,7 +100,7 @@ public void TestMsbuild_CoverletOutput_Folder_FileNameExtension()`
`100`	`100`	`using ClonedTemplateProject clonedTemplateProject = PrepareTemplateProject();`
`101`	`101`	`Assert.Equal(0, DotnetCli($"test -c {_buildConfiguration} -f {_buildTargetFramework} \"{clonedTemplateProject.ProjectRootPath}\" /p:CollectCoverage=true /p:Include=\"[{ClonedTemplateProject.AssemblyName}]*DeepThought\" /p:IncludeTestAssembly=true /p:CoverletOutput=\"{clonedTemplateProject.ProjectRootPath}\"\\file.ext", out string standardOutput, out string standardError));`
`102`	`102`	`Assert.Contains("Passed!", standardOutput, StringComparison.Ordinal);`
`103`		`- Assert.Contains($"{_expectedResult}", standardOutput, StringComparison.Ordinal);`
	`103`	`+ Assert.Contains($"{s_expectedResult}", standardOutput, StringComparison.Ordinal);`
`104`	`104`	`string coverageFileName = $"file.ext";`
`105`	`105`	`Assert.True(File.Exists(Path.Combine(clonedTemplateProject.ProjectRootPath, coverageFileName)));`
`106`	`106`	`AssertCoverage(clonedTemplateProject, coverageFileName);`
`@@ -124,7 +124,7 @@ public void TestMsbuild_CoverletOutput_Folder_FileNameExtension_SpecifyFramework`
`124`	`124`	`_output.WriteLine(standardOutput);`
`125`	`125`	`}`
`126`	`126`	`Assert.Contains("Passed!", standardOutput, StringComparison.Ordinal);`
`127`		`- Assert.Contains($"{_expectedResult}", standardOutput, StringComparison.Ordinal);`
	`127`	`+ Assert.Contains($"{s_expectedResult}", standardOutput, StringComparison.Ordinal);`
`128`	`128`	`Assert.True(File.Exists(Path.Combine(clonedTemplateProject.ProjectRootPath, "file.ext")));`
`129`	`129`	`AssertCoverage(clonedTemplateProject, "file.ext");`
`130`	`130`	`}`
`@@ -143,7 +143,7 @@ public void TestMsbuild_CoverletOutput_Folder_FileNameWithDoubleExtension()`
`143`	`143`	`_output.WriteLine(standardOutput);`
`144`	`144`	`}`
`145`	`145`	`Assert.Contains("Passed!", standardOutput, StringComparison.Ordinal);`
`146`		`- Assert.Contains($"{_expectedResult}", standardOutput, StringComparison.Ordinal);`
	`146`	`+ Assert.Contains($"{s_expectedResult}", standardOutput, StringComparison.Ordinal);`
`147`	`147`	`string coverageFileName = $"file.ext1.ext2";`
`148`	`148`	`Assert.True(File.Exists(Path.Combine(clonedTemplateProject.ProjectRootPath, coverageFileName)));`
`149`	`149`	`AssertCoverage(clonedTemplateProject, coverageFileName);`
`@@ -165,7 +165,7 @@ public void Test_MultipleTargetFrameworkReport_NoCoverletOutput()`
`165`	`165`	`_output.WriteLine(standardOutput);`
`166`	`166`	`}`
`167`	`167`	`Assert.Contains("Passed!", standardOutput, StringComparison.Ordinal);`
`168`		`- Assert.Contains($"{_expectedResult}", standardOutput, StringComparison.Ordinal);`
	`168`	`+ Assert.Contains($"{s_expectedResult}", standardOutput, StringComparison.Ordinal);`
`169`	`169`
`170`	`170`	`foreach (string targetFramework in targetFrameworks)`
`171`	`171`	`{`
`@@ -192,7 +192,7 @@ public void Test_MultipleTargetFrameworkReport_CoverletOutput_Folder()`
`192`	`192`	`}`
`193`	`193`	`Assert.Equal(0, result);`
`194`	`194`	`Assert.Contains("Passed!", standardOutput, StringComparison.Ordinal);`
`195`		`- Assert.Contains($"{_expectedResult}", standardOutput, StringComparison.Ordinal);`
	`195`	`+ Assert.Contains($"{s_expectedResult}", standardOutput, StringComparison.Ordinal);`
`196`	`196`
`197`	`197`	`foreach (string targetFramework in targetFrameworks)`
`198`	`198`	`{`
`@@ -219,7 +219,7 @@ public void Test_MultipleTargetFrameworkReport_CoverletOutput_Folder_FileNameWit`
`219`	`219`	`_output.WriteLine(standardOutput);`
`220`	`220`	`}`
`221`	`221`	`Assert.Contains("Passed!", standardOutput, StringComparison.Ordinal);`
`222`		`- Assert.Contains($"{_expectedResult}", standardOutput, StringComparison.Ordinal);`
	`222`	`+ Assert.Contains($"{s_expectedResult}", standardOutput, StringComparison.Ordinal);`
`223`	`223`
`224`	`224`	`foreach (string targetFramework in targetFrameworks)`
`225`	`225`	`{`
`@@ -249,7 +249,7 @@ public void Test_MultipleTargetFrameworkReport_CoverletOutput_Folder_FileNameWit`
`249`	`249`	`_output.WriteLine(standardOutput);`
`250`	`250`	`}`
`251`	`251`	`Assert.Contains("Passed!", standardOutput, StringComparison.Ordinal);`
`252`		`- Assert.Contains($"{_expectedResult}", standardOutput, StringComparison.Ordinal);`
	`252`	`+ Assert.Contains($"{s_expectedResult}", standardOutput, StringComparison.Ordinal);`
`253`	`253`
`254`	`254`	`foreach (string targetFramework in targetFrameworks)`
`255`	`255`	`{`
`@@ -282,7 +282,7 @@ public void Test_MultipleTargetFrameworkReport_CoverletOutput_Folder_FileNameWit`
`282`	`282`	`_output.WriteLine(standardOutput);`
`283`	`283`	`}`
`284`	`284`	`Assert.Contains("Passed!", standardOutput, StringComparison.Ordinal);`
`285`		`- Assert.Contains($"{_expectedResult}", standardOutput, StringComparison.Ordinal);`
	`285`	`+ Assert.Contains($"{s_expectedResult}", standardOutput, StringComparison.Ordinal);`
`286`	`286`
`287`	`287`	`foreach (string targetFramework in targetFrameworks)`
`288`	`288`	`{`
`@@ -308,7 +308,7 @@ public void Test_MultipleTargetFrameworkReport_CoverletOutput_Folder_FileNameWit`
`308`	`308`	`_output.WriteLine(standardOutput);`
`309`	`309`	`}`
`310`	`310`	`Assert.Contains("Passed!", standardOutput, StringComparison.Ordinal);`
`311`		`- Assert.Contains($"{_expectedResult}", standardOutput, StringComparison.Ordinal);`
	`311`	`+ Assert.Contains($"{s_expectedResult}", standardOutput, StringComparison.Ordinal);`
`312`	`312`
`313`	`313`	`foreach (string targetFramework in targetFrameworks)`
`314`	`314`	`{`