Skip to content

fix: add schema map tests to fix coverage CI failure#630

Merged
horza-bora merged 1 commit intomainfrom
fix/coverage-schema-map-tests
Feb 24, 2026
Merged

fix: add schema map tests to fix coverage CI failure#630
horza-bora merged 1 commit intomainfrom
fix/coverage-schema-map-tests

Conversation

@Dave-London
Copy link
Owner

Summary

  • Add unit tests for 5 schemaXxxMap functions introduced in feat: optimize output schemas across all 16 server packages #628 that lacked test coverage, causing the coverage CI check to fail on PR chore: version packages #625:
    • server-security: schemaTrivyScanMap, schemaSemgrepScanMap, schemaGitleaksScanMap (functions 71.87% → 90.62%)
    • server-http: schemaResponseMap, schemaHeadResponseMap (functions 79.16% → 87.5%)
  • Update SECURITY-AUDIT.md to reflect 26/26 security test coverage (9 previously missing packages now have security.test.ts)
  • Update COVERAGE-AUDIT.md with actual per-package test counts (total: 6,111)

Test plan

  • pnpm --filter @paretools/security exec vitest run --coverage — all thresholds pass
  • pnpm --filter @paretools/http exec vitest run --coverage — all thresholds pass
  • CI coverage check passes (unblocks chore: version packages #625)

🤖 Generated with Claude Code

…t docs

Add unit tests for untested schemaXxxMap functions introduced in #628:
- server-security: schemaTrivyScanMap, schemaSemgrepScanMap, schemaGitleaksScanMap
- server-http: schemaResponseMap, schemaHeadResponseMap

This fixes the CI coverage failure (functions below 80% threshold) blocking PR #625.

Also updates SECURITY-AUDIT.md to reflect 26/26 security test coverage
(9 previously missing packages now have security.test.ts files) and
COVERAGE-AUDIT.md with actual per-package test counts.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@horza-bora horza-bora merged commit c1298a6 into main Feb 24, 2026
11 checks passed
@horza-bora horza-bora deleted the fix/coverage-schema-map-tests branch February 24, 2026 16:28
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