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.

Sign up for GitHub

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

Jump to bottom

refactor: migrate to vitest #723

Merged
merged 19 commits into from
May 14, 2022
Merged

refactor: migrate to vitest #723

merged 19 commits into from
May 14, 2022

Conversation

PengYYYYY
Copy link
Collaborator

@PengYYYYY PengYYYYY commented May 7, 2022 •
edited
Loading

🤔 这个 PR 的性质是？

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

🔗 相关 Issue

💡 需求背景和解决方案

🚀 单测重构
使用vitest重构单元测试，提升单元测试速度

before

image

after

image

整合ssr和csr组件测试，去除test:demoscript，将批量组件render的单元测试整合。
后续输出单元测试规范文档

🐛 去除一些冗余代码

📝 更新日志

  • refacot(test): 使用vitest 对单元测试进行重构
  • fix(table): 修复在ssr环境中的兼容
  • fix(transfer): 修复对 pagination 组件的引用报错

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

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

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

@PengYYYYY PengYYYYY changed the title [WIP]feat: add vitest [WIP]test: migrate to vitest May 7, 2022
@github-actions
Copy link
Contributor

github-actions bot commented May 7, 2022 •
edited
Loading

完成

@github-actions
Copy link
Contributor

github-actions bot commented May 7, 2022

Size Change: 0 B

Total Size: 657 kB

ℹ️ View Unchanged
Filename Size
./dist/tdesign.css 43.3 kB
./dist/tdesign.js 367 kB
./dist/tdesign.min.css 38.4 kB
./dist/tdesign.min.js 209 kB

1 similar comment
@github-actions
Copy link
Contributor

github-actions bot commented May 7, 2022

Size Change: 0 B

Total Size: 657 kB

ℹ️ View Unchanged
Filename Size
./dist/tdesign.css 43.3 kB
./dist/tdesign.js 367 kB
./dist/tdesign.min.css 38.4 kB
./dist/tdesign.min.js 209 kB

@PengYYYYY PengYYYYY closed this May 7, 2022
@PengYYYYY PengYYYYY deleted the feat/vitest branch May 7, 2022 15:16
@PengYYYYY PengYYYYY restored the feat/vitest branch May 7, 2022 15:16
@PengYYYYY PengYYYYY reopened this May 7, 2022
@PengYYYYY PengYYYYY mentioned this pull request May 8, 2022
Closed
@github-actions
Copy link
Contributor

Size Change: 0 B

Total Size: 665 kB

ℹ️ View Unchanged
Filename Size
./dist/tdesign.css 44.7 kB
./dist/tdesign.js 370 kB
./dist/tdesign.min.css 39.7 kB
./dist/tdesign.min.js 211 kB

@github-actions
Copy link
Contributor

Size Change: +93 B (0%)

Total Size: 666 kB

Filename Size Change
./dist/tdesign.js 370 kB +64 B (0%)
./dist/tdesign.min.js 211 kB +29 B (0%)
ℹ️ View Unchanged
Filename Size
./dist/tdesign.css 44.7 kB
./dist/tdesign.min.css 39.7 kB

7 similar comments
@github-actions
Copy link
Contributor

Size Change: +93 B (0%)

Total Size: 666 kB

Filename Size Change
./dist/tdesign.js 370 kB +64 B (0%)
./dist/tdesign.min.js 211 kB +29 B (0%)
ℹ️ View Unchanged
Filename Size
./dist/tdesign.css 44.7 kB
./dist/tdesign.min.css 39.7 kB

@github-actions
Copy link
Contributor

Size Change: +93 B (0%)

Total Size: 666 kB

Filename Size Change
./dist/tdesign.js 370 kB +64 B (0%)
./dist/tdesign.min.js 211 kB +29 B (0%)
ℹ️ View Unchanged
Filename Size
./dist/tdesign.css 44.7 kB
./dist/tdesign.min.css 39.7 kB

@github-actions
Copy link
Contributor

Size Change: +93 B (0%)

Total Size: 666 kB

Filename Size Change
./dist/tdesign.js 370 kB +64 B (0%)
./dist/tdesign.min.js 211 kB +29 B (0%)
ℹ️ View Unchanged
Filename Size
./dist/tdesign.css 44.7 kB
./dist/tdesign.min.css 39.7 kB

