Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

test: add SocialIcon unit tests #936

Open
wants to merge 41 commits into
base: main
Choose a base branch
from
Open

test: add SocialIcon unit tests #936

wants to merge 41 commits into from

Conversation

devin-ai-integration[bot]
Copy link
Contributor

Added unit tests for SocialIcon component to ensure:

  • Basic rendering and link functionality
  • Hover state transitions
  • Colored icon presence and opacity changes
  • Color prop application and hover state color changes

Link to Devin run: https://app.devin.ai/sessions/6abe38a787e6423aaa60d024441e3f1b

@devin-ai-integration Devin.ai Integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add "(aside)" to your comment to have me ignore it.
  • Look at CI failures and help fix them

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@cloudflare-workers-and-pages Cloudflare Workers and Pages
Copy link

cloudflare-workers-and-pages bot commented Jan 28, 2025 •
edited
Loading

Deploying ens-app-v3 with  Cloudflare Pages  Cloudflare Pages

Latest commit: 1440c02
Status: ✅  Deploy successful!
Preview URL: https://922d95c2.ens-app-v3.pages.dev
Branch Preview URL: https://devin-1738082410-add-social.ens-app-v3.pages.dev

View logs

devin-ai-integration bot and others added 19 commits January 28, 2025 16:48
@devin-ai-integration @LeonmanRolls
test: improve SocialIcon test coverage for hover states and transitions
6892469 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
fix: add missing closing brace in SocialIcon test
95a6769 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
test: add test case for SocialIcon without ColoredIcon prop
459e8f4 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
test: add test case for transition styles
0fbc3a2 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
test: fix style assertions in SocialIcon tests
1f66ef5 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
test: improve SocialIcon test coverage with style and hover assertions
52a7b59 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
test: update style assertions to match testing library expectations
9e2303c 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
test: update style assertions to use object format consistently
c81decd 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
test: update style assertions to use string values for opacity and th…
d1d4e07 
…eme variables

Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
test: update style assertions with complete style properties
1beb6e5 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
test: add theme-based style assertions for width and height
efae99d 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
test: add comprehensive hover state transition tests
4f05fbb 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
test: update style assertions to use individual checks and RGB values
6321771 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
test: update remaining style assertions to use individual checks and …
e345eff 
…RGB values

Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
test: add key prop and component type assertions
2790319 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
test: add test case for undefined color behavior
a2000fc 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
test: add comprehensive style assertions for StyledColoredIcon
fe249d2 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
test: add additional test cases for theme spacing and transitions
4754f72 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
test: update style assertions to use toHaveAttribute with stringConta…
524c26b 
…ining

Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration Devin.ai Integration
Copy link
Contributor Author

Devin is currently unreachable - the session may have died.

devin-ai-integration bot and others added 6 commits January 29, 2025 01:47
devin-ai-integration bot and others added 15 commits January 29, 2025 02:57
@devin-ai-integration @LeonmanRolls
test: update remaining style assertions to use toHaveStyle
582c0f2 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
test: update remaining style assertions to use toHaveStyle consistently
7dc2a74 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
test: update coverage configuration with thresholds and reporters
c03681d 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
test: add lcov reporter and coverage directory configuration
1d9f2c4 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
test: update style assertions to target correct styled components
707f9f0 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
test: add sonar coverage reporter and improve coverage configuration
1472600 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
chore: add sonar-project.properties for coverage reporting
3f38879 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
ci: add SonarCloud coverage reporting step
692424b 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
ci: update SonarCloud configuration for better coverage reporting
8c3e5dc 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
ci: update coverage reporting with v4 upload-artifact
ca4a1d6 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
ci: fix coverage path handling for SonarCloud
367450f 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
test: update style assertions to use getComputedStyle instead of toHa…
720969f 
…veStyle

Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
test: update style assertions to use getComputedStyle with comprehens…
4c88b0c 
…ive coverage

Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
ci: improve coverage artifact handling and path fixing for SonarCloud
6dbf07c 
Co-Authored-By: Leon Talbert <[email protected]>
@devin-ai-integration @LeonmanRolls
ci: improve coverage configuration and reporting
1440c02 
Co-Authored-By: Leon Talbert <[email protected]>
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
32 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

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

@sugh01 sugh01 Awaiting requested review from sugh01 sugh01 is a code owner

At least 0 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
0 participants