implement stepsecurity policy for release workflow #51

Bump docker/metadata-action from 3.8.0 to 4.0.1

.github/dependabot.yml

@@ -1,7 +1,13 @@
 version: 2
 updates:
 - package-ecosystem: nuget
-  directory: "/"
+  directory: "/src"
   schedule:
     interval: daily
   open-pull-requests-limit: 10
+
+- package-ecosystem: "github-actions"
+  directory: "/"
+  schedule:
+    interval: "daily"
+  open-pull-requests-limit: 5

.github/workflows/assign-to-project.yml

@@ -8,34 +8,47 @@ on:
 env:
   GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
+permissions:
+  contents: read
+
 jobs:
   assign-to-project:
+    permissions:
+      repository-projects: write  # for srggrs/assign-one-project-github-action to assign issues and PRs to repo project
     runs-on: ubuntu-latest
     name: Assign to Project
     steps:
+    
+    - name: Harden Runner
+      uses: step-security/harden-runner@74b568e8591fbb3115c70f3436a0c6b0909a8504
+      with:
+        egress-policy: block
+        allowed-endpoints: >
+          api.github.com:443
+
     - name: Assign Issues to Bugs
-      uses: srggrs/assign-one-project-github-action@1.3.1
+      uses: srggrs/assign-one-project-github-action@4d59cc619499b55ca689fb13cfcc72324a8b8435
       if: contains(github.event.issue.labels.*.name, 'bug')
       with:
         project: 'https://github.com/irongut/CodeCoverageSummary/projects/1'
         column_name: 'Needs triage'
 
     - name: Assign Issues to Enhancements
-      uses: srggrs/assign-one-project-github-action@1.3.1
+      uses: srggrs/assign-one-project-github-action@4d59cc619499b55ca689fb13cfcc72324a8b8435
       if: contains(github.event.issue.labels.*.name, 'enhancement')
       with:
         project: 'https://github.com/irongut/CodeCoverageSummary/projects/2'
         column_name: 'To do'
 
     - name: Assign PRs to Bugs
-      uses: srggrs/assign-one-project-github-action@1.3.1
+      uses: srggrs/assign-one-project-github-action@4d59cc619499b55ca689fb13cfcc72324a8b8435
       if: contains(github.event.pull_request.labels.*.name, 'bug')
       with:
         project: 'https://github.com/irongut/CodeCoverageSummary/projects/1'
         column_name: 'In Progress'
 
     - name: Assign PRs to Enhancements
-      uses: srggrs/assign-one-project-github-action@1.3.1
+      uses: srggrs/assign-one-project-github-action@4d59cc619499b55ca689fb13cfcc72324a8b8435
       if: contains(github.event.pull_request.labels.*.name, 'enhancement')
       with:
         project: 'https://github.com/irongut/CodeCoverageSummary/projects/2'

.github/workflows/auto-assign-pr.yml

@@ -11,7 +11,15 @@ jobs:
   assignAuthor:
     runs-on: ubuntu-latest
     steps:
+
+    - name: Harden Runner
+      uses: step-security/harden-runner@74b568e8591fbb3115c70f3436a0c6b0909a8504
+      with:
+        egress-policy: block
+        allowed-endpoints: >
+          api.github.com:443
+
     - name: Auto Assign PR
-      uses: samspills/assign-pr-to-author@v1.0.1
+      uses: samspills/assign-pr-to-author@b313feb250ff414d3aff26525b986f080ee7bd7a
       with:
         repo-token: '${{ secrets.GITHUB_TOKEN }}'

.github/workflows/ci-build.yml

@@ -6,16 +6,36 @@ on:
   pull_request:
     branches: [ master ]
 
+permissions:
+  contents: read
+
+env:
+  DOTNET_NOLOGO: true                     # Disable the .NET logo in the console output
+  DOTNET_SKIP_FIRST_TIME_EXPERIENCE: true # Disable the .NET first time experience to skip caching NuGet packages and speed up the build
+  DOTNET_CLI_TELEMETRY_OPTOUT: true       # Disable sending .NET CLI telemetry to Microsoft
+
 jobs:
   build:
     runs-on: ubuntu-latest
     name: CI Build
     steps:
+    
+    - name: Harden Runner
+      uses: step-security/harden-runner@74b568e8591fbb3115c70f3436a0c6b0909a8504
+      with:
+        egress-policy: block
+        allowed-endpoints: >
+          api.nuget.org:443
+          dotnetbuilds.azureedge.net:443
+          dotnetcli.azureedge.net:443
+          dotnetcli.blob.core.windows.net:443
+          github.com:443
+
     - name: Checkout
-      uses: actions/checkout@v2
+      uses: actions/checkout@d171c3b028d844f2bf14e9fdec0c58114451e4bf
 
     - name: Setup .Net
-      uses: actions/setup-dotnet@v1
+      uses: actions/setup-dotnet@c0d4ad69d8bd405d234f1c9166d383b7a4f69ed8
       with:
         dotnet-version: 6.0.x
 
@@ -25,5 +45,5 @@ jobs:
     - name: Build CodeCoverageSummary
       run: dotnet build src/CodeCoverageSummary.sln --configuration Release --no-restore
 
-    - name: Test with sample file
+    - name: Test with sample files
-      run: dotnet src/CodeCoverageSummary/bin/Release/net6.0/CodeCoverageSummary.dll --files src/coverage.cobertura.xml --badge true
+      run: dotnet src/CodeCoverageSummary/bin/Release/net6.0/CodeCoverageSummary.dll --files **/coverage.*.xml --badge true

.github/workflows/codeql-scan.yml

@@ -0,0 +1,73 @@
+name: CodeQL Scan
+
+on:
+  push:
+    branches: [ "master" ]
+    paths-ignore:
+      - '**/*.md'
+      - '**/*.gitignore'
+      - '**/*.gitattributes'
+  pull_request:
+    branches: [ "master" ]
+  schedule:
+    - cron: '15 5 * * 5'
+
+permissions:  # added using https://github.com/step-security/secure-workflows
+  contents: read
+
+env:
+  DOTNET_NOLOGO: true                     # Disable the .NET logo in the console output
+  DOTNET_SKIP_FIRST_TIME_EXPERIENCE: true # Disable the .NET first time experience to skip caching NuGet packages and speed up the build
+  DOTNET_CLI_TELEMETRY_OPTOUT: true       # Disable sending .NET CLI telemetry to Microsoft
+
+jobs:
+  analyze:
+    name: Analyze
+    runs-on: ubuntu-latest
+    permissions:
+      actions: read
+      contents: read
+      security-events: write
+
+    strategy:
+      fail-fast: false
+      matrix:
+        language: [ 'csharp' ]
+
+    steps:
+    
+    - name: Harden Runner
+      uses: step-security/harden-runner@74b568e8591fbb3115c70f3436a0c6b0909a8504
+      with:
+        egress-policy: block
+        allowed-endpoints: >
+          api.github.com:443
+          api.nuget.org:443
+          dotnetbuilds.azureedge.net:443
+          dotnetcli.azureedge.net:443
+          dotnetcli.blob.core.windows.net:443
+          github.com:443
+          uploads.github.com:443
+
+    - name: Checkout repository
+      uses: actions/checkout@d0651293c4a5a52e711f25b41b05b2212f385d28
+
+    - name: Initialize CodeQL
+      uses: github/codeql-action/init@74e8f231851deb9b54c3e408f88638dd39727868
+      with:
+        languages: ${{ matrix.language }}
+        # queries: security-extended,security-and-quality # https://docs.github.com/en/code-security/code-scanning/automatically-scanning-your-code-for-vulnerabilities-and-errors/configuring-code-scanning#using-queries-in-ql-packs
+
+    - name: Setup .Net
+      uses: actions/setup-dotnet@c0d4ad69d8bd405d234f1c9166d383b7a4f69ed8
+      with:
+        dotnet-version: 6.0.x
+
+    - name: Restore Dependencies
+      run: dotnet restore src/CodeCoverageSummary.sln
+
+    - name: Build CodeCoverageSummary
+      run: dotnet build src/CodeCoverageSummary.sln --configuration Release --no-restore /p:UseSharedCompilation=false
+
+    - name: Perform CodeQL Analysis
+      uses: github/codeql-action/analyze@74e8f231851deb9b54c3e408f88638dd39727868

.github/workflows/mark-stale.yml

@@ -4,14 +4,27 @@ on:
   schedule:
   - cron: "30 1 * * *"
 
+permissions:
+  contents: read
+
 jobs:
   stale:
 
+    permissions:
+      issues: write  # for actions/stale to close stale issues
+      pull-requests: write  # for actions/stale to close stale PRs
     runs-on: ubuntu-latest
-
     steps:
+
+    - name: Harden Runner
+      uses: step-security/harden-runner@74b568e8591fbb3115c70f3436a0c6b0909a8504
+      with:
+        egress-policy: block
+        allowed-endpoints: >
+          api.github.com:443
+
     - name: Mark Stale
