Update testing docs to reference Bun test and preload setup#398
Conversation
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
et3 | 318a4ae | Commit Preview URL Branch Preview URL |
Jan 14 2026, 03:39 PM |
Summary of ChangesHello @zz-plant, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request updates the project's documentation to accurately reflect the current testing infrastructure, transitioning all references from Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request updates the testing documentation to replace references to Vitest with bun test, aligning the docs with the project's current tooling. The changes in docs/testing-todos.md and docs/diagnostics-capacity-forecaster.md are accurate and improve clarity. I have provided one suggestion for docs/testing-todos.md to re-introduce the command for running tests in watch mode, as this is a common developer workflow that was mentioned in the previous version of the documentation.
| modules. Run `bun test` for local coverage and `bun run test:unit:ci` for a full run with the lcov | ||
| reporter. |
There was a problem hiding this comment.
The previous version of the documentation mentioned running tests in watch mode, which is a very common development workflow. This information was lost in the update. It would be helpful to include how to run tests in watch mode with bun test for completeness.
| modules. Run `bun test` for local coverage and `bun run test:unit:ci` for a full run with the lcov | |
| reporter. | |
| modules. Run `bun test` for local coverage or `bun test --watch` for watch mode. Use `bun run test:unit:ci` | |
| for a full run with the lcov reporter. |
Motivation
vitest.config.ts, which could confuse contributors running tests.Description
docs/testing-todos.mdto replace Vitest references withbun testand to describe thebunfig.tomlpreload ofsrc/test/preload.tsfor happy-dom and Astro virtual module mocks.docs/diagnostics-capacity-forecaster.mdto reference Bun test coverage for the projection engine tests.bunx prettier --writeto keep style consistent.Testing
bun run checkwas intentionally skipped.bunx prettier --writeand completed without modifying content beyond the intended edits.Codex Task