Skip to content

test(avatar): cover explicit initials prop override - #5417

Closed
TruptiAgrawal wants to merge 1 commit into
uber:mainfrom
TruptiAgrawal:test/avatar-initials-prop
Closed

TruptiAgrawal wants to merge 1 commit into
uber:mainfrom
TruptiAgrawal:test/avatar-initials-prop

Conversation

@TruptiAgrawal

@TruptiAgrawal TruptiAgrawal commented Sep 10, 2026 •

Copy link
Copy Markdown

Description

Added a test to verify that the explicit initials prop takes precedence when provided to the Avatar component.

The test ensures that the Avatar renders the explicitly provided initials correctly.

Scope

Patch: Bug Fix

@CLAassistant

CLAassistant commented Sep 10, 2026 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@dyesin

dyesin commented Sep 25, 2026

Copy link
Copy Markdown
Collaborator

Thanks for the PR. Before we review it, please note that "check this" isn't a submission we can accept. It hands the verification work to maintainers, and our review time is limited.

We welcome AI-assisted contributions, but you are the author and must stand behind every line. Before requesting review, please:

Run it. Set up the dev environment, run the full test suite, and confirm the change works as intended.
Understand it. Be ready to explain why each change was made and what could break. "The model suggested it" isn't an answer.
Justify it. Describe the actual problem this solves and how you confirmed it's fixed. For performance changes, include evidence.
Test it. Add or update tests following the patterns already used in the repo.
Keep it focused. One concern per PR, with no unrelated edits.

Once you can confirm all of the above in the PR description, we're happy to take a look. Until then, we'll mark this as a draft.

@dyesin dyesin closed this Sep 25, 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.

3 participants