Skip to content

test(api/v2): ✅ Add UtilityController tests#117

Merged
Nudelsuppe42 merged 6 commits into
api/v2from
api-v2/testing
May 20, 2026
Merged

test(api/v2): ✅ Add UtilityController tests#117
Nudelsuppe42 merged 6 commits into
api/v2from
api-v2/testing

Conversation

@kyanvde
Copy link
Copy Markdown
Contributor

@kyanvde kyanvde commented May 20, 2026

Add tests for the UtilityController as a start of making the api fully covered by automatic tests. This PR is mainly to setup the testing suite.

@kyanvde kyanvde requested review from Nudelsuppe42 and Copilot May 20, 2026 18:22
@kyanvde kyanvde self-assigned this May 20, 2026
@kyanvde kyanvde added feature/api New feature or request dependencies Pull requests that update a dependency file labels May 20, 2026
@kyanvde kyanvde added this to the v2.0.0-api release milestone May 20, 2026
Copy link
Copy Markdown

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

Adds an initial Jest-based unit test suite for api-v2 by introducing tests for UtilityController and updating the project’s TypeScript/Jest configuration to support running tests from the test/ directory.

Changes:

  • Add UtilityController unit tests covering getHealth, getVersion, and deprecated route behavior.
  • Update Jest configuration to run tests from apps/api-v2/test and map src/* imports.
  • Update TypeScript config and dev dependencies/lockfile to support the testing setup.

Reviewed changes

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

File Description
yarn.lock Updates lockfile for Jest-related packages and bumps @nestjs/testing.
apps/api-v2/tsconfig.json Adds Jest/Node types and enables isolatedModules.
apps/api-v2/test/sections/utility/utility.controller.spec.ts Introduces initial unit tests for UtilityController.
apps/api-v2/package.json Updates dev deps (@nestjs/testing, @types/jest) and adjusts Jest rootDir + module mapping.

Comment thread apps/api-v2/tsconfig.json Outdated
Comment thread apps/api-v2/package.json Outdated
Comment thread apps/api-v2/package.json Outdated
Comment thread apps/api-v2/test/sections/utility/utility.controller.spec.ts Outdated
kyanvde and others added 4 commits May 20, 2026 20:31
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown

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

Copilot reviewed 4 out of 5 changed files in this pull request and generated 3 comments.

Comment thread apps/api-v2/tsconfig.spec.json Outdated
Comment thread apps/api-v2/package.json Outdated
Comment thread apps/api-v2/test/sections/utility/utility.controller.spec.ts Outdated
Copy link
Copy Markdown

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

Copilot reviewed 3 out of 4 changed files in this pull request and generated no new comments.

@github-project-automation github-project-automation Bot moved this from Backlog to In Progress in @BuildTheEarth/web Tracker May 20, 2026
@Nudelsuppe42 Nudelsuppe42 merged commit 2d354b0 into api/v2 May 20, 2026
2 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in @BuildTheEarth/web Tracker May 20, 2026
@Nudelsuppe42 Nudelsuppe42 deleted the api-v2/testing branch May 20, 2026 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file feature/api New feature or request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants