Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/migrate2vitest #1687

Merged
merged 18 commits into from
Oct 29, 2022
Merged

Feat/migrate2vitest #1687

merged 18 commits into from
Oct 29, 2022

Conversation

xiaosansiji
Copy link
Collaborator

@xiaosansiji xiaosansiji commented Oct 26, 2022

🤔 这个 PR 的性质是?

  • 日常 bug 修复
  • 新特性提交
  • 文档改进
  • 演示代码改进
  • 组件样式/交互改进
  • CI/CD 改进
  • 重构
  • 代码风格优化
  • 测试用例
  • 分支合并
  • 其他

🔗 相关 Issue

💡 需求背景和解决方案

vitest:
image

Vitest 大部分语法跟原先的 Jest 一样,且默认启用了全局 API,你依然可以在在测试文件中直接使用 describe、it 等直接编写用例,需要改写的只有 jest.fn() => vi.fn() 等 mock 方法,及异步回调等,大部分语法可以复用,详情可以参考官方的 迁移指南

📝 更新日志

  • 由 jest 切换到 vitest,与 Vue next/React 仓库一致,便于复用测试用例

  • 优化 Tree 等测试用例,更新语法

  • 使用了 jsx 语法的测试用例后缀由 .js 改为 .jsx,原先是在 jest 中做了兼容编译时讲 js 文件作为 jsx 处理,vitest 中做兼容的配置比较 hack,不做兼容

  • 去除各组件目录下单独的 src/xxcomponent/__tests__/demo.test.js 文件,改为 test/snap/csr.test.js 中统一渲染测试,只保留一份全部组件 demo 的 snapshot 记录

  • 本条 PR 不需要纳入 Changelog

☑️ 请求合并前的自查清单

⚠️ 请自检并全部勾选全部选项⚠️

  • 文档已补充或无须补充
  • 代码演示已提供或无须提供
  • TypeScript 定义已补充或无须补充
  • Changelog 已提供或无须提供

@xiaosansiji xiaosansiji changed the title Feat/migrate2vitest [WIP]Feat/migrate2vitest Oct 26, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Oct 26, 2022

完成

@xiaosansiji xiaosansiji changed the title [WIP]Feat/migrate2vitest Feat/migrate2vitest Oct 29, 2022
@chaishi chaishi merged commit bd39cc6 into develop Oct 29, 2022
@chaishi chaishi deleted the feat/migrate2vitest branch October 29, 2022 10:38
@github-actions github-actions bot mentioned this pull request Nov 2, 2022
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.

2 participants