-
-
Notifications
You must be signed in to change notification settings - Fork 46.8k
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: moving to testing-library in Tree
#35674
Conversation
import debounce from 'lodash/debounce'; | ||
import Tree from '../index'; | ||
import React from 'react'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
import 顺序看起来是自动的🤔
Codecov Report
@@ Coverage Diff @@
## next #35674 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 469 469
Lines 8490 8490
Branches 2404 2404
=========================================
Hits 8490 8490 Continue to review full report at Codecov.
|
CI failed in React 18 |
rebased |
Tree
Tree
react 18 会把 css-motion 运行结束后再匹配 snapshot,所以 dom 结构中不含有 motion 的部分。 不确定为什么 17 和 18 会有这个区别,猜测和 我是否可以通过在测试中加入额外的 jest.runAllTimers() 跳过动画,使得 react 17 和 react 18 的测试保持一致。同时,这个应该更符合 test 的初衷(测最终结果而不是动画的中间状态) |
Tree
Tree
[中文版模板 / Chinese template]
🤔 This is a ...
🔗 Related issue link
💡 Background and solution
📝 Changelog
☑️ Self Check before Merge