Fix CI to send coverage results on push (#529)

Author: sungam3r

.github/workflows/test.yml

@@ -5,31 +5,24 @@ on:
     branches:
       - master
       - develop
-
+    paths:
+      - src/**
+      - .github/workflows/**
   # Upload code coverage results when PRs are merged
   push:
     branches:
       - master
       - develop
+    paths:
+      - src/**
+      - .github/workflows/**
 
 env:
   DOTNET_NOLOGO: true
   DOTNET_CLI_TELEMETRY_OPTOUT: true
 
 jobs:
-  should_test:
-    runs-on: ubuntu-latest
-    outputs:
-      should_skip: ${{ steps.skip_check.outputs.should_skip }}
-    steps:
-      - id: skip_check
-        uses: fkirc/[email protected]
-        with:
-          github_token: ${{ github.token }}
-          paths_ignore: '["README.md", "LICENSE.md", ".gitignore"]'
   test:
-    needs: should_test
-    if: ${{ needs.should_test.outputs.should_skip != 'true' }}
     strategy:
       matrix:
         os: [ubuntu-latest, windows-latest]
@@ -79,8 +72,8 @@ jobs:
     if: always()
     steps:
       - name: Pass build check
-        if: ${{ needs.test.result == 'success' || needs.test.result == 'skipped' }}
+        if: ${{ needs.test.result == 'success' }}
         run: exit 0
       - name: Fail build check
-        if: ${{ needs.test.result != 'success' && needs.test.result != 'skipped' }}
+        if: ${{ needs.test.result != 'success' }}
         run: exit 1

src/Authorization.AspNetCore/AuthorizationError.cs

@@ -75,7 +75,7 @@ public static void AppendFailureHeader(StringBuilder error, OperationType? opera
         {
             error.Append("You are not authorized to run this ")
                 .Append(GetOperationType(operationType))
-                .Append(".");
+                .Append('.');
         }
 
         /// <summary>

tests/Authorization.AspNetCore.Tests/AuthorizationValidationRuleTests.cs

@@ -264,7 +264,7 @@ public void fails_on_missing_claim_on_connection_type()
             });
         }
 
-        private ISchema BasicSchema<T>()
+        private static ISchema BasicSchema<T>()
         {
             string defs = @"
                 type Query {