Fix missing code coverage for Steeltoe.Management.Endpoint (#1537)

* Fix missing code coverage for Steeltoe.Management.Endpoint

* Tweak runsettings
- Hide coverage for RazorPagesTestWebApp
- Remove settings that match the defaults

* Coverage tweaks, fix Sonar error

Author: bart-vmware

coverlet.runsettings

@@ -5,12 +5,10 @@
       <DataCollector friendlyName="XPlat Code Coverage">
         <Configuration>
           <Format>opencover</Format>
-          <Exclude>[*.Test*]*,[*]Microsoft.Diagnostics*</Exclude>
-          <Include>[Steeltoe.*]*</Include>
-          <ExcludeByAttribute>Obsolete,GeneratedCodeAttribute,CompilerGeneratedAttribute</ExcludeByAttribute>
+          <Exclude>[ConfigurationSchemaGenerator]*</Exclude>
+          <ExcludeByFile>**/test/**/*.*</ExcludeByFile>
+          <ExcludeByAttribute>ObsoleteAttribute,GeneratedCodeAttribute,CompilerGeneratedAttribute</ExcludeByAttribute>
           <UseSourceLink>true</UseSourceLink>
-          <IncludeTestAssembly>false</IncludeTestAssembly>
-          <SkipAutoProps>false</SkipAutoProps>
         </Configuration>
       </DataCollector>
     </DataCollectors>

src/Common/src/Common/ConfigurationSchemaAttributes.cs

@@ -6,12 +6,15 @@
 // Steeltoe: This file was copied from the .NET Aspire Configuration Schema generator
 // at https://github.com/dotnet/aspire/tree/cb7cc4d78f8dd2b4df1053a229493cdbf88f50df/src/Tools/ConfigurationSchemaGenerator.
 
+using System.Diagnostics.CodeAnalysis;
+
 namespace Aspire;
 
 /// <summary>
 /// Attribute used to automatically generate a JSON schema for a component's configuration.
 /// </summary>
 [AttributeUsage(AttributeTargets.Assembly, AllowMultiple = true)]
+[ExcludeFromCodeCoverage(Justification = "Used while compiling Steeltoe, does not ship with Steeltoe.")]
 internal sealed class ConfigurationSchemaAttribute : Attribute
 {
     public ConfigurationSchemaAttribute(string path, Type type, string[]? exclusionPaths = null)

src/Management/src/Endpoint/Actuators/Health/Contributors/FileSystem/NetworkShareWrapper.cs

@@ -1,7 +1,8 @@
-// Licensed to the .NET Foundation under one or more agreements.
+// Licensed to the .NET Foundation under one or more agreements.
 // The .NET Foundation licenses this file to you under the Apache 2.0 License.
 // See the LICENSE file in the project root for more information.
 
+using System.Diagnostics.CodeAnalysis;
 using System.Runtime.InteropServices;
 
 namespace Steeltoe.Management.Endpoint.Actuators.Health.Contributors.FileSystem;
@@ -26,6 +27,7 @@ private NetworkShareWrapper(ulong freeBytesAvailable, ulong totalNumberOfBytes)
             : null;
     }
 
+    [ExcludeFromCodeCoverage(Justification = "Workaround for https://github.com/coverlet-coverage/coverlet/issues/1762")]
     private static partial class NativeMethods
     {
         [LibraryImport("kernel32.dll", EntryPoint = "GetDiskFreeSpaceExW", StringMarshalling = StringMarshalling.Utf16, SetLastError = true)]

src/Management/src/Endpoint/Actuators/ThreadDump/EventPipeThreadDumper.cs

@@ -77,7 +77,9 @@ public async Task<IList<ThreadInfo>> DumpThreadsAsync(CancellationToken cancella
             }
             finally
             {
+#pragma warning disable S1215 // "GC.Collect" should not be called
                 long totalMemory = GC.GetTotalMemory(true);
+#pragma warning restore S1215 // "GC.Collect" should not be called
                 _logger.LogDebug("Total memory: {Memory}.", totalMemory);
             }
         }, cancellationToken);