Add global exclusion from code coverage to test directory

Remove old exclusion attributes
Append code coverage to action summary.

Author: blowdart

.github/workflows/ci-build.yml

@@ -21,6 +21,7 @@ permissions:
 env:
   DOTNET_GENERATE_ASPNET_CERTIFICATE: false
   DOTNET_SKIP_FIRST_TIME_EXPERIENCE: true
+  COVERAGE_REPORT: ${{ github.workspace}}/coveragereport
 
 jobs:
   build:
@@ -74,12 +75,25 @@ jobs:
         show: "fail, skip"
       if: always()
 
+    - name: Publish Test Results
+      uses: EnricoMi/publish-unit-test-result-action@3a74b2957438d0b6e2e61d67b05318aa25c9e6c6 # v2.20.0
+      if: always()
+      with:
+        files: "test/**/TestResults.xml"
+
+    - name: Upload Test Artifacts
+      uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
+      with:
+        name: test-results
+        path: "test/**/TestResults.xml"
+        retention-days: 5
+
     - name: 'Generate Coverage Reports'
       uses: danielpalme/ReportGenerator-GitHub-Action@82ef2bc4c83f42bbeec82a5cd9a8e3f6156b64b9 # 5.4.9
       with:
         reports: "test/**/coverage.cobertura.xml"
-        targetdir: "${{ github.workspace }}"
-        reporttypes: "Cobertura" 
+        targetdir: "${{ env.COVERAGE_REPORT }}"
+        reporttypes: "Cobertura,MarkdownSummaryGithub" 
         verbosity: "Info" 
         title: "Code Coverage" 
         tag: "${{ github.run_number }}_${{ github.run_id }}"
@@ -103,20 +117,9 @@ jobs:
       uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
       with:
         name: coverage-results
-        path: |
-          ${{ github.workspace }}/Cobertura.xml
-          ${{ github.workspace }}/code-coverage-results.md
+        path: ${{ env.COVERAGE_REPORT }}
         retention-days: 5
 
-    - name: Publish Test Results
-      uses: EnricoMi/publish-unit-test-result-action@3a74b2957438d0b6e2e61d67b05318aa25c9e6c6 # v2.20.0
-      if: always()
-      with:
-        files: "test/**/TestResults.xml"
-
-    - name: Upload Test Artifacts
-      uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
-      with:
-        name: test-results
-        path: "test/**/TestResults.xml"
-        retention-days: 5
+    - name: Add coverage to build summary
+      shell: bash
+      run: cat ${COVERAGE_REPORT}/SummaryGithub.md >> $GITHUB_STEP_SUMMARY

test/Directory.Build.props

@@ -5,6 +5,10 @@
     <IsPublishable>False</IsPublishable>
   </PropertyGroup>
 
+  <ItemGroup>
+    <AssemblyAttribute Include="System.Diagnostics.CodeAnalysis.ExcludeFromCodeCoverage" />
+  </ItemGroup>
+
   <ItemGroup>
     <Using Include="System.Diagnostics.CodeAnalysis" />
     <Using Include="Xunit" />
@@ -23,4 +27,5 @@
     </PackageReference>
     <PackageReference Include="JunitXml.TestLogger" />
   </ItemGroup>
+
 </Project>

test/idunno.AtProto.Test/AtCidTests.cs

@@ -3,7 +3,6 @@
 
 namespace idunno.AtProto.Test
 {
-    [ExcludeFromCodeCoverage]
     public class AtCidTests
     {
         const string ValidCid = "bafyreievgu2ty7qbiaaom5zhmkznsnajuzideek3lo7e65dwqlrvrxnmo4";

test/idunno.AtProto.Test/AtIdentifierTests.cs

@@ -3,7 +3,6 @@
 
 namespace idunno.AtProto.Test
 {
-    [ExcludeFromCodeCoverage]
     public class AtIdentifierTests
     {
         // Test cases from https://github.com/bluesky-social/atproto/blob/main/interop-test-files/syntax/atidentifier_syntax_valid.txt

test/idunno.AtProto.Test/AtUriTests.cs

@@ -3,7 +3,6 @@
 
 namespace idunno.AtProto.Test
 {
-    [ExcludeFromCodeCoverage]
     public class AtUriTests
     {
         [Theory]

test/idunno.AtProto.Test/BuilderTests.cs

@@ -2,16 +2,12 @@
 // Licensed under the MIT License.
 
 using System.Net.Http.Headers;
-using System.Threading.Tasks;
 using idunno.AtProto.Authentication;
 using Microsoft.Extensions.Logging;
 using Microsoft.Extensions.Logging.Abstractions;
-using Microsoft.Extensions.Options;
-using static System.Formats.Asn1.AsnWriter;
 
 namespace idunno.AtProto.Test
 {
-    [ExcludeFromCodeCoverage]
     public class BuilderTests
     {
         [Fact]

test/idunno.AtProto.Test/DidTests.cs

@@ -3,7 +3,6 @@
 
 namespace idunno.AtProto.Test
 {
-    [ExcludeFromCodeCoverage]
     public class DidTests
     {
         // Test cases from https://github.com/bluesky-social/atproto/blob/main/interop-test-files/syntax/did_syntax_valid.txt

test/idunno.AtProto.Test/HandleTests.cs

@@ -3,7 +3,6 @@
 
 namespace idunno.AtProto.Test
 {
-    [ExcludeFromCodeCoverage]
     public class HandleTests
     {
         [Fact]

test/idunno.AtProto.Test/HttpResultTests.cs

@@ -5,7 +5,6 @@
 
 namespace idunno.AtProto.Test
 {
-    [ExcludeFromCodeCoverage]
     public class HttpResultTests
     {
         [Fact]

test/idunno.AtProto.Test/NsidTests.cs

@@ -3,7 +3,6 @@
 
 namespace idunno.AtProto.Test
 {
-    [ExcludeFromCodeCoverage]
     public class NsidTests
     {
         // Test cases from https://github.com/bluesky-social/atproto/blob/290a7e67b8e6417b00352cc1d54bac006c2f6f93/interop-test-files/syntax/nsid_syntax_valid.txt