Skip to content

Conversation

@aspekt19
Copy link

This PR adds a new unit test case to the FundButton component to ensure full test coverage for its props.

Specifically:

  • Adds a test for the hideIcon prop, verifying that the fund icon is correctly removed from the DOM when hideIcon={true} is passed to the component.

This contribution helps to prevent future regressions related to the icon's visibility and contributes to maintaining the project's required 100% test coverage standard.

Added a test to verify that the fund icon is hidden when the hideIcon prop is true.
@cb-heimdall
Copy link

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@vercel
Copy link

vercel bot commented Nov 19, 2025

@aspekt19 is attempting to deploy a commit to the Coinbase Team on Vercel.

A member of the Team first needs to authorize it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants