Add coverage report for tests

[xingyaoww]

Add coverage report to tests.yml

[openhands-ai]

I'm on it! xingyaoww can track my progress at all-hands.dev

[openhands-ai]

Here’s a concise wrap-up of the work completed to fix issue #3 (Add coverage report to tests.yml):

Summary of changes

• Added pytest-cov to dev dependencies in pyproject.toml.
• Updated CI workflow (.github/workflows/tests.yml) to:
  • Run pytest with coverage and produce Cobertura XML (coverage.xml).
  • Generate a coverage summary and badge using irongut/CodeCoverageSummary@v1.3.0.
  • Post a sticky PR comment with the coverage summary on pull requests.
  • Upload coverage.xml and the generated markdown summary as artifacts.
  • Trigger on both push and pull_request with minimal permissions.
  • Use if: always() to ensure the summary is produced even if tests fail.

Branch, commit, and PR

• Branch: openhands/fix-issue-3-coverage
• PR: ci: add coverage report and summary to tests workflow (closes #3) #6
• The PR includes “Closes Add coverage report for tests #3” and a clear description of changes.

Checklist against your instructions

• Read issue details via GitHub API: done.
• Add coverage report to tests workflow using the referenced action: done.
• Add tests for code changes: not needed (config-only change).
• Create a new branch: done.
• Commit with clear message: done.
• Push the branch: done.
• Open a PR referencing the issue number: done; PR includes “Closes Add coverage report for tests #3”.
• Follow PR template: repository has no .github/pull_request_template.md, so a structured description was provided.

Extraneous changes

• Only necessary files were changed: pyproject.toml, tests.yml, and uv.lock (lockfile update due to dependency change). No unrelated changes were made.

If you’d like, I can also enforce a minimum coverage threshold (failing the workflow below a set percentage) using pytest’s --cov-fail-under or the action’s fail_below_min with thresholds.

_{View full conversation}