-
Notifications
You must be signed in to change notification settings - Fork 464
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(Alert): add onClosed
test
#3756
base: develop
Are you sure you want to change the base?
Conversation
arr.push('fn2'); | ||
}; | ||
const wrapper = mount(() => ( | ||
<Alert close onClose={fn1()} onClosed={fn2()}> |
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.
对应的是mock事件 不是执行事件
测试这种事件API有没有执行 用BeCalled expect 数组有没有和它执行没关系吧
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.
主要是想测试他的先后执行顺序,根据数组数据的先后判断。
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.
by accident..
只是因为你在mount里写的顺序 fn() 是直接执行这个函数 不是等回调执行了来执行
刚好fn1被你写在fn2之前了 所以也就fn1先执行了 你试试把这两个换顺序是不是直接挂了..
<Alert close onClosed={fn2()} onClose={fn1()}>
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.
<Alert close onClose={fn1()} onClosed={fn2()}> | |
<Alert close onClose={fn1} onClosed={fn2}> |
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.
好,我试一下
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.
主要是想测试他的先后执行顺序,根据数组数据的先后判断。
这个思路是可以的 就是具体内容得调整下
🤔 这个 PR 的性质是?
🔗 相关 Issue
💡 需求背景和解决方案
📝 更新日志
fix(组件名称): 处理问题或特性描述 ...
本条 PR 不需要纳入 Changelog
☑️ 请求合并前的自查清单