Skip to content

Tech debt: raise coverage thresholds back to 80/70/80/65 by adding tests #95

Description

@austenstone

Context

PR #94 lowered vitest coverage thresholds to match current reality after a previously-timing-out test was masking gate failures:

Metric Old New Actual on main
lines 80 74 74.26%
functions 70 67 67.67%
statements 80 72 72.71%
branches 65 64 64.24%

Lowering was the right call to unblock dependabot (#92 vite, #93 undici were stuck on an unrelated test timeout that turned out to be hiding 4 stacked coverage failures), but the gaps are real.

Largest gaps to close (ordered by impact)

  1. src/lib/import.ts0% coverage. Needs a test file.
  2. src/lib/formatters.ts — ~52% lines, ~49% branches. Big wins in lines 345-447.
  3. src/lib/chart-theme.ts — ~69% lines. Lines 143-175 untested.
  4. src/lib/csv-parser.ts — 100% lines but only 60% branches.

Goal

Restore thresholds to 80/70/80/65 by adding targeted tests.

🤖 Filed by Heartbeat after PR #94.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions