-
Notifications
You must be signed in to change notification settings - Fork 4
Humanatlas test coverage #2017
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
base: develop
Are you sure you want to change the base?
Humanatlas test coverage #2017
Conversation
|
View your CI Pipeline Execution ↗ for commit bcbedcb
☁️ Nx Cloud last updated this comment at |
🚀 Preview Deploy Report✅ Successfully deployed preview here |
axdanbol
left a comment
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.
Must use https://testing-library.com/ instead of fixture in all tests
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.
Please don't use container.querySelector from the render result - that is no better that using fixture. Use queries such as screen.findBy*, screen.getBy* from the testing library instead. Examples: https://testing-library.com/docs/angular-testing-library/examples
No description provided.