Skip to content

test(sample-11): add unit and e2e tests for swagger#16410

Merged
kamilmysliwiec merged 1 commit into
nestjs:v12.0.0from
zaewc:test/sample-11-swagger
Mar 1, 2026
Merged

test(sample-11): add unit and e2e tests for swagger#16410
kamilmysliwiec merged 1 commit into
nestjs:v12.0.0from
zaewc:test/sample-11-swagger

Conversation

@zaewc

@zaewc zaewc commented Feb 21, 2026

Copy link
Copy Markdown

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe: Add missing unit and e2e tests to sample application

What is the current behavior?

Issue Number: #1539

What is the new behavior?

Added unit tests (CatsController, CatsService) and e2e tests (cats.e2e-spec.ts) to the 11-swagger sample application to ensure the API endpoints and Swagger integration work as expected

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

Related to testing improvements across sample applications

@coveralls

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 68811bff-55fc-4f4e-9a5b-2b94863319fc

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 89.863%

Totals Coverage Status
Change from base Build 686e685e-1e78-471a-bebf-44d42af2d077: 0.0%
Covered Lines: 7473
Relevant Lines: 8316

💛 - Coveralls

@kamilmysliwiec

Copy link
Copy Markdown
Member

Could you rebase to v12.0.0 branch?

@zaewc zaewc force-pushed the test/sample-11-swagger branch from 2e0af80 to 6cb7d3d Compare March 1, 2026 08:37
@zaewc zaewc changed the base branch from master to v12.0.0 March 1, 2026 08:39
@zaewc

zaewc commented Mar 1, 2026

Copy link
Copy Markdown
Author

@kamilmysliwiec

Could you rebase to v12.0.0 branch?

Rebased onto the v12.0.0 branch
Since this branch uses Vitest, I’ve also migrated the test files from Jest to Vitest

@kamilmysliwiec kamilmysliwiec merged commit fc480c8 into nestjs:v12.0.0 Mar 1, 2026
1 check 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.

5 participants