Skip to content

Increase test coverage for report utilities - #600

Merged
Schiano-NOAA merged 4 commits into
mainfrom
copilot/nmfs-ostasar-599-increase-code-coverage
Oct 20, 2026
Merged

Schiano-NOAA merged 4 commits into
mainfrom
copilot/nmfs-ostasar-599-increase-code-coverage

Conversation

Copilot AI commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

What is the feature?

  • Expand test coverage for accessibility and table-export utilities to support the goal of exceeding 70% code coverage.

How have you implemented the solution?

  • Add fixture-based tests for tagged and tooltip alt-text insertion, including existing text, unmatched labels, renaming, and missing-file errors.
  • Cover LaTeX tagging, table-header insertion, documents without tables, and split-table export.

Does the PR impact any other area of the project, maybe another repo?

  • No.

Copilot AI linked an issue Oct 14, 2026 that may be closed by this pull request
Co-authored-by: Schiano-NOAA <125507018+Schiano-NOAA@users.noreply.github.com>
Copilot AI changed the title [WIP] Increase code coverage to exceed 70% Increase test coverage for report utilities Oct 14, 2026
Copilot AI requested a review from Schiano-NOAA October 14, 2026 15:02
Co-authored-by: Schiano-NOAA <125507018+Schiano-NOAA@users.noreply.github.com>
@Schiano-NOAA

Copy link
Copy Markdown
Collaborator

@copilot please see if there are any other functions or processes that need a test to help increase our code coverage of the package

Co-authored-by: Schiano-NOAA <125507018+Schiano-NOAA@users.noreply.github.com>

Copilot AI commented Oct 19, 2026

Copy link
Copy Markdown
Contributor Author

@copilot please see if there are any other functions or processes that need a test to help increase our code coverage of the package

I added focused tests for add_child() and format_quarto() in commit 687e041, covering labeled child chunks, pagebreak/table handling, PDF/HTML formatting, and unsupported-format errors. I couldn't execute the R tests locally because R/Rscript isn't installed.

@Schiano-NOAA
Schiano-NOAA marked this pull request as ready for review October 20, 2026 21:03
@Schiano-NOAA
Schiano-NOAA merged commit 28b2bdc into main Oct 20, 2026
9 checks passed
@Schiano-NOAA
Schiano-NOAA deleted the copilot/nmfs-ostasar-599-increase-code-coverage branch October 20, 2026 21: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.

Increase code coverage to >70%

2 participants