Skip to content

test: implement unit tests#7

Merged
nipsysdev merged 8 commits intomainfrom
test/implement_unit_tests
Jul 7, 2025
Merged

test: implement unit tests#7
nipsysdev merged 8 commits intomainfrom
test/implement_unit_tests

Conversation

@nipsysdev
Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings July 7, 2025 14:38
Copy link
Copy Markdown
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 sets up Vitest and adds a comprehensive suite of unit tests across utilities, hooks, contexts, and components, alongside the necessary configuration and CI updates.

  • Introduce vitest.config.mjs and test setup.
  • Add tests for all utility functions, hooks, contexts, and UI components.
  • Update package.json, Lefthook config, and GitHub workflow to integrate testing.

Reviewed Changes

Copilot reviewed 43 out of 44 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
vitest.config.mjs Configure Vitest for unit testing and coverage
package.json Add test scripts, devDependencies for Vitest & RTL
lefthook.yml Define pre-commit and pre-push hooks invoking tests
lefthook.yaml Now empty/duplicate config
.github/workflows/check.yml Add test job to run tests in CI
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported
Comments suppressed due to low confidence (1)

Comment thread .github/workflows/check.yml Outdated
@nipsysdev nipsysdev force-pushed the test/implement_unit_tests branch from c441bdf to a915b46 Compare July 7, 2025 14:49
@nipsysdev nipsysdev force-pushed the test/implement_unit_tests branch from 7bbe3ef to 75cd179 Compare July 7, 2025 14:59
@nipsysdev nipsysdev merged commit 3bdd223 into main Jul 7, 2025
4 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