Skip to content

Fix default HTML bundled template resolution#9

Merged
oscarcv merged 1 commit intomainfrom
fix-html-default-template-resolution
Feb 20, 2026
Merged

Fix default HTML bundled template resolution#9
oscarcv merged 1 commit intomainfrom
fix-html-default-template-resolution

Conversation

@oscarcv
Copy link
Owner

@oscarcv oscarcv commented Feb 20, 2026

Summary

  • Fix default HTML template lookup so snapshot-report can always render HTML without requiring --html-template.
  • Prefer loading the bundled template via Bundle.module first.
  • Expand filesystem fallback candidates to support both bundle layouts used in packaging:
    • SnapshotReportKit_SnapshotReportCore.bundle/default-report.stencil
    • SnapshotReportKit_SnapshotReportCore.bundle/Contents/Resources/default-report.stencil
  • Update template-candidate tests to assert both bundle layouts.

Checklist

  • I ran swift test
  • I updated docs if behavior changed
  • I added/updated tests for new behavior
  • I verified CI-related files if workflow behavior changed

Validation

  • Commands run:
    • swift test --filter htmlRendererTemplateCandidatesIncludeResolvedSymlinkPath
    • swift test --filter htmlRendererTemplateCandidatesIncludeSourceFallback
    • swift test

Screenshots / Artifacts (if applicable)

  • N/A

@oscarcv oscarcv merged commit d9d7662 into main Feb 20, 2026
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.

1 participant