-      uses: actions/stale@v3
+      uses: actions/stale@9c1b1c6e115ca2af09755448e0dbba24e5061cc8
       with:
         repo-token: ${{ secrets.GITHUB_TOKEN }}
         exempt-all-milestones: true

.github/workflows/pr-labeler.yml

@@ -7,10 +7,24 @@ name: PR Labeler
 on: 
   pull_request_target:
 
+permissions:
+  contents: read
+
 jobs:
   label:
+    permissions:
+      contents: read  # for actions/labeler to determine modified files
+      pull-requests: write  # for actions/labeler to add labels to PRs
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/labeler@v3
+    
+    - name: Harden Runner
+      uses: step-security/harden-runner@74b568e8591fbb3115c70f3436a0c6b0909a8504
+      with:
+        egress-policy: block
+        allowed-endpoints: >
+          api.github.com:443
+
+    - uses: actions/labeler@472c5d3aaacde439785e94966eb2e545627f4935
       with:
         repo-token: "${{ secrets.GITHUB_TOKEN }}"

.github/workflows/release-build.yml

@@ -4,7 +4,13 @@ on:
   release:
     types: [published]
 
+permissions:
+  contents: read
+
 env:
+  DOTNET_NOLOGO: true                     # Disable the .NET logo in the console output
+  DOTNET_SKIP_FIRST_TIME_EXPERIENCE: true # Disable the .NET first time experience to skip caching NuGet packages and speed up the build
+  DOTNET_CLI_TELEMETRY_OPTOUT: true       # Disable sending .NET CLI telemetry to Microsoft
   REGISTRY: ghcr.io
   IMAGE_NAME: ${{ github.repository }}
 
@@ -13,13 +19,25 @@ jobs:
     name: Test Build
     runs-on: ubuntu-latest
     steps:
+    
+    - name: Harden Runner
+      uses: step-security/harden-runner@74b568e8591fbb3115c70f3436a0c6b0909a8504
+      with:
+        egress-policy: block
+        allowed-endpoints: >
+          api.nuget.org:443
+          dotnetbuilds.azureedge.net:443
+          dotnetcli.azureedge.net:443
+          dotnetcli.blob.core.windows.net:443
+          github.com:443
+
     - name: Checkout
-      uses: actions/checkout@v2
+      uses: actions/checkout@d171c3b028d844f2bf14e9fdec0c58114451e4bf
       with:
         fetch-depth: 0
 
     - name: Setup .Net
-      uses: actions/setup-dotnet@v1
+      uses: actions/setup-dotnet@c0d4ad69d8bd405d234f1c9166d383b7a4f69ed8
       with:
         dotnet-version: 6.0.x
 
@@ -30,7 +48,7 @@ jobs:
       run: dotnet build src/CodeCoverageSummary.sln --configuration Release --no-restore
 
     - name: Test with sample file
-      run: dotnet src/CodeCoverageSummary/bin/Release/net6.0/CodeCoverageSummary.dll --files src/coverage.cobertura.xml --badge true
+      run: dotnet src/CodeCoverageSummary/bin/Release/net6.0/CodeCoverageSummary.dll --files **/coverage.*.xml --badge true
 
   deploy:
     name: Deploy to GHCR
@@ -39,12 +57,40 @@ jobs:
     permissions:
       contents: read
       packages: write
+      id-token: write # Used for identity challenge with sigstore/fulcio
+
     steps:
+
+      - name: Harden Runner
+        uses: step-security/harden-runner@74b568e8591fbb3115c70f3436a0c6b0909a8504
+        with:
+          egress-policy: block
+          allowed-endpoints: >
+            api.github.com:443
+            api.nuget.org:443
+            auth.docker.io:443
+            fulcio.sigstore.dev:443
+            ghcr.io:443
+            github.com:443
+            mcr.microsoft.com:443
+            pipelines.actions.githubusercontent.com:443
+            pkg-containers.githubusercontent.com:443
+            registry-1.docker.io:443
+            storage.googleapis.com:443
+
       - name: Checkout
-        uses: actions/checkout@v2
+        uses: actions/checkout@d171c3b028d844f2bf14e9fdec0c58114451e4bf
+
+      - name: Install Cosign
+        uses: sigstore/cosign-installer@c68f43abf1ae5df2528c9c250088fa14ed2d0ef5
+        with:
+          cosign-release: 'v1.9.0'
+
+      - name: Setup Docker Buildx
+        uses: docker/setup-buildx-action@dc7b9719a96d48369863986a06765841d7ea23f6
 
       - name: Login to GitHub Container Registry
-        uses: docker/login-action@v1
+        uses: docker/login-action@49ed152c8eca782a232dede0303416e8f356c37b
         with:
           registry: ${{ env.REGISTRY }}
           username: ${{ github.repository_owner }}
@@ -52,14 +98,23 @@ jobs:
 
       - name: Extract Docker metadata
         id: meta
-        uses: docker/metadata-action@v3
+        uses: docker/metadata-action@69f6fc9d46f2f8bf0d5491e4aabe0bb8c6a4678a
         with:
           images: ${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}
 
       - name: Build + Push Docker image
-        uses: docker/build-push-action@v2
+        id: build-and-push
+        uses: docker/build-push-action@1cb9d22b932e4832bb29793b7777ec860fc1cde0
         with:
           context: .
           push: true
           tags: ${{ steps.meta.outputs.tags }}
           labels: ${{ steps.meta.outputs.labels }}
+
+      # Sign the Docker image digest
+      # Uses the identity token to provision an ephemeral certificate against the community Fulcio instance
+      # https://github.com/sigstore/cosign
+      - name: Sign the Docker image
+        env:
+          COSIGN_EXPERIMENTAL: "true"
+        run: echo "${{ steps.meta.outputs.tags }}" | xargs -I {} cosign sign {}@${{ steps.build-and-push.outputs.digest }}

.github/workflows/test-linux.yml

@@ -10,10 +10,14 @@ jobs:
     runs-on: ubuntu-latest
     name: CI Build
     steps:
+    
+    - name: Checkout
+      uses: actions/checkout@v2
+      
     - name: Test Action
       uses: irongut/CodeCoverageSummary@master
       with:
-        filename: /app/sample.coverage.xml,/app/sample.coverage.xml
+        filename: /src/coverage.*.xml
         badge: true
         fail_below_min: true
         format: markdown

README.md

@@ -1,20 +1,24 @@
 # Code Coverage Summary
 
 <div align="center">
