Skip to content

test(web): cover classNames utility#97

Open
imbhargav5 wants to merge 4 commits into
mainfrom
automation/nextbase-maintainer-20260627-103446
Open

test(web): cover classNames utility#97
imbhargav5 wants to merge 4 commits into
mainfrom
automation/nextbase-maintainer-20260627-103446

Conversation

@imbhargav5

Copy link
Copy Markdown
Owner

Summary

  • replace the placeholder Vitest assertion with focused coverage for the classNames helper
  • cover joining multiple class names and omitting undefined entries

Verification

  • pnpm --filter web test
  • pnpm --filter web lint
  • pnpm --filter web typecheck

@codesandbox

codesandbox Bot commented Jun 27, 2026

Copy link
Copy Markdown

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

@imbhargav5

Copy link
Copy Markdown
Owner Author

CI did not pass, so this PR was not merged. The blocking check is Playwright e2e (24.x), failing in existing logged-in private item flows with Supabase permission denied for table private_items. Local verification for this test-only change passed: pnpm --filter web test, pnpm --filter web lint, and pnpm --filter web typecheck.

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.

1 participant