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

test: 新增组件测试用例 #280

Merged
merged 16 commits into from
Sep 29, 2022
Merged

Conversation

LadyChatterleyLover
Copy link
Contributor

@LadyChatterleyLover LadyChatterleyLover commented Aug 16, 2022

📝 更新日志

  • feat(Result): 新增单元测试
  • feat(Divider): 新增单元测试
  • feat(Grid): 新增单元测试
  • feat(Image): 新增单元测试
  • feat(Input): 新增单元测试
  • feat(Rate): 新增单元测试
  • feat(Radio): 新增单元测试
  • feat(Skeleton): 新增单元测试
  • feat(Cell): 新增单元测试
  • feat(Badge): 新增单元测试
  • feat(Avatar): 新增单元测试
  • feat(Tag): 新增单元测试
  • feat(Button): 新增单元测试
  • feat(Textarea): 新增单元测试

@anlyyao
Copy link
Collaborator

anlyyao commented Aug 16, 2022

@LadyChatterleyLover 我们是一个组件对应一 个 index.test.jsx,不需要用组件名命名,avatar和avatar-group可以合并到一起,用不同的 describe 区分即可~ 辛苦在修改一波

@anlyyao anlyyao mentioned this pull request Aug 17, 2022
50 tasks
@github-actions
Copy link
Contributor

github-actions bot commented Aug 19, 2022

完成

site/vite.config.js Outdated Show resolved Hide resolved
src/search/__test__/index.test.jsx Outdated Show resolved Hide resolved
src/image/__test__/index.test.jsx Outdated Show resolved Hide resolved
src/image/__test__/index.test.jsx Outdated Show resolved Hide resolved
src/search/search.vue Outdated Show resolved Hide resolved
src/navbar/navbar.vue Outdated Show resolved Hide resolved
@anlyyao
Copy link
Collaborator

anlyyao commented Sep 8, 2022

@LadyChatterleyLover 辛苦rebase develop分支,解决下冲突~ 然后补充一下相应组件的徽标

@anlyyao
Copy link
Collaborator

anlyyao commented Sep 15, 2022

@LadyChatterleyLover 辛苦继续跟进一下哦~ 或者也可以按组件拆分成多个pr, test-coverage.js文件避免冲突可以不提交

@anlyyao anlyyao merged commit 757c5ee into Tencent:develop Sep 29, 2022
@github-actions github-actions bot mentioned this pull request Oct 10, 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