-  
+
-[![CI Build](https://github.com/irongut/CodeCoverageSummary/actions/workflows/ci-build.yml/badge.svg)](https://github.com/irongut/CodeCoverageSummary/actions/workflows/ci-build.yml)
-&nbsp;
-[![GitHub](https://img.shields.io/badge/GitHub-irongut/CodeCoverageSummary-informational?style=flat&logo=github)](https://github.com/irongut/CodeCoverageSummary)
-&nbsp;
 ![.NET 6.0](https://img.shields.io/badge/Version-.NET%206.0-informational?style=flat&logo=dotnet)
 &nbsp;
 ![Built With Docker](https://img.shields.io/badge/Built_With-Docker-informational?style=flat&logo=docker)
-  
+&nbsp;
+[![OpenSSF Best Practices](https://bestpractices.coreinfrastructure.org/projects/6292/badge)](https://bestpractices.coreinfrastructure.org/projects/6292)
+&nbsp;
+[![CI Build](https://github.com/irongut/CodeCoverageSummary/actions/workflows/ci-build.yml/badge.svg)](https://github.com/irongut/CodeCoverageSummary/actions/workflows/ci-build.yml)
+&nbsp;
+[![CodeQL Scan](https://github.com/irongut/CodeCoverageSummary/actions/workflows/codeql-scan.yml/badge.svg)](https://github.com/irongut/CodeCoverageSummary/actions/workflows/codeql-scan.yml)
+
 </div>
 
 A GitHub Action that reads Cobertura format code coverage files from your test suite and outputs a text or markdown summary. This summary can be posted as a Pull Request comment or included in Release Notes by other actions to give you an immediate insight into the health of your code without using a third-party site.
 
-Code Coverage Summary is designed for use with [Coverlet](https://github.com/coverlet-coverage/coverlet) and [gcovr](https://github.com/gcovr/gcovr) but it should work with any test framework that outputs coverage in Cobertura format. If it doesn't work with your tooling please [open an issue][new-issue] to discuss the problem.
+Code Coverage Summary is designed for use with any test framework that outputs coverage in Cobertura XML format including [Coverlet](https://github.com/coverlet-coverage/coverlet), [gcovr](https://github.com/gcovr/gcovr), [simplecov](https://github.com/simplecov-ruby/simplecov) and [MATLAB](https://uk.mathworks.com/help/matlab/ref/matlab.unittest.plugins.codecoverageplugin-class.html). See the [FAQ](https://github.com/irongut/CodeCoverageSummary/wiki/Frequently-Asked-Questions#which-testing-tools-does-ccs-work-with) for more details. If it doesn't work with your tooling please [open an issue][new-issue] to discuss the problem.
+
+Code Coverage Summary is compatible with [StepSecurity Secure Workflows](https://github.com/step-security/secure-workflows) and uses a Docker image that is cryptographically signed using [Sigstore](https://www.sigstore.dev/). For instructions how to verify the Docker image please see the [Wiki](https://github.com/irongut/CodeCoverageSummary/wiki/Verify-the-Docker-Image).
 
 As a Docker based action Code Coverage Summary requires a Linux runner, see [Types of Action](https://docs.github.com/en/actions/creating-actions/about-custom-actions#types-of-actions). If you need to build with a Windows or MacOS runner a workaround would be to upload the coverage file as an artifact and use a separate job with a Linux runner to generate the summary.
 
@@ -24,7 +28,7 @@ As a Docker based action Code Coverage Summary requires a Linux runner, see [Typ
 ### `filename`
 **Required**
 
-A comma separated list of code coverage files to analyse. If there are any spaces in a path or filename this value must be in quotes.
+A comma separated list of code coverage files to analyse. Also supports using glob patterns to match multiple files. If there are any spaces in a path or filename this value must be in quotes.
 
 Note: Coverlet creates the coverage file in a random named directory (guid) so you need to copy it to a predictable path before running this Action, see the [.Net Workflow Example](#net-workflow-example) below.
 
@@ -54,12 +58,12 @@ Output Format - `markdown` or `text` (default).
 
 ### `hide_branch_rate`
 
-Hide Branch Rate values in the output - `true` or `false` (default).
+Hide Branch Rate metrics in the output - `true` or `false` (default).
 
 
 ### `hide_complexity`
 
-Hide Complexity values in the output - `true` or `false` (default).
+Hide Complexity metrics in the output - `true` or `false` (default).
 
 
 ### `indicators`
@@ -121,7 +125,7 @@ Minimum allowed line rate is 50%
 
 ```yaml
 name: Code Coverage Summary Report
-uses: irongut/CodeCoverageSummary@v1.2.0
+uses: irongut/CodeCoverageSummary@v1.3.0
 with:
   filename: coverage.cobertura.xml
 ```
@@ -164,7 +168,7 @@ jobs:
       run: cp coverage/**/coverage.cobertura.xml coverage.cobertura.xml
 
     - name: Code Coverage Summary Report
-      uses: irongut/CodeCoverageSummary@v1.2.0
+      uses: irongut/CodeCoverageSummary@v1.3.0
       with:
         filename: coverage.cobertura.xml
         badge: true
@@ -185,6 +189,16 @@ jobs:
 ```
 
 
+## Version Numbers
+
+Version numbers will be assigned according to the [Semantic Versioning](https://semver.org/) scheme.
+This means, given a version number MAJOR.MINOR.PATCH, we will increment the:
+
+1. MAJOR version when we make incompatible API changes
+2. MINOR version when we add functionality in a backwards compatible manner
+3. PATCH version when we make backwards compatible bug fixes
+
+
 ## Contributing
 
 ### Report Bugs

SECURITY.md

@@ -0,0 +1,26 @@
+# Security Policy
+
+## Supported Versions
+
+Version | Supported         
+------- | ------------------
+1.3.0 | :white_check_mark:
+1.3.0-beta | :x:
+1.2.0 | :x:
+1.1.0 | :x:
+1.0.x | :x:
+
+Only the latest version is supported. If you find a security vulnerability in an eariler version please check it exists in the latest version before reporting.
+
+
+## Reporting a Vulnerability
+
+Please make sure the vulnerability is not already reported by searching existing [issues].
+
+If you're unable to find an existing issue addressing the vulnerability please [open a new issue][new-issue]. 
+Be sure to include a title and clear description and as much relevant information as possible.
+Please tag the issue with the `Security` label.
+
+
+[issues]: https://github.com/irongut/CodeCoverageSummary/issues
+[new-issue]: https://github.com/irongut/CodeCoverageSummary/issues/new

action.yml

@@ -6,7 +6,7 @@ branding:
   color: purple
 inputs:
   filename:
-    description: 'A comma separated list of code coverage files to analyse.'
+    description: 'A comma separated list of code coverage files to analyse. Also supports using glob patterns to match multiple files.'
     required: true
   badge:
     description: 'Include a Line Rate coverage badge in the output using shields.io - true / false (default).'
@@ -42,7 +42,7 @@ inputs:
     default: '50 75'
 runs:
   using: 'docker'
-  image: 'docker://ghcr.io/irongut/codecoveragesummary:v1.2.0'
+  image: 'docker://ghcr.io/irongut/codecoveragesummary:v1.3.0'
   args:
     - '--files'
     - ${{ inputs.filename }}

src/CodeCoverageSummary/CodeCoverageSummary.csproj

@@ -7,19 +7,20 @@
     <Company>Taranis Software</Company>
     <Authors>Irongut</Authors>
     <Description>A GitHub Action that reads Cobertura format code coverage files and outputs a text or markdown summary.</Description>
-    <Copyright>Copyright © 2021 Taranis Software</Copyright>
+    <Copyright>Copyright © 2021 - 2022 Taranis Software</Copyright>
     <PackageId>Taranis.CodeCoverageSummary</PackageId>
     <PackageLicenseExpression>MIT</PackageLicenseExpression>
     <PackageProjectUrl>https://github.com/irongut/CodeCoverageSummary</PackageProjectUrl>
     <RepositoryUrl>https://github.com/irongut/CodeCoverageSummary</RepositoryUrl>
     <RepositoryType>git</RepositoryType>
     <PackageTags>coverage test-coverage cobertura action code-coverage coverlet github-actions</PackageTags>
-    <Version>1.2.0</Version>
+    <Version>1.3.0</Version>
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="CommandLineParser" Version="2.8.0" />
+    <PackageReference Include="CommandLineParser" Version="2.9.1" />
-    <PackageReference Include="Microsoft.VisualStudio.Azure.Containers.Tools.Targets" Version="1.14.0" />
+    <PackageReference Include="Microsoft.Extensions.FileSystemGlobbing" Version="6.0.0" />
+    <PackageReference Include="Microsoft.VisualStudio.Azure.Containers.Tools.Targets" Version="1.16.1" />
   </ItemGroup>
 
 </Project>

src/CodeCoverageSummary/CommandLineOptions.cs

@@ -6,7 +6,7 @@ namespace CodeCoverageSummary
 {
     public class CommandLineOptions
     {
-        [Option(longName: "files", Separator = ',', Required = true, HelpText = "A comma separated list of code coverage files to analyse.")]
+        [Option(longName: "files", Separator = ',', Required = true, HelpText = "A comma separated list of code coverage files to analyse. Also accepts glob patterns.")]
         public IEnumerable<string> Files { get; set; }
 
         [Option(longName: "badge", Required = false, HelpText = "Include a Line Rate coverage badge in the output using shields.io - true or false.", Default = "false")]

src/CodeCoverageSummary/Program.cs

@@ -1,5 +1,7 @@
 using CommandLine;
+using Microsoft.Extensions.FileSystemGlobbing;
 using System;
+using System.Collections.Generic;
 using System.IO;
 using System.Linq;
 using System.Text;
@@ -19,33 +21,53 @@ namespace CodeCoverageSummary
                                  {
                                      try
                                      {
+                                         // use glob patterns to match files
+                                         Matcher matcher = new();
+                                         matcher.AddIncludePatterns(o.Files.ToArray());
+                                         IEnumerable<string> matchingFiles = matcher.GetResultsInFullPath(".");
+
+                                         if (matchingFiles?.Any() == false)
+                                         {
+                                             Console.WriteLine("Error: No files found matching glob pattern.");
+                                             return -2; // error
+                                         }
+
                                          // check files exist
-                                         foreach (var file in o.Files)
+                                         foreach (var file in matchingFiles)
                                          {
                                              if (!File.Exists(file))
                                              {
-                                                 Console.WriteLine($"Error: Code coverage file not found - {file}.");
+                                                 Console.WriteLine($"Error: Coverage file not found - {file}.");
                                                  return -2; // error
                                              }
                                          }
 
                                          // parse code coverage file
                                          CodeSummary summary = new();
-                                         foreach (var file in o.Files)
+                                         foreach (var file in matchingFiles)
                                          {
-                                             Console.WriteLine($"Code Coverage File: {file}");
+                                             Console.WriteLine($"Coverage File: {file}");
                                              summary = ParseTestResults(file, summary);
                                          }
-                                         summary.LineRate /= o.Files.Count();
+
-                                         summary.BranchRate /= o.Files.Count();
+                                         if (summary == null)
+                                             return -2; // error
+
+                                         summary.LineRate /= matchingFiles.Count();
+                                         summary.BranchRate /= matchingFiles.Count();
 
                                          if (summary.Packages.Count == 0)
                                          {
-                                             Console.WriteLine("Error: Parsing code coverage file, no packages found.");
+                                             Console.WriteLine("Parsing Error: No packages found in coverage files.");
                                              return -2; // error
                                          }
                                          else
                                          {
+                                             // hide branch rate if metrics missing
+                                             bool hideBranchRate = o.HideBranchRate;
+                                             if (summary.BranchRate == 0 && summary.BranchesCovered == 0 && summary.BranchesValid == 0)
+                                                 hideBranchRate = true;
+
                                              // set health badge thresholds
                                              if (!string.IsNullOrWhiteSpace(o.Thresholds))
                                                  SetThresholds(o.Thresholds);
@@ -59,14 +81,14 @@ namespace CodeCoverageSummary
                                              if (o.Format.Equals("text", StringComparison.OrdinalIgnoreCase))
                                              {
                                                  fileExt = "txt";
-                                                 output = GenerateTextOutput(summary, badgeUrl, o.Indicators, o.HideBranchRate, o.HideComplexity);
+                                                 output = GenerateTextOutput(summary, badgeUrl, o.Indicators, hideBranchRate, o.HideComplexity);
                                                  if (o.FailBelowThreshold)
                                                      output += $"Minimum allowed line rate is {lowerThreshold * 100:N0}%{Environment.NewLine}";
                                              }
                                              else if (o.Format.Equals("md", StringComparison.OrdinalIgnoreCase) || o.Format.Equals("markdown", StringComparison.OrdinalIgnoreCase))
                                              {
                                                  fileExt = "md";
-                                                 output = GenerateMarkdownOutput(summary, badgeUrl, o.Indicators, o.HideBranchRate, o.HideComplexity);
+                                                 output = GenerateMarkdownOutput(summary, badgeUrl, o.Indicators, hideBranchRate, o.HideComplexity);
                                                  if (o.FailBelowThreshold)
                                                      output += $"{Environment.NewLine}_Minimum allowed line rate is `{lowerThreshold * 100:N0}%`_{Environment.NewLine}";
                                              }
@@ -79,6 +101,7 @@ namespace CodeCoverageSummary
                                              // output
                                              if (o.Output.Equals("console", StringComparison.OrdinalIgnoreCase))
                                              {
+                                                 Console.WriteLine();
                                                  Console.WriteLine(output);
                                              }
                                              else if (o.Output.Equals("file", StringComparison.OrdinalIgnoreCase))
@@ -87,6 +110,7 @@ namespace CodeCoverageSummary
                                              }
                                              else if (o.Output.Equals("both", StringComparison.OrdinalIgnoreCase))
                                              {
+                                                 Console.WriteLine();
                                                  Console.WriteLine(output);
                                                  File.WriteAllText($"code-coverage-results.{fileExt}", output);
                                              }
@@ -116,6 +140,9 @@ namespace CodeCoverageSummary
 
         private static CodeSummary ParseTestResults(string filename, CodeSummary summary)
         {
+            if (summary == null)
+                return null;
+
             try
             {
                 string rss = File.ReadAllText(filename);
@@ -125,49 +152,73 @@ namespace CodeCoverageSummary
                 var coverage = from item in xdoc.Descendants("coverage")
                                select item;
 
+                if (!coverage.Any())
+                    throw new Exception("Coverage file invalid, data not found");
+
                 var lineR = from item in coverage.Attributes()
                             where item.Name == "line-rate"
                             select item;
+
+                if (!lineR.Any())
+                    throw new Exception("Overall line rate not found");
+
                 summary.LineRate += double.Parse(lineR.First().Value);
 
                 var linesCovered = from item in coverage.Attributes()
                                    where item.Name == "lines-covered"
                                    select item;
+
+                if (!linesCovered.Any())
+                    throw new Exception("Overall lines covered not found");
+
                 summary.LinesCovered += int.Parse(linesCovered.First().Value);
 
                 var linesValid = from item in coverage.Attributes()
                                  where item.Name == "lines-valid"
                                  select item;
+
+                if (!linesValid.Any())
+                    throw new Exception("Overall lines valid not found");
+
                 summary.LinesValid += int.Parse(linesValid.First().Value);
 
                 var branchR = from item in coverage.Attributes()
                               where item.Name == "branch-rate"
                               select item;
-                summary.BranchRate += double.Parse(branchR.First().Value);
 
-                var branchesCovered = from item in coverage.Attributes()
+                if (branchR.Any())
-                                      where item.Name == "branches-covered"
+                {
-                                      select item;
+                    summary.BranchRate += double.TryParse(branchR.First().Value, out double bRate) ? bRate : 0;
-                summary.BranchesCovered += int.Parse(branchesCovered.First().Value);
 
-                var branchesValid = from item in coverage.Attributes()
+                    var branchesCovered = from item in coverage.Attributes()
-                                    where item.Name == "branches-valid"
+                                          where item.Name == "branches-covered"
-                                    select item;
+                                          select item;
-                summary.BranchesValid += int.Parse(branchesValid.First().Value);
+
+                    summary.BranchesCovered += int.TryParse(branchesCovered?.First().Value ?? "0", out int bCovered) ? bCovered : 0;
+
+                    var branchesValid = from item in coverage.Attributes()
+                                        where item.Name == "branches-valid"
+                                        select item;
+
+                    summary.BranchesValid += int.TryParse(branchesValid?.First().Value ?? "0", out int bValid) ? bValid : 0;
+                }
 
                 // test coverage for individual packages
                 var packages = from item in coverage.Descendants("package")
                                select item;
 
+                if (!packages.Any())
+                    throw new Exception("No package data found");
+
                 int i = 1;
                 foreach (var item in packages)
                 {
                     CodeCoverage packageCoverage = new()
                     {
-                        Name = string.IsNullOrWhiteSpace(item.Attribute("name").Value) ? $"Package {i}" : item.Attribute("name").Value,
+                        Name = string.IsNullOrWhiteSpace(item.Attribute("name")?.Value) ? $"{Path.GetFileNameWithoutExtension(filename)} Package {i}" : item.Attribute("name").Value,
                         LineRate = double.Parse(item.Attribute("line-rate")?.Value ?? "0"),
-                        BranchRate = double.Parse(item.Attribute("branch-rate")?.Value ?? "0"),
+                        BranchRate = double.TryParse(item.Attribute("branch-rate")?.Value ?? "0", out double bRate) ? bRate : 0,
-                        Complexity = double.Parse(item.Attribute("complexity")?.Value ?? "0")
+                        Complexity = double.TryParse(item.Attribute("complexity")?.Value ?? "0", out double complex) ? complex : 0
                     };
                     summary.Packages.Add(packageCoverage);
                     summary.Complexity += packageCoverage.Complexity;
@@ -178,7 +229,7 @@ namespace CodeCoverageSummary
             }
             catch (Exception ex)
             {
-                Console.WriteLine($"Parse Error: {ex.Message}");
+                Console.WriteLine($"Parsing Error: {ex.Message} - {filename}");
                 return null;
             }
         }
@@ -199,10 +250,10 @@ namespace CodeCoverageSummary
             }
             else
             {
-                if (!int.TryParse(thresholds.Substring(0, s), out lowerPercentage))
+                if (!int.TryParse(thresholds.AsSpan(0, s), out lowerPercentage))
                     throw new ArgumentException("Threshold parameter set incorrectly.");
 
-                if (!int.TryParse(thresholds.Substring(s + 1), out upperPercentage))
+                if (!int.TryParse(thresholds.AsSpan(s + 1), out upperPercentage))
                     throw new ArgumentException("Threshold parameter set incorrectly.");
             }
             lowerThreshold = lowerPercentage / 100.0;

src/CodeCoverageSummary/Properties/launchSettings.json

@@ -2,7 +2,7 @@
   "profiles": {
     "CodeCoverageSummary": {
       "commandName": "Project",
-      "commandLineArgs": "--files ../../../../coverage.cobertura.xml,../../../../coverage.cobertura.xml --format=text --badge true --thresholds=\"85 90\" --fail true"
+      "commandLineArgs": "--files **/coverage.*.xml --format=text --badge true --thresholds=\"85 90\" --fail true"
     },
     "Docker": {
       "commandName": "Docker",

src/coverage.MATLAB.xml

@@ -0,0 +1,476 @@
+<?xml version="1.0" encoding="UTF-8" standalone="no" ?>
+<coverage branch-rate="NaN" branches-covered="NaN" branches-valid="NaN" complexity="NaN" line-rate="0.3705" lines-covered="103" lines-valid="278" timestamp="1656158672.7938" version="">
+  <sources>
+    <source>/home/runner/work/climatedatastore/climatedatastore/climatedatastoreToolbox/</source>
+  </sources>
+  <packages>
+    <package branch-rate="NaN" complexity="NaN" line-rate="0.3705" name="">
+      <classes>
+        <class branch-rate="NaN" complexity="NaN" filename="climateDataStoreDownload.m" line-rate="0.91667" name="climateDataStoreDownload">
+          <methods/>
+          <lines>
+            <line hits="5" number="42"/>
+            <line hits="5" number="44"/>
+            <line hits="5" number="45"/>
+            <line hits="5" number="46"/>
+            <line hits="0" number="47"/>
+            <line hits="5" number="50"/>
+            <line hits="5" number="51"/>
+            <line hits="5" number="52"/>
+            <line hits="3" number="53"/>
+            <line hits="2" number="55"/>
+            <line hits="2" number="57"/>
+            <line hits="2" number="58"/>
+          </lines>
+        </class>
+        <class branch-rate="NaN" complexity="NaN" filename="climateDataStoreDownloadAsync.m" line-rate="0.8" name="climateDataStoreDownloadAsync">
+          <methods/>
+          <lines>
+            <line hits="5" number="54"/>
+            <line hits="4" number="56"/>
+            <line hits="5" number="57"/>
+            <line hits="0" number="58"/>
+            <line hits="5" number="61"/>
+          </lines>
+        </class>
+        <class branch-rate="NaN" complexity="NaN" filename="climateDataStoreDownloadFuture.m" line-rate="0.86275" name="climateDataStoreDownloadFuture">
+          <methods>
+            <method branch-rate="NaN" line-rate="0.90909" name="climateDataStoreDownloadFuture" signature="obj = climateDataStoreDownloadFuture(datasetName, datasetOptions, options)">
+              <lines>
+                <line hits="10" number="83"/>
+                <line hits="10" number="84"/>
+                <line hits="10" number="85"/>
+                <line hits="10" number="86"/>
+                <line hits="10" number="89"/>
+                <line hits="10" number="99"/>
+                <line hits="10" number="101"/>
+                <line hits="10" number="102"/>
+                <line hits="7" number="103"/>
+                <line hits="3" number="104"/>
+                <line hits="3" number="105"/>
+                <line hits="3" number="106"/>
+                <line hits="3" number="107"/>
+                <line hits="2" number="108"/>
+                <line hits="1" number="109"/>
+                <line hits="1" number="110"/>
+                <line hits="0" number="112"/>
+                <line hits="0" number="115"/>
+                <line hits="3" number="117"/>
+                <line hits="3" number="118"/>
+                <line hits="7" number="122"/>
+                <line hits="7" number="123"/>
+              </lines>
+            </method>
+            <method branch-rate="NaN" line-rate="0.875" name="cancel" signature="cancel(obj)">
+              <lines>
+                <line hits="1" number="131"/>
+                <line hits="0" number="132"/>
+                <line hits="1" number="135"/>
+                <line hits="1" number="136"/>
+                <line hits="1" number="138"/>
+                <line hits="1" number="139"/>
+                <line hits="1" number="140"/>
+                <line hits="1" number="141"/>
+              </lines>
+            </method>
+            <method branch-rate="NaN" line-rate="0.9" name="wait" signature="wait(obj, timeout)">
+              <lines>
+                <line hits="8" number="158"/>
+                <line hits="2" number="159"/>
+                <line hits="6" number="162"/>
+                <line hits="6" number="163"/>
+                <line hits="6" number="164"/>
+                <line hits="2" number="165"/>
+                <line hits="2" number="166"/>
+                <line hits="2" number="167"/>
+                <line hits="6" number="169"/>
+                <line hits="0" number="170"/>
+              </lines>
+            </method>
+            <method branch-rate="NaN" line-rate="1" name="get.OutputArguments" signature="result = get.OutputArguments(obj)">
+              <lines>
+                <line hits="16" number="178"/>
+                <line hits="16" number="179"/>
+              </lines>
+            </method>
+            <method branch-rate="NaN" line-rate="1" name="get.RunningDuration" signature="result = get.RunningDuration(obj)">
+              <lines>
+                <line hits="3" number="183"/>
+                <line hits="3" number="184"/>
+                <line hits="2" number="186"/>
+                <line hits="1" number="189"/>
+              </lines>
+            </method>
+            <method branch-rate="NaN" line-rate="1" name="get.State" signature="result = get.State(obj)">
+              <lines>
+                <line hits="15" number="194"/>
+                <line hits="15" number="195"/>
+              </lines>
+            </method>
+            <method branch-rate="NaN" line-rate="0.5625" name="update" signature="update(obj)">
+              <lines>
+                <line hits="50" number="210"/>
+                <line hits="5" number="211"/>
+                <line hits="45" number="214"/>
+                <line hits="45" number="215"/>
+                <line hits="0" number="216"/>
+                <line hits="0" number="217"/>
+                <line hits="0" number="218"/>
+                <line hits="0" number="219"/>
+                <line hits="0" number="221"/>
+                <line hits="0" number="223"/>
+                <line hits="0" number="224"/>
+                <line hits="45" number="228"/>
+                <line hits="45" number="229"/>
+                <line hits="34" number="230"/>
+                <line hits="11" number="231"/>
+                <line hits="3" number="232"/>
+              </lines>
+            </method>
+            <method branch-rate="NaN" line-rate="1" name="getResultsIfAvailable" signature="getResultsIfAvailable(obj)">
+              <lines>
+                <line hits="34" number="237"/>
+                <line hits="29" number="238"/>
+                <line hits="5" number="243"/>
+                <line hits="5" number="244"/>
+                <line hits="5" number="246"/>
+                <line hits="5" number="247"/>
+                <line hits="5" number="248"/>
+                <line hits="5" number="249"/>
+                <line hits="5" number="252"/>
+                <line hits="5" number="255"/>
+                <line hits="5" number="259"/>
+                <line hits="3" number="261"/>
+                <line hits="3" number="262"/>
+                <line hits="3" number="265"/>
+                <line hits="2" number="268"/>
+                <line hits="2" number="269"/>
+                <line hits="2" number="270"/>
+                <line hits="5" number="272"/>
+                <line hits="5" number="273"/>
+              </lines>
+            </method>
+            <method branch-rate="NaN" line-rate="0.72727" name="getErrorInfo" signature="getErrorInfo(obj)">
+              <lines>
+                <line hits="3" number="278"/>
+                <line hits="2" number="279"/>
+                <line hits="1" number="282"/>
+                <line hits="1" number="293"/>
+                <line hits="1" number="294"/>
+                <line hits="0" number="295"/>
+                <line hits="0" number="296"/>
+                <line hits="0" number="298"/>
+                <line hits="1" number="300"/>
+                <line hits="1" number="301"/>
+                <line hits="1" number="302"/>
+              </lines>
+            </method>
+            <method branch-rate="NaN" line-rate="1" name="makeStringsChars" signature="theStruct = makeStringsChars(theStruct)">
+              <lines>
+                <line hits="10" number="308"/>
+                <line hits="10" number="309"/>
+                <line hits="70" number="310"/>
+                <line hits="70" number="311"/>
+                <line hits="60" number="313"/>
+                <line hits="10" number="316"/>
+              </lines>
+            </method>
+          </methods>
+          <lines>
+            <line hits="10" number="73"/>
+            <line hits="10" number="83"/>
+            <line hits="10" number="84"/>
+            <line hits="10" number="85"/>
+            <line hits="10" number="86"/>
+            <line hits="10" number="89"/>
+            <line hits="10" number="99"/>
+            <line hits="10" number="101"/>
+            <line hits="10" number="102"/>
+            <line hits="7" number="103"/>
+            <line hits="3" number="104"/>
+            <line hits="3" number="105"/>
+            <line hits="3" number="106"/>
+            <line hits="3" number="107"/>
+            <line hits="2" number="108"/>
+            <line hits="1" number="109"/>
+            <line hits="1" number="110"/>
+            <line hits="0" number="112"/>
+            <line hits="0" number="115"/>
+            <line hits="3" number="117"/>
+            <line hits="3" number="118"/>
+            <line hits="7" number="122"/>
+            <line hits="7" number="123"/>
+            <line hits="1" number="131"/>
+            <line hits="0" number="132"/>
+            <line hits="1" number="135"/>
+            <line hits="1" number="136"/>
+            <line hits="1" number="138"/>
+            <line hits="1" number="139"/>
+            <line hits="1" number="140"/>
+            <line hits="1" number="141"/>
+            <line hits="8" number="155"/>
+            <line hits="8" number="158"/>
+            <line hits="2" number="159"/>
+            <line hits="6" number="162"/>
+            <line hits="6" number="163"/>
+            <line hits="6" number="164"/>
+            <line hits="2" number="165"/>
+            <line hits="2" number="166"/>
+            <line hits="2" number="167"/>
+            <line hits="6" number="169"/>
+            <line hits="0" number="170"/>
+            <line hits="16" number="178"/>
+            <line hits="16" number="179"/>
+            <line hits="3" number="183"/>
+            <line hits="3" number="184"/>
+            <line hits="2" number="186"/>
+            <line hits="1" number="189"/>
+            <line hits="15" number="194"/>
+            <line hits="15" number="195"/>
+            <line hits="50" number="210"/>
+            <line hits="5" number="211"/>
+            <line hits="45" number="214"/>
+            <line hits="45" number="215"/>
+            <line hits="0" number="216"/>
+            <line hits="0" number="217"/>
+            <line hits="0" number="218"/>
+            <line hits="0" number="219"/>
+            <line hits="0" number="221"/>
+            <line hits="0" number="223"/>
+            <line hits="0" number="224"/>
+            <line hits="45" number="228"/>
+            <line hits="45" number="229"/>
+            <line hits="34" number="230"/>
+            <line hits="11" number="231"/>
+            <line hits="3" number="232"/>
+            <line hits="34" number="237"/>
+            <line hits="29" number="238"/>
+            <line hits="5" number="243"/>
+            <line hits="5" number="244"/>
+            <line hits="5" number="246"/>
+            <line hits="5" number="247"/>
+            <line hits="5" number="248"/>
+            <line hits="5" number="249"/>
+            <line hits="5" number="252"/>
+            <line hits="5" number="255"/>
+            <line hits="5" number="259"/>
+            <line hits="3" number="261"/>
+            <line hits="3" number="262"/>
+            <line hits="3" number="265"/>
+            <line hits="2" number="268"/>
+            <line hits="2" number="269"/>
+            <line hits="2" number="270"/>
+            <line hits="5" number="272"/>
+            <line hits="5" number="273"/>
+            <line hits="3" number="278"/>
+            <line hits="2" number="279"/>
+            <line hits="1" number="282"/>
+            <line hits="1" number="293"/>
+            <line hits="1" number="294"/>
+            <line hits="0" number="295"/>
+            <line hits="0" number="296"/>
+            <line hits="0" number="298"/>
+            <line hits="1" number="300"/>
+            <line hits="1" number="301"/>
+            <line hits="1" number="302"/>
+            <line hits="10" number="308"/>
+            <line hits="10" number="309"/>
+            <line hits="70" number="310"/>
+            <line hits="70" number="311"/>
+            <line hits="60" number="313"/>
+            <line hits="10" number="316"/>
+          </lines>
+        </class>
+        <class branch-rate="NaN" complexity="NaN" filename="doc/readSatelliteSeaIceThickness.m" line-rate="0" name="readSatelliteSeaIceThickness">
+          <methods/>
+          <lines>
+            <line hits="0" number="4"/>
+            <line hits="0" number="7"/>
+            <line hits="0" number="8"/>
+            <line hits="0" number="9"/>
+            <line hits="0" number="10"/>
+            <line hits="0" number="11"/>
+            <line hits="0" number="12"/>
+            <line hits="0" number="13"/>
+            <line hits="0" number="14"/>
+            <line hits="0" number="15"/>
+            <line hits="0" number="16"/>
+            <line hits="0" number="17"/>
+            <line hits="0" number="18"/>
+            <line hits="0" number="19"/>
+          </lines>
+        </class>
+        <class branch-rate="NaN" complexity="NaN" filename="doc/ComparingIceThickness.mlx" line-rate="0" name="ComparingIceThickness">
+          <methods/>
+          <lines>
+            <line hits="0" number="1"/>
+            <line hits="0" number="2"/>
+            <line hits="0" number="3"/>
+            <line hits="0" number="4"/>
+            <line hits="0" number="5"/>
+            <line hits="0" number="6"/>
+            <line hits="0" number="7"/>
+            <line hits="0" number="8"/>
+            <line hits="0" number="9"/>
+            <line hits="0" number="10"/>
+            <line hits="0" number="11"/>
+            <line hits="0" number="12"/>
+            <line hits="0" number="13"/>
+            <line hits="0" number="14"/>
+            <line hits="0" number="15"/>
+            <line hits="0" number="16"/>
+            <line hits="0" number="17"/>
+            <line hits="0" number="18"/>
+            <line hits="0" number="19"/>
+            <line hits="0" number="21"/>
+            <line hits="0" number="22"/>
+            <line hits="0" number="23"/>
+            <line hits="0" number="24"/>
+            <line hits="0" number="25"/>
+            <line hits="0" number="26"/>
+            <line hits="0" number="27"/>
+            <line hits="0" number="28"/>
+            <line hits="0" number="29"/>
+            <line hits="0" number="30"/>
+          </lines>
+        </class>
+        <class branch-rate="NaN" complexity="NaN" filename="doc/GettingStarted.mlx" line-rate="0" name="GettingStarted">
+          <methods/>
+          <lines>
+            <line hits="0" number="1"/>
+            <line hits="0" number="2"/>
+            <line hits="0" number="3"/>
+            <line hits="0" number="4"/>
+            <line hits="0" number="5"/>
+            <line hits="0" number="6"/>
+            <line hits="0" number="7"/>
+            <line hits="0" number="8"/>
+            <line hits="0" number="9"/>
+            <line hits="0" number="10"/>
+            <line hits="0" number="11"/>
+            <line hits="0" number="12"/>
+            <line hits="0" number="13"/>
+            <line hits="0" number="14"/>
+            <line hits="0" number="15"/>
+            <line hits="0" number="16"/>
+            <line hits="0" number="17"/>
+            <line hits="0" number="19"/>
+            <line hits="0" number="20"/>
+            <line hits="0" number="21"/>
+            <line hits="0" number="22"/>
+            <line hits="0" number="23"/>
+            <line hits="0" number="24"/>
+            <line hits="0" number="25"/>
+            <line hits="0" number="26"/>
+            <line hits="0" number="27"/>
+            <line hits="0" number="28"/>
+          </lines>
+        </class>
+        <class branch-rate="NaN" complexity="NaN" filename="util/getUserDirectory.m" line-rate="0" name="getUserDirectory">
+          <methods/>
+          <lines>
+            <line hits="0" number="5"/>
+            <line hits="0" number="6"/>
+            <line hits="0" number="7"/>
+            <line hits="0" number="9"/>
+          </lines>
+        </class>
+        <class branch-rate="NaN" complexity="NaN" filename="util/setupCDSAPIIfNeeded.m" line-rate="0" name="setupCDSAPIIfNeeded">
+          <methods/>
+          <lines>
+            <line hits="0" number="8"/>
+            <line hits="0" number="10"/>
+            <line hits="0" number="12"/>
+            <line hits="0" number="14"/>
+            <line hits="0" number="18"/>
+            <line hits="0" number="19"/>
+            <line hits="0" number="21"/>
+            <line hits="0" number="22"/>
+            <line hits="0" number="23"/>
+            <line hits="0" number="28"/>
+            <line hits="0" number="30"/>
+            <line hits="0" number="32"/>
+            <line hits="0" number="33"/>
+            <line hits="0" number="35"/>
+            <line hits="0" number="36"/>
+            <line hits="0" number="37"/>
+            <line hits="0" number="38"/>
+            <line hits="0" number="39"/>
+            <line hits="0" number="42"/>
+            <line hits="0" number="43"/>
+            <line hits="0" number="46"/>
+            <line hits="0" number="47"/>
+            <line hits="0" number="50"/>
+            <line hits="0" number="51"/>
+            <line hits="0" number="52"/>
+            <line hits="0" number="53"/>
+            <line hits="0" number="55"/>
+            <line hits="0" number="56"/>
+          </lines>
+        </class>
+        <class branch-rate="NaN" complexity="NaN" filename="util/setupPythonIfNeeded.m" line-rate="0" name="setupPythonIfNeeded">
+          <methods/>
+          <lines>
+            <line hits="0" number="8"/>
+            <line hits="0" number="9"/>
+            <line hits="0" number="13"/>
+            <line hits="0" number="15"/>
+            <line hits="0" number="17"/>
+            <line hits="0" number="22"/>
+            <line hits="0" number="25"/>
+            <line hits="0" number="27"/>
+            <line hits="0" number="29"/>
+            <line hits="0" number="31"/>
+            <line hits="0" number="34"/>
+            <line hits="0" number="35"/>
+            <line hits="0" number="36"/>
+            <line hits="0" number="37"/>
+            <line hits="0" number="38"/>
+            <line hits="0" number="39"/>
+            <line hits="0" number="41"/>
+            <line hits="0" number="42"/>
+            <line hits="0" number="43"/>
+            <line hits="0" number="44"/>
+            <line hits="0" number="45"/>
+            <line hits="0" number="51"/>
+            <line hits="0" number="53"/>
+            <line hits="0" number="55"/>
+            <line hits="0" number="63"/>
+            <line hits="0" number="64"/>
+            <line hits="0" number="65"/>
+            <line hits="0" number="66"/>
+            <line hits="0" number="67"/>
+            <line hits="0" number="68"/>
+            <line hits="0" number="69"/>
+            <line hits="0" number="70"/>
+            <line hits="0" number="71"/>
+            <line hits="0" number="72"/>
+            <line hits="0" number="73"/>
+            <line hits="0" number="74"/>
+            <line hits="0" number="75"/>
+            <line hits="0" number="76"/>
+            <line hits="0" number="77"/>
+            <line hits="0" number="80"/>
+            <line hits="0" number="85"/>
+            <line hits="0" number="86"/>
+            <line hits="0" number="87"/>
+            <line hits="0" number="89"/>
+            <line hits="0" number="90"/>
+            <line hits="0" number="95"/>
+            <line hits="0" number="96"/>
+            <line hits="0" number="97"/>
+            <line hits="0" number="99"/>
+            <line hits="0" number="100"/>
+            <line hits="0" number="106"/>
+            <line hits="0" number="107"/>
+            <line hits="0" number="108"/>
+            <line hits="0" number="109"/>
+            <line hits="0" number="112"/>
+            <line hits="0" number="113"/>
+            <line hits="0" number="114"/>
+          </lines>
+        </class>
+      </classes>
+    </package>
+  </packages>
+</coverage>

src/coverage.simplecov.xml

@@ -0,0 +1,441 @@
+<?xml version='1.0'?> 
+<!DOCTYPE coverage SYSTEM "http://cobertura.sourceforge.net/xml/coverage-04.dtd">
+<!-- Generated by simplecov-cobertura version 2.1.0 (https://github.com/dashingrocket/simplecov-cobertura) -->
+<coverage line-rate="0.97" lines-covered="350" lines-valid="362" complexity="0" version="0" timestamp="1645060052">
+  <sources>
+    <source>/Users/pboling/src/my/oauth2</source>
+  </sources>
+  <packages>
+    <package name="oauth2" line-rate="0.97" complexity="0">
+      <classes>
+        <class name="oauth2" filename="lib/oauth2.rb" line-rate="1.0" complexity="0">
+          <methods/>
+          <lines>
+            <line number="1" hits="1"/>
+            <line number="2" hits="1"/>
+            <line number="3" hits="1"/>
+            <line number="4" hits="1"/>
+            <line number="5" hits="1"/>
+            <line number="6" hits="1"/>
+            <line number="7" hits="1"/>
+            <line number="8" hits="1"/>
+            <line number="9" hits="1"/>
+            <line number="10" hits="1"/>
+            <line number="11" hits="1"/>
+            <line number="12" hits="1"/>
+          </lines>
+        </class>
+        <class name="access_token" filename="lib/oauth2/access_token.rb" line-rate="0.96" complexity="0">
+          <methods/>
+          <lines>
+            <line number="1" hits="1"/>
+            <line number="2" hits="1"/>
+            <line number="3" hits="1"/>
+            <line number="4" hits="1"/>
+            <line number="7" hits="1"/>
+            <line number="13" hits="1"/>
+            <line number="14" hits="7"/>
+            <line number="15" hits="7"/>
+            <line number="23" hits="1"/>
+            <line number="24" hits="1"/>
+            <line number="41" hits="1"/>
+            <line number="42" hits="151"/>
+            <line number="43" hits="151"/>
+            <line number="44" hits="151"/>
+            <line number="45" hits="151"/>
+            <line number="46" hits="453"/>
+            <line number="48" hits="151"/>
+            <line number="49" hits="151"/>
+            <line number="50" hits="151"/>
+            <line number="51" hits="151"/>
+            <line number="52" hits="151"/>
+            <line number="55" hits="151"/>
+            <line number="61" hits="1"/>
+            <line number="62" hits="6"/>
+            <line number="68" hits="1"/>
+            <line number="69" hits="11"/>
+            <line number="75" hits="1"/>
+            <line number="76" hits="3"/>
+            <line number="83" hits="1"/>
+            <line number="84" hits="2"/>
+            <line number="86" hits="2"/>
+            <line number="87" hits="2"/>
+            <line number="88" hits="2"/>
+            <line number="89" hits="2"/>
+            <line number="90" hits="2"/>
+            <line number="91" hits="2"/>
+            <line number="97" hits="1"/>
+            <line number="98" hits="1"/>
+            <line number="107" hits="1"/>
+            <line number="108" hits="20"/>
+            <line number="109" hits="20"/>
+            <line number="115" hits="1"/>
+            <line number="116" hits="2"/>
+            <line number="122" hits="1"/>
+            <line number="123" hits="18"/>
+            <line number="129" hits="1"/>
+            <line number="130" hits="2"/>
+            <line number="136" hits="1"/>
+            <line number="137" hits="0"/>
+            <line number="143" hits="1"/>
+            <line number="144" hits="2"/>
+            <line number="148" hits="1"/>
+            <line number="149" hits="8"/>
+            <line number="152" hits="1"/>
+            <line number="154" hits="1"/>
+            <line number="155" hits="20"/>
+            <line number="157" hits="8"/>
+            <line number="158" hits="8"/>
+            <line number="160" hits="8"/>
+            <line number="161" hits="8"/>
+            <line number="163" hits="4"/>
+            <line number="164" hits="4"/>
+            <line number="165" hits="4"/>
+            <line number="167" hits="0"/>
+            <line number="171" hits="0"/>
+            <line number="175" hits="1"/>
+            <line number="176" hits="13"/>
+            <line number="178" hits="12"/>
+          </lines>
+        </class>
+        <class name="authenticator" filename="lib/oauth2/authenticator.rb" line-rate="1.0" complexity="0">
+          <methods/>
+          <lines>
+            <line number="1" hits="1"/>
+            <line number="3" hits="1"/>
+            <line number="4" hits="1"/>
+            <line number="5" hits="1"/>
+            <line number="7" hits="1"/>
+            <line number="8" hits="106"/>
+            <line number="9" hits="106"/>
+            <line number="10" hits="106"/>
+            <line number="22" hits="1"/>
+            <line number="23" hits="106"/>
+            <line number="25" hits="14"/>
+            <line number="27" hits="89"/>
+            <line number="29" hits="1"/>
+            <line number="31" hits="1"/>
+            <line number="33" hits="1"/>
+            <line number="37" hits="1"/>
+            <line number="38" hits="15"/>
+            <line number="41" hits="1"/>
+            <line number="45" hits="1"/>
+            <line number="46" hits="89"/>
+            <line number="51" hits="1"/>
+            <line number="52" hits="1"/>
+            <line number="57" hits="1"/>
+            <line number="58" hits="14"/>
+            <line number="59" hits="14"/>
+            <line number="60" hits="14"/>
+            <line number="64" hits="1"/>
+            <line number="65" hits="14"/>
+          </lines>
+        </class>
+        <class name="client" filename="lib/oauth2/client.rb" line-rate="0.95" complexity="0">
+          <methods/>
+          <lines>
+            <line number="1" hits="1"/>
+            <line number="2" hits="1"/>
+            <line number="4" hits="1"/>
+            <line number="6" hits="1"/>
+            <line number="7" hits="1"/>
+            <line number="9" hits="1"/>
+            <line number="10" hits="1"/>
+            <line number="11" hits="1"/>
+            <line number="31" hits="1"/>
+            <line number="32" hits="204"/>
+            <line number="33" hits="204"/>
+            <line number="34" hits="204"/>
+            <line number="35" hits="204"/>
+            <line number="36" hits="204"/>
+            <line number="39" hits="204"/>
+            <line number="49" hits="204"/>
+            <line number="55" hits="1"/>
+            <line number="56" hits="0"/>
+            <line number="57" hits="0"/>
+            <line number="61" hits="1"/>
+            <line number="62" hits="423"/>
+            <line number="63" hits="158"/>
+            <line number="64" hits="158"/>
+            <line number="65" hits="134"/>
+            <line number="66" hits="134"/>
+            <line number="69" hits="158"/>
+            <line number="76" hits="1"/>
+            <line number="77" hits="11"/>
+            <line number="78" hits="11"/>
+            <line number="84" hits="1"/>
+            <line number="85" hits="100"/>
+            <line number="100" hits="1"/>
+            <line number="101" hits="141"/>
+            <line number="103" hits="141"/>
+            <line number="105" hits="141"/>
+            <line number="106" hits="141"/>
+            <line number="107" hits="141"/>
+            <line number="109" hits="141"/>
+            <line number="111" hits="141"/>
+            <line number="113" hits="3"/>
+            <line number="114" hits="3"/>
+            <line number="115" hits="3"/>
+            <line number="117" hits="2"/>
+            <line number="118" hits="1"/>
+            <line number="119" hits="1"/>
+            <line number="121" hits="2"/>
+            <line number="124" hits="127"/>
+            <line number="126" hits="11"/>
+            <line number="127" hits="11"/>
+            <line number="129" hits="2"/>
+            <line number="130" hits="2"/>
+            <line number="132" hits="0"/>
+            <line number="133" hits="0"/>
+            <line number="143" hits="1"/>
+            <line number="144" hits="97"/>
+            <line number="145" hits="182"/>
+            <line number="146" hits="2"/>
+            <line number="148" hits="180"/>
+            <line number="151" hits="97"/>
+            <line number="153" hits="97"/>
+            <line number="154" hits="97"/>
+            <line number="155" hits="97"/>
+            <line number="156" hits="97"/>
+            <line number="157" hits="79"/>
+            <line number="158" hits="79"/>
+            <line number="160" hits="18"/>
+            <line number="161" hits="18"/>
+            <line number="163" hits="97"/>
+            <line number="164" hits="97"/>
+            <line number="167" hits="97"/>
+            <line number="169" hits="0"/>
+            <line number="172" hits="97"/>
+            <line number="173" hits="3"/>
+            <line number="174" hits="3"/>
+            <line number="176" hits="94"/>
+            <line number="182" hits="1"/>
+            <line number="183" hits="48"/>
+            <line number="189" hits="1"/>
+            <line number="190" hits="5"/>
+            <line number="196" hits="1"/>
+            <line number="197" hits="11"/>
+            <line number="203" hits="1"/>
+            <line number="204" hits="21"/>
+            <line number="207" hits="1"/>
+            <line number="208" hits="9"/>
+            <line number="227" hits="1"/>
+            <line number="228" hits="55"/>
+            <line number="229" hits="2"/>
+            <line number="231" hits="53"/>
+            <line number="235" hits="1"/>
+            <line number="236" hits="91"/>
+            <line number="237" hits="91"/>
+            <line number="240" hits="1"/>
+            <line number="242" hits="1"/>
+            <line number="243" hits="97"/>
+            <line number="244" hits="97"/>
+            <line number="246" hits="93"/>
+            <line number="250" hits="93"/>
+            <line number="251" hits="1"/>
+            <line number="253" hits="92"/>
+          </lines>
+        </class>
+        <class name="error" filename="lib/oauth2/error.rb" line-rate="1.0" complexity="0">
+          <methods/>
+          <lines>
+            <line number="1" hits="1"/>
+            <line number="2" hits="1"/>
+            <line number="3" hits="1"/>
+            <line number="7" hits="1"/>
+            <line number="8" hits="14"/>
+            <line number="9" hits="14"/>
+            <line number="11" hits="14"/>
+            <line number="12" hits="7"/>
+            <line number="13" hits="7"/>
+            <line number="14" hits="7"/>
+            <line number="17" hits="14"/>
+            <line number="23" hits="1"/>
+            <line number="24" hits="14"/>
+            <line number="26" hits="14"/>
+            <line number="28" hits="14"/>
+            <line number="29" hits="7"/>
+            <line number="30" hits="7"/>
+            <line number="32" hits="7"/>
+            <line number="35" hits="14"/>
+            <line number="37" hits="14"/>
+          </lines>
+        </class>
+        <class name="mac_token" filename="lib/oauth2/mac_token.rb" line-rate="0.95" complexity="0">
+          <methods/>
+          <lines>
+            <line number="1" hits="1"/>
+            <line number="2" hits="1"/>
+            <line number="3" hits="1"/>
+            <line number="4" hits="1"/>
+            <line number="6" hits="1"/>
+            <line number="7" hits="1"/>
+            <line number="14" hits="1"/>
+            <line number="15" hits="3"/>
+            <line number="18" hits="1"/>
+            <line number="30" hits="1"/>
+            <line number="31" hits="19"/>
+            <line number="32" hits="19"/>
+            <line number="34" hits="18"/>
+            <line number="43" hits="1"/>
+            <line number="44" hits="4"/>
+            <line number="46" hits="4"/>
+            <line number="47" hits="4"/>
+            <line number="49" hits="4"/>
+            <line number="53" hits="1"/>
+            <line number="54" hits="1"/>
+            <line number="61" hits="1"/>
+            <line number="62" hits="9"/>
+            <line number="63" hits="9"/>
+            <line number="65" hits="9"/>
+            <line number="67" hits="8"/>
+            <line number="69" hits="7"/>
+            <line number="71" hits="7"/>
+            <line number="80" hits="1"/>
+            <line number="82" hits="8"/>
+            <line number="91" hits="8"/>
+            <line number="97" hits="1"/>
+            <line number="98" hits="19"/>
+            <line number="101" hits="1"/>
+            <line number="103" hits="0"/>
+            <line number="107" hits="17"/>
+            <line number="109" hits="0"/>
+            <line number="112" hits="1"/>
+            <line number="116" hits="1"/>
+            <line number="120" hits="1"/>
+            <line number="124" hits="1"/>
+            <line number="125" hits="8"/>
+          </lines>
+        </class>
+        <class name="response" filename="lib/oauth2/response.rb" line-rate="1.0" complexity="0">
+          <methods/>
+          <lines>
+            <line number="1" hits="1"/>
+            <line number="2" hits="1"/>
+            <line number="3" hits="1"/>
+            <line number="5" hits="1"/>
+            <line number="7" hits="1"/>
+            <line number="8" hits="1"/>
+            <line number="9" hits="1"/>
+            <line number="14" hits="62"/>
+            <line number="15" hits="44"/>
+            <line number="16" hits="3"/>
+            <line number="20" hits="1"/>
+            <line number="32" hits="1"/>
+            <line number="33" hits="3"/>
+            <line number="34" hits="3"/>
+            <line number="35" hits="3"/>
+            <line number="36" hits="6"/>
+            <line number="46" hits="1"/>
+            <line number="47" hits="147"/>
+            <line number="48" hits="147"/>
+            <line number="52" hits="1"/>
+            <line number="53" hits="6"/>
+            <line number="57" hits="1"/>
+            <line number="58" hits="150"/>
+            <line number="62" hits="1"/>
+            <line number="63" hits="154"/>
+            <line number="69" hits="1"/>
+            <line number="70" hits="134"/>
+            <line number="72" hits="131"/>
+            <line number="76" hits="1"/>
+            <line number="77" hits="244"/>
+            <line number="81" hits="1"/>
+            <line number="82" hits="244"/>
+            <line number="84" hits="244"/>
+            <line number="89" hits="1"/>
+            <line number="90" hits="1"/>
+          </lines>
+        </class>
+        <class name="assertion" filename="lib/oauth2/strategy/assertion.rb" line-rate="0.89" complexity="0">
+          <methods/>
+          <lines>
+            <line number="1" hits="1"/>
+            <line number="3" hits="1"/>
+            <line number="4" hits="1"/>
+            <line number="23" hits="1"/>
+            <line number="27" hits="1"/>
+            <line number="28" hits="1"/>
+            <line number="45" hits="1"/>
+            <line number="46" hits="8"/>
+            <line number="47" hits="8"/>
+            <line number="50" hits="1"/>
+            <line number="51" hits="8"/>
+            <line number="53" hits="8"/>
+            <line number="60" hits="1"/>
+            <line number="62" hits="8"/>
+            <line number="67" hits="8"/>
+            <line number="68" hits="8"/>
+            <line number="69" hits="0"/>
+            <line number="70" hits="0"/>
+          </lines>
+        </class>
+        <class name="auth_code" filename="lib/oauth2/strategy/auth_code.rb" line-rate="1.0" complexity="0">
+          <methods/>
+          <lines>
+            <line number="1" hits="1"/>
+            <line number="2" hits="1"/>
+            <line number="6" hits="1"/>
+            <line number="10" hits="1"/>
+            <line number="11" hits="3"/>
+            <line number="17" hits="1"/>
+            <line number="18" hits="3"/>
+            <line number="27" hits="1"/>
+            <line number="28" hits="44"/>
+            <line number="30" hits="44"/>
+          </lines>
+        </class>
+        <class name="base" filename="lib/oauth2/strategy/base.rb" line-rate="1.0" complexity="0">
+          <methods/>
+          <lines>
+            <line number="1" hits="1"/>
+            <line number="2" hits="1"/>
+            <line number="3" hits="1"/>
+            <line number="4" hits="1"/>
+            <line number="5" hits="95"/>
+          </lines>
+        </class>
+        <class name="client_credentials" filename="lib/oauth2/strategy/client_credentials.rb" line-rate="1.0" complexity="0">
+          <methods/>
+          <lines>
+            <line number="1" hits="1"/>
+            <line number="2" hits="1"/>
+            <line number="6" hits="1"/>
+            <line number="10" hits="1"/>
+            <line number="11" hits="1"/>
+            <line number="18" hits="1"/>
+            <line number="19" hits="20"/>
+            <line number="20" hits="20"/>
+          </lines>
+        </class>
+        <class name="implicit" filename="lib/oauth2/strategy/implicit.rb" line-rate="1.0" complexity="0">
+          <methods/>
+          <lines>
+            <line number="1" hits="1"/>
+            <line number="2" hits="1"/>
+            <line number="6" hits="1"/>
+            <line number="10" hits="1"/>
+            <line number="11" hits="3"/>
+            <line number="17" hits="1"/>
+            <line number="18" hits="3"/>
+            <line number="24" hits="1"/>
+            <line number="25" hits="1"/>
+          </lines>
+        </class>
+        <class name="password" filename="lib/oauth2/strategy/password.rb" line-rate="1.0" complexity="0">
+          <methods/>
+          <lines>
+            <line number="1" hits="1"/>
+            <line number="2" hits="1"/>
+            <line number="6" hits="1"/>
+            <line number="10" hits="1"/>
+            <line number="11" hits="1"/>
+            <line number="19" hits="1"/>
+            <line number="20" hits="10"/>
+            <line number="23" hits="10"/>
+          </lines>
+        </class>
+      </classes>
+    </package>
+  </packages>
+</coverage>