🧪 test: Add unit tests for getIdeDisplayName and detectIde in core#57
🧪 test: Add unit tests for getIdeDisplayName and detectIde in core#57TrueAlpha-spiral wants to merge 1 commit intomainfrom
Conversation
This commit adds a new test file, `packages/core/src/ide/detect-ide.test.ts`, which tests the `getIdeDisplayName` and `detectIde` functions. The tests cover the current single implementation case for VS Code and utilize `vi.stubEnv` to effectively verify the environment-based detection logic. Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
1 participant
🎯 What: The lack of test coverage for the simple
getIdeDisplayNameanddetectIdefunctions inpackages/core/src/ide/detect-ide.tshas been addressed.📊 Coverage: The new tests in
packages/core/src/ide/detect-ide.test.tsverify theDetectedIde.VSCodecase forgetIdeDisplayName. Furthermore, test coverage has been provided fordetectIdeusing Vitest'svi.stubEnvto cover cases whereTERM_PROGRAMis "vscode", something else, or undefined.✨ Result: Test coverage for
packages/core/src/ide/detect-ide.tsis vastly improved. Code regressions in these functions will now be caught, and new IDE cases added to the enum will be properly unit-testable.PR created automatically by Jules for task 8252235490452098005 started by @TrueAlpha-spiral