@github-actions
Copy link
Contributor

Size Change: +93 B (0%)

Total Size: 666 kB

Filename Size Change
./dist/tdesign.js 370 kB +64 B (0%)
./dist/tdesign.min.js 211 kB +29 B (0%)
ℹ️ View Unchanged
Filename Size
./dist/tdesign.css 44.7 kB
./dist/tdesign.min.css 39.7 kB

@github-actions
Copy link
Contributor

Size Change: +93 B (0%)

Total Size: 666 kB

Filename Size Change
./dist/tdesign.js 370 kB +64 B (0%)
./dist/tdesign.min.js 211 kB +29 B (0%)
ℹ️ View Unchanged
Filename Size
./dist/tdesign.css 44.7 kB
./dist/tdesign.min.css 39.7 kB

@github-actions
Copy link
Contributor

Size Change: +93 B (0%)

Total Size: 666 kB

Filename Size Change
./dist/tdesign.js 370 kB +64 B (0%)
./dist/tdesign.min.js 211 kB +29 B (0%)
ℹ️ View Unchanged
Filename Size
./dist/tdesign.css 44.7 kB
./dist/tdesign.min.css 39.7 kB

@github-actions
Copy link
Contributor

Size Change: +93 B (0%)

Total Size: 666 kB

Filename Size Change
./dist/tdesign.js 370 kB +64 B (0%)
./dist/tdesign.min.js 211 kB +29 B (0%)
ℹ️ View Unchanged
Filename Size
./dist/tdesign.css 44.7 kB
./dist/tdesign.min.css 39.7 kB

@PengYYYYY PengYYYYY changed the title [WIP]test: migrate to vitest refactor: migrate to vitest May 12, 2022
@github-actions
Copy link
Contributor

Size Change: +93 B (0%)

Total Size: 666 kB

Filename Size Change
./dist/tdesign.js 370 kB +64 B (0%)
./dist/tdesign.min.js 211 kB +29 B (0%)
ℹ️ View Unchanged
Filename Size
./dist/tdesign.css 44.7 kB
./dist/tdesign.min.css 39.7 kB

@github-actions
Copy link
Contributor

Size Change: -211 B (0%)

Total Size: 665 kB

Filename Size Change
./dist/tdesign.js 370 kB -64 B (0%)
./dist/tdesign.min.js 211 kB -147 B (0%)
ℹ️ View Unchanged
Filename Size
./dist/tdesign.css 44.7 kB
./dist/tdesign.min.css 39.7 kB

@github-actions
Copy link
Contributor

Size Change: +86 B (0%)

Total Size: 666 kB

Filename Size Change
./dist/tdesign.js 370 kB +59 B (0%)
./dist/tdesign.min.js 211 kB +27 B (0%)
ℹ️ View Unchanged
Filename Size
./dist/tdesign.css 45.3 kB
./dist/tdesign.min.css 40.2 kB

1 similar comment
@github-actions
Copy link
Contributor

Size Change: +86 B (0%)

Total Size: 666 kB

Filename Size Change
./dist/tdesign.js 370 kB +59 B (0%)
./dist/tdesign.min.js 211 kB +27 B (0%)
ℹ️ View Unchanged
Filename Size
./dist/tdesign.css 45.3 kB
./dist/tdesign.min.css 40.2 kB

@github-actions
Copy link
Contributor

Size Change: +86 B (0%)

Total Size: 666 kB

Filename Size Change
./dist/tdesign.js 370 kB +59 B (0%)
./dist/tdesign.min.js 211 kB +27 B (0%)
ℹ️ View Unchanged
Filename Size
./dist/tdesign.css 45.3 kB
./dist/tdesign.min.css 40.2 kB

@PengYYYYY PengYYYYY merged commit 63bb185 into develop May 14, 2022
@PengYYYYY PengYYYYY deleted the feat/vitest branch May 14, 2022 16:00
@github-actions github-actions bot mentioned this pull request May 20, 2022
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chaishi chaishi chaishi left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@PengYYYYY @chaishi