react-with-direction icon indicating copy to clipboard operation
react-with-direction copied to clipboard

Updates DirectionProvider to use componentDidUpdate instead of UNSAFE_componentWillReceiveProps

Open dclowd9901 opened this issue 6 years ago • 0 comments

This change migrates DirectionProvider to cease using UNSAFE_componentWillMount in favor of componentDidUpdate in accordance with

If you need to perform a side effect (for example, data fetching or an animation) in response to a change in props, use componentDidUpdate lifecycle instead. from: https://reactjs.org/docs/react-component.html

Unfortunately, since Enzyme lacks support for shallow mounting that fires componentDidUpdate, there is also a tangential change that includes jsdom as the environment that DirectionProvider's test is run in. And further, because this causes a warning to appear in the test regarding requestAnimationFrame, I used a method found in this thread to quell the warning: https://github.com/facebook/jest/issues/4545

One last thing to note: Some of the testing here was switched to mount in favor of shallow. This was to accommodate an ongoing issue with Enzyme being unable to trigger a call to cDU with setProps being called on a shallow wrapper. https://github.com/airbnb/enzyme/issues/1452

dclowd9901 avatar Jun 11 '18 22:06 dclowd9901