-
Notifications
You must be signed in to change notification settings - Fork 54
Conversation
Hi @sadickisaac I see you are removing the shallowRendering functionality. We currently have an on-going issue to make use of the Do you know if |
|
possibly major refactors to our Simorgh tests 👀(but I suppose it is inevitable since we want to move away from shallowRendering) I'm going to put my running issue on blocked until you get this one done: bbc/simorgh#3264 👍 |
…into remove-react-test-renderer
Co-Authored-By: Khoa Phan <30599794+thekp@users.noreply.github.com>
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.
This is ok for psammead, as we don't use shallowRender anywhere here.
This issue is for using test-helper in Simorgh: bbc/simorgh#3264
Contributes to #774
Overall change: Remove
react-test-renderer
and shallow rendering forpsammead-test-helpers
Code changes:
shallowRender
andshouldShallowMatchSnapshot
.isNull
andshouldMatchSnapshot
render
from@testing-library/react