Skip to content

Conversation

@cte
Copy link
Collaborator

@cte cte commented Jun 17, 2025

Description

Related: #4607


Important

Convert all webview-ui tests from Jest to Vitest, updating configurations, mocks, and test file naming conventions.

  • Test Conversion:
    • Convert all test files in webview-ui from Jest to Vitest, including spec.ts and spec.tsx files.
    • Update test imports and mocks to use vi instead of jest.
    • Remove Jest-specific setup files like setupTests.tsx and replace with vitest.setup.ts.
  • Configuration:
    • Add vitest.config.ts for Vitest configuration, specifying test environment, setup files, and file inclusion patterns.
    • Update package.json scripts to use vitest instead of jest for running tests.
  • Mocking and Setup:
    • Replace Jest mocks with Vitest mocks using vi.mock().
    • Add vitest.setup.ts to configure global mocks and environment settings for Vitest.
    • Remove Jest-specific mock files and configurations.
  • Miscellaneous:
    • Update .dockerignore to remove jest.config* entries.
    • Ensure all test files are renamed from .test.tsx to .spec.tsx to align with Vitest conventions.

This description was created by Ellipsis for 7474cc2. You can customize this summary. It will automatically update as commits are pushed.

@cte cte requested review from jr and mrubens as code owners June 17, 2025 06:15
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Jun 17, 2025
@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Jun 17, 2025
@daniel-lxs daniel-lxs moved this from Triage to PR [Draft / In Progress] in Roo Code Roadmap Jun 17, 2025
@hannesrudolph hannesrudolph added PR - Draft / In Progress and removed Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. labels Jun 17, 2025
@daniel-lxs daniel-lxs marked this pull request as draft June 17, 2025 14:17
@cte cte marked this pull request as ready for review June 17, 2025 22:06
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Jun 17, 2025
@cte cte changed the title Start converting webview-ui tests to vitest Converting all webview-ui tests to vitest Jun 17, 2025
@cte cte changed the title Converting all webview-ui tests to vitest Convert all webview-ui tests: jest -> vitest Jun 17, 2025
@cte cte merged commit 44f3a84 into main Jun 17, 2025
10 checks passed
@cte cte deleted the cte/webview-ui-jest-to-vitest branch June 17, 2025 22:29
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jun 17, 2025
@github-project-automation github-project-automation bot moved this from PR [Draft / In Progress] to Done in Roo Code Roadmap Jun 17, 2025
valekseev pushed a commit to valekseev/Roo-Code that referenced this pull request Jun 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR - Draft / In Progress size:XL This PR changes 500-999 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants