Add Vitest, CI test gates, and add unit tests #4

Merged

araemer merged 21 commits from add-vitest into meal-plan 2026-06-14 06:40:19 +00:00

Conversation 0 Commits 21 Files changed 50 +7733 -33

araemer commented 2026-06-13 06:17:46 +00:00

Owner

Copy link

Summary

• Installs Vitest and @vitest/coverage-v8; adds test, test:watch, and test:coverage scripts
• Adds a pr_check.yaml workflow that runs tests on every PR targeting stage or main; adds a test step to both deploy workflows so a failing suite blocks deployments
• Adds reflect-metadata setup file required for TypeORM decorators in tests
• Adds 13 unit tests for CommentHandler covering createComment, updateComment, and deleteComment — validation, ownership, admin bypass, not-found, and forbidden paths

Test plan

• npm test passes locally
• Review test structure and coverage of CommentHandler
• Confirm branch protection rule is set up in Forgejo for stage and main requiring the test status check

## Summary - Installs Vitest and `@vitest/coverage-v8`; adds `test`, `test:watch`, and `test:coverage` scripts - Adds a `pr_check.yaml` workflow that runs tests on every PR targeting `stage` or `main`; adds a test step to both deploy workflows so a failing suite blocks deployments - Adds `reflect-metadata` setup file required for TypeORM decorators in tests - Adds 13 unit tests for `CommentHandler` covering `createComment`, `updateComment`, and `deleteComment` — validation, ownership, admin bypass, not-found, and forbidden paths ## Test plan - [x] `npm test` passes locally - [x] Review test structure and coverage of `CommentHandler` - [ ] Confirm branch protection rule is set up in Forgejo for `stage` and `main` requiring the `test` status check

araemer added 3 commits 2026-06-13 06:17:46 +00:00

chore(test) Add Vitest with coverage setup ... d66f70cf30

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

chore(ci) Add test step to all workflows and add PR check workflow ... 6bda390a40

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

test(comment) Add unit tests for CommentHandler ... 9e51401185

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

araemer added 1 commit 2026-06-13 06:21:50 +00:00

refactor(test) Rename __tests__ to tests ... 45a2b7b848

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

araemer added 1 commit 2026-06-13 06:25:14 +00:00

docs(testing) Document testing conventions in CLAUDE.md and architecture.md ... f8de9e9195

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

araemer added 1 commit 2026-06-13 06:29:17 +00:00

docs(test) Explain double cast in makeRepo comment ... 9b569e4b0d

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

araemer added 1 commit 2026-06-13 06:29:28 +00:00

docs(claude) Document the double cast pattern for mock repositories ... e7ddc79e75

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

araemer added 1 commit 2026-06-13 06:32:40 +00:00

docs(test) Add detailed explanatory comments to CommentHandler tests ... f24d1eff1a

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

araemer added 1 commit 2026-06-13 06:35:39 +00:00

docs(test) Clarify createComment test — input vs return value of repo.save ... fb74f05043

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

araemer added 1 commit 2026-06-13 06:38:09 +00:00

docs(test) Remove Java references from comments ... 4283ea4bf5

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

araemer added 1 commit 2026-06-13 06:43:40 +00:00

test(rating) Add unit tests for RatingHandler ... 91ed13d401

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

araemer added 1 commit 2026-06-13 06:46:33 +00:00

test(compactRecipe) Add unit tests for CompactRecipeHandler ... a6a2d4df84

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

araemer added 1 commit 2026-06-13 06:47:58 +00:00

test(recipe) Add unit tests for RecipeHandler ... ecb8f85db3

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

araemer added 1 commit 2026-06-13 06:48:33 +00:00

test(tag) Add unit tests for TagHandler ... b058894f21

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

araemer added 1 commit 2026-06-13 06:49:12 +00:00

test(recipeImage) Add unit tests for RecipeImageHandler ... 35bdd062fa

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

araemer added 1 commit 2026-06-13 06:50:01 +00:00

test(user) Add unit tests for UserHandler ... 642bb02972

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

araemer added 1 commit 2026-06-13 06:51:24 +00:00

test(mealPlan) Add unit tests for MealPlanHandler ... c67c651964

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

araemer added 1 commit 2026-06-13 06:52:21 +00:00

test(auth) Add unit tests for AuthHandler ... 965d6a25d9

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

araemer added 1 commit 2026-06-13 07:24:02 +00:00

test(mappers) Add unit tests for all mapper classes ... 4f8b2f3d05

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

araemer added 1 commit 2026-06-13 07:25:51 +00:00

test(types,utils) Add unit tests for Uuid, SessionCookie, dateUtils, asyncHandler, and encryptionUtils ... 64303213dc

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

araemer added 1 commit 2026-06-13 07:38:51 +00:00

test(services) Add unit tests for all application service classes ... 332e5e71c4

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

araemer changed title from Add Vitest, CI test gates, and CommentHandler unit tests to Add Vitest, CI test gates, and add unit tests 2026-06-14 05:54:35 +00:00

araemer merged commit d046d9c01c into meal-plan 2026-06-14 06:40:19 +00:00

araemer referenced this pull request from a commit 2026-06-14 06:40:20 +00:00

Merge pull request 'Add Vitest, CI test gates, and add unit tests' (#4) from add-vitest into meal-plan

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

araemer/recipe-backend!4

No description provided.