Repository navigation
Upgrades Vitest to v5 and fixes async assertions - #118
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
There was a problem hiding this comment.
Copilot review overview
🔵 Needs a closer look
Successful CI validation of the major test-toolchain upgrade was not verified.
Review effort: Balanced
Findings: 1
Open (1)
What changed in this PR
Upgrades the JWT validator’s test toolchain to Vitest 5 and updates tests and contributor guidance.
Changes:
- Upgrades Vitest and V8 coverage, locking both to 5.0.3.
- Awaits rejected-promise assertions in JWKS failure tests.
- Documents development requirements and testing commands.
| File | Description |
|---|---|
| readme.md | Adds development and testing guidance. |
| pnpm-lock.yaml | Locks Vitest 5 and updated dependencies. |
| package.json | Updates Vitest and coverage version ranges. |
| lib/tests/badPathTest.test.ts | Awaits asynchronous rejection assertions. |
Files not reviewed (1)
- pnpm-lock.yaml: Generated file
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| The development toolchain requires Node.js 22.12 or newer and the pnpm version | ||
| pinned in `package.json`. These are development requirements, not consumer runtime | ||
| requirements. |

Upgrades the Vitest testing framework and its coverage tool to version 5 to stay up-to-date with the latest testing features, performance improvements, and dependencies.
Kills #110
Key Changes
.rejects), which otherwise fail silently or cause test instability.