Skip to content

test(generator): add coverage for hide_title parameter in generateSVG#901

Open
Subhooo5 wants to merge 1 commit into
JhaSourav07:mainfrom
Subhooo5:test/hide-title-generator-coverage
Open

test(generator): add coverage for hide_title parameter in generateSVG#901
Subhooo5 wants to merge 1 commit into
JhaSourav07:mainfrom
Subhooo5:test/hide-title-generator-coverage

Conversation

@Subhooo5
Copy link
Copy Markdown
Contributor

Description

Fixes #722

  • Added generator-level test coverage for the hide_title parameter, verifying that the username title is correctly omitted when enabled.
  • Added a complementary assertion ensuring the uppercase username title still renders normally when hide_title is disabled.

Pillar

  • 🛠️ Other (Bug fix, refactoring, docs)

Checklist before requesting a review:

  • I have read the CONTRIBUTING.md file.
  • I have tested these changes locally (localhost:3000/api/streak?user=YOUR_USERNAME).
  • I have run npm run format and npm run lint locally and resolved all errors.
  • My commits follow the Conventional Commits format.
  • I have starred the repo.
  • I have made sure that i have only one commit to merge in this PR.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 28, 2026

@Subhooo5 is attempting to deploy a commit to the jhasourav07's projects Team on Vercel.

A member of the Team first needs to authorize it.

@Subhooo5
Copy link
Copy Markdown
Contributor Author

Hey @JhaSourav07 Successfully added the required test. All existing tests pass locally. Feel free to review, add necessary labels and merge it. 🫂🚀

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.

test(svg generator): verify hide_title=true removes username text from output

2 participants