Skip to content

ci: publish coverage-summary artifact for eng-dashboard - #8

Merged
yamarkov merged 1 commit into
mainfrom
ci/coverage-summary-artifact
Sep 3, 2026
Merged

yamarkov merged 1 commit into
mainfrom
ci/coverage-summary-artifact

Conversation

@yamarkov

@yamarkov yamarkov commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds a tests GitHub Actions workflow (repo had none) that runs the existing pytest suite on PRs and pushes to main, and publishes a coverage-summary artifact containing {"coverage": <float pct>} so voygr-tech/eng-dashboard can harvest it for the "Test coverage %" chart. Mirrors the pattern already used in poi_core/poi_sampling/poi_enrichment.

  • Coverage target: --cov=voygr (the only package under src/, pyproject.toml [project] name = "voygr").
  • Coverage is informational only — --cov-fail-under=0 is passed explicitly (there's no fail_under config in this repo's pyproject.toml today, but the flag guards against one being added later without gating CI).
  • [project.optional-dependencies].dev has pytest + pytest-mock but not pytest-cov, so it's installed as a separate step.
  • All GitHub Actions pinned to the exact SHAs from the established pattern (actions/checkout v4, actions/setup-python v5, actions/upload-artifact v4).

Local verification

Ran locally in a fresh venv before opening this PR:

pip install -e ".[dev]" pytest-cov
pytest --cov=voygr --cov-report=json:coverage-report.json --cov-fail-under=0

Result: 77 passed, coverage 93.3%. No credentials/services required — the "integration" tests in tests/test_integration.py are fully mocked (click CliRunner + unittest.mock.patch).

Part of VOYG-57 — coverage harvested weekly by voygr-tech/eng-dashboard.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@yamarkov

yamarkov commented Sep 3, 2026

Copy link
Copy Markdown
Contributor Author

@greptileai

@greptile-apps

greptile-apps Bot commented Sep 3, 2026

Copy link
Copy Markdown

Greptile Summary

Adds the repository’s first GitHub Actions test workflow, running the existing pytest suite with informational coverage on pull requests and pushes to main.

  • Uses Python 3.12 and installs the project’s development dependencies plus pytest-cov.
  • Converts the coverage report into a compact {"coverage": <percentage>} summary.
  • Uploads the summary as a pinned coverage-summary artifact for dashboard collection.

Confidence Score: 5/5

The PR appears safe to merge, with no concrete changed-code failure identified.

The configured Python version is supported, the editable package and coverage target resolve correctly, the tests have no external prerequisites, and the generated summary is uploaded only after a successful test run.

Important Files Changed

Filename Overview
.github/workflows/tests.yml Adds a SHA-pinned test-and-coverage workflow whose runtime, package target, report generation, and artifact upload are consistent with the repository configuration.

Reviews (1): Last reviewed commit: "ci: publish coverage-summary artifact fo..." | Re-trigger Greptile

@yamarkov
yamarkov merged commit a195fc8 into main Sep 3, 2026
4 checks passed
@yamarkov
yamarkov deleted the ci/coverage-summary-artifact branch September 3, 2026 02:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant