Skip to content

Add HTML report smoke tests - #172

Merged
yakew7 merged 4 commits into
yakew7:mainfrom
ahmdkaml:add-report-html-tests
Aug 6, 2026
Merged

Add HTML report smoke tests#172
yakew7 merged 4 commits into
yakew7:mainfrom
ahmdkaml:add-report-html-tests

Conversation

@ahmdkaml

@ahmdkaml ahmdkaml commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds baseline test coverage for to_html() and compare_to_html().

The tests use small mock profile/compare results to exercise the HTML rendering layer directly, verify the generated reports render successfully, and check that key computed values appear in the output.

Using mocked inputs keeps the tests focused on the report functions themselves and provides a simple starting point for expanding coverage in the future without having to drive the full profiling/comparison pipeline.

Type

  • Audit
  • Explainer
  • Bug fix
  • Other

Linked issue

Closes #130

@vercel

vercel Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

@ahmdkaml is attempting to deploy a commit to the yashkewlani2020-gmailcom's projects Team on Vercel.

A member of the Team first needs to authorize it.

@ahmdkaml
ahmdkaml marked this pull request as ready for review August 6, 2026 12:34
Copilot AI lite review requested due to automatic review settings August 6, 2026 12:34

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@yakew7
yakew7 merged commit 4f94354 into yakew7:main Aug 6, 2026
13 of 14 checks passed
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.

faircode/report.py has no dedicated test coverage

3 participants