Skip to content

test: add coverage for presentational components#31

Closed
Soengkit wants to merge 1 commit into
StableRoute-Org:mainfrom
Soengkit:codex/stableroute-component-tests-13
Closed

test: add coverage for presentational components#31
Soengkit wants to merge 1 commit into
StableRoute-Org:mainfrom
Soengkit:codex/stableroute-component-tests-13

Conversation

@Soengkit

Copy link
Copy Markdown

Summary

  • Add Button tests for default/secondary/danger variants, disabled state, and prop/className forwarding.
  • Add EmptyState and PageHeading tests for optional description/action slots being present and omitted.
  • Extend Card coverage for omitted optional regions and div prop forwarding.

Closes #13

Validation

  • npm run lint
  • npm test -- --coverage
  • NEXT_TELEMETRY_DISABLED=1 npm run build

Coverage note: Button, Card, EmptyState, and PageHeading report 100% statement/branch/function/line coverage.

@mikewheeleer

Copy link
Copy Markdown
Contributor

thanks for jumping in @Soengkit! 🙏 we assign issues before work starts so two people don't build the same thing, and #13 isn't assigned to you. please grab an open unassigned issue and PR against that. closing this one for now, hope to see you back 💪

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.

Add component tests for Button, Card, EmptyState, and PageHeading

2 participants