Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Track Source Files Excluded from Code Coverage Reports #9311

Merged
merged 6 commits into from
Jul 17, 2020

Conversation

kj4ezj
Copy link
Contributor

@kj4ezj kj4ezj commented Jul 14, 2020

Change Description

From AUTO-117, Blockchain wishes to collect metrics on what source files are not being reported upon by the code coverage reports. The new metrics collection code references the pipeline.jsonc file to determine which files it should expect coverage reports from, and which files it should ignore in the coverage report. This pull request creates initial versions of these declarations, and also updates some other pipeline.jsonc entries.

Schema

This script looks for a pipeline.jsonc file in the root of the repo in which it is running, filters by pipeline slug, then automagically imports two properties, if they are present:

{
    "eosio-code-coverage":
    {
        "exclude": [ // ignore coverage reports from source filenames matching these PCREs
            "^/build/",
            "^/coverage/",
            "^/.git/",
            "^/libraries/eos-vm/",
            "^/libraries/fc/",
            "^/libraries/wabt/",
            "^/libraries/wasm-jit/",
            "^/node_modules/",
            "^/unittests/"
        ],
        "expect": [ // expect coverage reports from source filenames matching these PCREs
            "[.]c(pp)?([.]in)?$",
            "[.]h(pp)?([.]in)?$"
        ]
    }
}

The script will still work if this file or either one of these properties are missing.

See Also

Change Type

Select ONE

  • Documentation
  • Stability bug fix
  • Other

CI

  • Other - special case

Consensus Changes

  • Consensus Changes

None.

API Changes

  • API Changes

None.

Documentation Additions

  • Documentation Additions

None.

@kj4ezj kj4ezj requested a review from scottarnette July 16, 2020 17:01
@kj4ezj kj4ezj marked this pull request as ready for review July 16, 2020 21:57
@kj4ezj kj4ezj merged commit b074937 into release/2.0.x Jul 17, 2020
@kj4ezj kj4ezj deleted the zach-2.0-coverage-coverage branch July 17, 2020 02:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants