Skip to content

ci: compare unit coverage against the coverage-history branch - #1024

Merged
dkhawk merged 3 commits into
mainfrom
ci/unit-coverage-baseline
Oct 1, 2026
Merged

dkhawk merged 3 commits into
mainfrom
ci/unit-coverage-baseline

Conversation

@kikoso

@kikoso kikoso commented Oct 1, 2026

Copy link
Copy Markdown
Collaborator

Problem

The unit coverage comment on every PR says "No unit baseline recorded in coverage/history.csv yet" and only shows absolute numbers. The coverage history moved from main to the coverage-history branch, but test.yml still runs compare against coverage/history.csv in the checkout, which no longer exists.

Fix

  • Before compare, fetch history.csv from coverage-history and pass it with --csv. If the fetch fails, compare falls back to absolute numbers as today, and the step is already continue-on-error.
  • The comment footer said history is recorded in coverage/history.csv; it now points at the coverage-history branch.

Testing

Ran the same fetch and compare locally against current unit reports. The comment now reads "Compared against 109b74a on main (#1020)" with a change column per module instead of the no-baseline message. This PR's own coverage comment should show the same.

@github-actions

github-actions Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

Coverage (unit tests)

Compared against 61c3cf0 on main (#1024).

Module Line % Change Branch % Change
maps-compose 0.00% no change 0.00% no change
maps-compose-utils 2.04% no change 0.49% no change
maps-compose-widgets 0.00% no change 0.00% no change
TOTAL 0.42% no change 0.09% no change

Line and branch coverage from unit test reports. History is recorded on the coverage-history branch after each merge to main.

@googlemaps-bot

googlemaps-bot commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Code Coverage

Overall Project 24.66% ❌

There is no coverage information present for the Files changed

@kikoso
kikoso marked this pull request as ready for review October 1, 2026 17:30
@kikoso
kikoso requested a review from a team as a code owner October 1, 2026 17:30
The coverage history moved from main to the coverage-history branch, but
the unit coverage comment still read coverage/history.csv from the
checkout. It no longer exists, so every PR comment said no baseline was
recorded. Fetch history.csv from coverage-history before comparing; if
that fails, compare still falls back to absolute numbers. Also fix the
comment footer, which still pointed at coverage/history.csv.
@kikoso
kikoso force-pushed the ci/unit-coverage-baseline branch from a8c6474 to dd88f44 Compare October 1, 2026 19:02
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