Skip to content

#525: Github_graph.rb excluded from coverage#533

Merged
yegor256 merged 2 commits into
zerocracy:masterfrom
VasilevNStas:fix/525-github-graph-tests
Jun 8, 2026
Merged

#525: Github_graph.rb excluded from coverage#533
yegor256 merged 2 commits into
zerocracy:masterfrom
VasilevNStas:fix/525-github-graph-tests

Conversation

@VasilevNStas

@VasilevNStas VasilevNStas commented Jun 3, 2026

Copy link
Copy Markdown
Contributor
  • Removed add_filter lib/fbe/github_graph.rb from SimpleCov config in test__helper.rb
  • Added 16 new tests for the real Fbe::Graph class by stubbing the query method
  • Coverage improved from 52% → 89% for github_graph.rb, above the 80% minimum
  • All 299 existing tests pass, no new RuboCop offenses

…ph tests

The add_filter excluded github_graph.rb from coverage despite having tests.
Removed the filter and added 16 new tests for the real Fbe::Graph class methods
by stubbing the query method with mock objects. This achieves 89% coverage
for the file, above the 80% minimum.

- Added tests for: total_commits, total_issues_and_pulls, resolved_conversations,
  issue_type_event, total_commits_pushed, total_issues_created,
  total_releases_published, pull_requests_with_reviews, pull_request_reviews
@VasilevNStas VasilevNStas changed the title #525 github_graph.rb excluded from coverage #525: Github_graph.rb excluded from coverage Jun 8, 2026
@VasilevNStas

Copy link
Copy Markdown
Contributor Author

@yegor256 please review

@yegor256 yegor256 merged commit 71bfdc5 into zerocracy:master Jun 8, 2026
10 checks passed
@0crat

0crat commented Jun 8, 2026

Copy link
Copy Markdown

@VasilevNStas Thanks for the contribution! You've earned +20 points for this: +24 as a basis, +16 for hits-of-code (315 * 0.075 capped at 16), -4 for exceeding 133 hits-of-code, and -16 for excessive review comments. Please, keep them coming. Your running score is +824; don't forget to check your Zerocracy account too.

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.

3 participants