mirror of
https://github.com/navidrome/navidrome.git
synced 2026-10-07 18:07:11 +02:00
cb045b8ef3The exclusion only worked on master. Coverage profiles name files by import
path; octocov shortens those to repo-relative paths using the checked-out
source, but coverage-on-pr.yml sparse-checks-out only .octocov.yml, so the
paths stay as github.com/navidrome/navidrome/tests/mock_*.go and 'tests/**'
never matched. '**/*_gen.go' matched either way, which is why only the 30
tests/ files leaked.
Every pull request since b77fb45 therefore reported ~-3.7% against master:
447 files on the base side, 477 on the pull request side (#6002, #6069).
44 lines
1.4 KiB
YAML
44 lines
1.4 KiB
YAML
# Code coverage reporting for pull requests. See https://github.com/k1LoW/octocov
|
|
# The 30s default is not enough: scanning this repo's artifacts for the baseline
|
|
# eats most of it, leaving none for the report upload.
|
|
timeout: 5m
|
|
coverage:
|
|
# A single pre-merged profile: octocov reports statements for one path, but
|
|
# switches to line counting when it merges several itself.
|
|
paths:
|
|
- coverage.out
|
|
# Not code under test: tests/ holds the mocks and helpers, *_gen.go is generated.
|
|
# Both patterns need the '**/' prefix: the comment workflow has no source tree,
|
|
# so octocov cannot shorten the profile's import paths to repo-relative ones.
|
|
exclude:
|
|
- '**/tests/**'
|
|
- '**/*_gen.go'
|
|
codeToTestRatio:
|
|
# Needs the pull request's own source, which the comment workflow must not
|
|
# check out: it holds a write token.
|
|
if: env.COVERAGE_COMMENT != 'true'
|
|
code:
|
|
- '**/*.go'
|
|
- '!**/*_test.go'
|
|
- '!**/*_gen.go'
|
|
test:
|
|
- '**/*_test.go'
|
|
testExecutionTime:
|
|
if: true
|
|
steps:
|
|
- Test with coverage
|
|
- Test plugins
|
|
diff:
|
|
datastores:
|
|
- artifact://${GITHUB_REPOSITORY}
|
|
comment:
|
|
# Only the 'Report coverage on PR' workflow sets this: a pull_request run from
|
|
# a fork gets a read-only token, so commenting from here 403s.
|
|
if: env.COVERAGE_COMMENT == 'true'
|
|
updatePrevious: true
|
|
summary:
|
|
if: true
|
|
report:
|
|
if: is_default_branch
|
|
datastores:
|
|
- artifact://${GITHUB_REPOSITORY}
|