Skip to content

Conversation

@AlexJSully
Copy link
Owner

Realized in #533 that there were no unit tests within this project. Added support for Jest unit testing.

@AlexJSully AlexJSully requested a review from Copilot September 15, 2025 01:52
@AlexJSully AlexJSully self-assigned this Sep 15, 2025
@vercel
Copy link

vercel bot commented Sep 15, 2025

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

Project Deployment Preview Comments Updated (UTC)
portfolio Ready Ready Preview Comment Sep 15, 2025 2:03am

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds Jest unit testing infrastructure to the project, addressing the gap identified in issue #533. It includes comprehensive test coverage for components and utilities, along with CI/CD integration.

  • Sets up Jest testing framework with necessary configuration and dependencies
  • Adds unit tests for key components (Avatar, Navbar, CookieSnackbar, etc.) and utility functions
  • Updates CI/CD workflows to include unit test execution in quality assurance checks

Reviewed Changes

Copilot reviewed 15 out of 20 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
package.json Adds Jest dependencies and test scripts, updates validation workflow
jest/setup.ts Jest configuration with polyfills for Node.js environment
jest/svg-mock.js Mock for SVG imports in Jest tests
src/util/isNetworkFast.test.ts Unit tests for network speed detection utility
src/helpers/ascii.test.ts Unit tests for ASCII logo functionality
src/helpers/aaaahhhh.test.ts Unit tests for image replacement helper functions
src/components/*/test.tsx Unit tests for React components with mocking and event testing
src/components/banner/Avatar.tsx Fixes import path from @Helper to @helpers
src/app/page.tsx Fixes import path from @Helper to @helpers
.github/workflows/*.yaml Updates Node.js action versions and adds unit test steps

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@AlexJSully AlexJSully merged commit b6ffcd9 into main Sep 15, 2025
8 checks passed
@AlexJSully AlexJSully deleted the alexjsully-250914-setup-jest branch September 15, 2025 02:07
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