Skip to content

Enable coverage reporting for test code#734

Closed
2253781369 wants to merge 2 commits into
twisted:trunkfrom
2253781369:enable-test-coverage
Closed

Enable coverage reporting for test code#734
2253781369 wants to merge 2 commits into
twisted:trunkfrom
2253781369:enable-test-coverage

Conversation

@2253781369

Copy link
Copy Markdown

Summary

Remove test files from the coverage omit list so that test code is included in coverage tracking. This helps detect tests that are always skipped — they would show 0% coverage.

Changes

  • pyproject.toml: Remove src/towncrier/test/* from coverage omit list
  • noxfile.py: Update coverage_report session to show separate production and test coverage reports
  • .github/workflows/ci.yml: CI now reports full coverage (including tests) to the step summary, while still enforcing 100% coverage for production code only
  • src/towncrier/newsfragments/711.misc: Required changelog fragment

Fixes #711

2253781369 added 2 commits May 22, 2026 14:50
Remove test files from the coverage omit list so that test code
is included in coverage tracking. This helps detect tests that
are always skipped.

The CI coverage job now reports full coverage to the step summary
(including test code) while still enforcing 100% coverage for
production code only.

Fixes twisted#711
@adiroiban

Copy link
Copy Markdown
Member

Is this AI generated without any review from a person?

If this was previously reviewed by a person, please double-check the diff.
it looks like this introduces newline changes.

I am closing this for now since I suspect that this is AI spam, but we can reopen it.

@adiroiban adiroiban closed this May 22, 2026
@glyph

glyph commented May 22, 2026

Copy link
Copy Markdown
Member

I've blocked this user. In the future I think we need to be aggressive about such bans because otherwise we leave a vector for abuse open; they can request reconsideration outside of Github if they feel it's unfair.

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.

Enable coverage report for the test code

3 participants