Skip to content

Backfill coverage gaps and exclude non-product files from coverage - #290

Merged
gabrieldeespindula merged 1 commit into
masterfrom
close-coverage-gaps
Aug 20, 2026
Merged

Backfill coverage gaps and exclude non-product files from coverage#290
gabrieldeespindula merged 1 commit into
masterfrom
close-coverage-gaps

Conversation

@gabrieldeespindula

Copy link
Copy Markdown
Member

Supersedes #288, rebuilt on top of current master (post-octocov, #289). The content is @pnzrfst's work, cherry-picked as a single clean commit with his authorship — the original branch carried the couve commits from #285/#286, which were closed in favor of #289.

What it does

Result

Line coverage goes from 99.5% to 100.0% (3595/3595) — verified locally with the full suite green.

The octocov comment on this PR is also the first real demonstration of the delta against the master baseline recorded since #289 landed.

🤖 Generated with Claude Code

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

Copy link
Copy Markdown

Code Metrics Report

master (1c84fe4) #290 (f3cdb31) +/-
Coverage 99.5% 100.0% +0.4%
Test Execution Time 21s 19s -2s
Details
  |                     | master (1c84fe4) | #290 (f3cdb31) |  +/-  |
  |---------------------|------------------|----------------|-------|
+ | Coverage            |            99.5% |         100.0% | +0.4% |
  |   Files             |              114 |            111 |    -3 |
  |   Lines             |             3616 |           3595 |   -21 |
- |   Covered           |             3598 |           3595 |    -3 |
+ | Test Execution Time |              21s |            19s |   -2s |

Code coverage of files in pull request scope (94.7% → 100.0%)

Files Coverage +/- Status
lib/br_danfe/cce_lib/document.rb 100.0% +8.0% affected
lib/br_danfe/danfe_lib/nfce_lib/document.rb 100.0% +12.5% affected
lib/br_danfe/danfe_lib/nfce_lib/header.rb 100.0% +2.0% modified
lib/br_danfe/danfe_lib/nfe_lib/document.rb 100.0% +2.7% affected
lib/prawn/font.rb 0.0% -100.0% affected
lib/prawn/font_metric_cache.rb 0.0% -93.4% affected
spec/br_danfe/cce_lib/document_spec.rb 100.0% 0.0% modified
spec/br_danfe/danfe_lib/nfce_lib/document_spec.rb 100.0% 0.0% modified
spec/br_danfe/danfe_lib/nfe_lib/document_spec.rb 100.0% 0.0% modified
spec/support/have_same_content_of.rb 0.0% -44.5% affected

Reported by octocov

@gabrieldeespindula gabrieldeespindula self-assigned this Aug 20, 2026
@gabrieldeespindula
gabrieldeespindula merged commit 3624211 into master Aug 20, 2026
2 checks passed
@gabrieldeespindula
gabrieldeespindula deleted the close-coverage-gaps branch August 20, 2026 13:03
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.

2 participants