Skip to content

Add unit tests, Vitest setup, and CI test workflow#34

Merged
viignacio merged 3 commits into
mainfrom
chore/setup-unit-tests
Mar 16, 2026
Merged

Add unit tests, Vitest setup, and CI test workflow#34
viignacio merged 3 commits into
mainfrom
chore/setup-unit-tests

Conversation

@viignacio

Copy link
Copy Markdown
Owner

Summary

  • Install Vitest + Testing Library + jsdom; add npm run test and npm run test:ui scripts
  • Add 124 unit tests covering all hooks (useIsDesktop, useMouseGlow, useSwipe) and all React components
  • Add GitHub Actions workflow (.github/workflows/test.yml) to run tests on every PR open/update
  • Remove unused components/icons/technologies/ — confirmed nothing imports from it

- Install Vitest, Testing Library, and jsdom; add npm test scripts
- Add 124 unit tests across all hooks and React components
- Remove unused components/icons/technologies (not referenced anywhere)
- Add GitHub Actions workflow to run tests on pull requests

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@vercel

vercel Bot commented Mar 16, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
van-portfolio Ready Ready Preview, Comment Mar 16, 2026 5:28pm

- NotFound page was completely untested; adds 4 tests covering the
  404 code, heading, body message, and back-to-home link
- AboutSection: covers ProficiencyIcon for all four levels, the
  "< 1 year" / "1 year" getDuration edge cases, category defaulting
  to "Other", and the title fallback to "About Me"
- Layout: covers body scroll lock/restore on mobile menu, footer
  mailto/tel href attributes, social link target+rel attributes,
  and the expand-navigation pill button

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Replace 4 copy-paste proficiency tests with a single it.each table
- Collapse 4 separate NotFound renders into one beforeEach render
- Remove dead getByRole('generic') variable in MediaRenderer test
- Remove redundant setWindowWidth(1024) shadowed by beforeEach
- Clarify CertificationsSection dot re-query comment

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@viignacio
viignacio merged commit 2e521ce into main Mar 16, 2026
3 checks passed
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.

2 participants