-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
add useImport tests #1026
add useImport tests #1026
Conversation
Hello from PR HelperIs your PR ready for review and processing? Mark the PR ready by including If you still have work to do, even after marking this ready. Put the PR on hold by including |
Jest Coverage
|
file: file as unknown as UploadFile, | ||
}); | ||
|
||
setTimeout(() => { |
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.
setTimeout'lar yerine testing-library'nin await'i gibi şeyler kullanılamaz mı?
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.
hayır, kullanılamıyor. daha önce belirttiğim gibi, useImport
işlemin bittiğine dair bir kontrol mekanizması vermiyor dışarı, bence bunu yapmalı. fakat şu an yapmıyor, yani mümkün değil. mutationResult
da burada işe yaramıyor
Test me! 'MASTER'
Link to USE-IMPORT-TESTS
Test me! 'MASTER'
Link to USE-IMPORT-TESTS
100% coverage
closes #988