React 36 - nextProps issue about fcc-react-tests-module

[u-rogel]

I am new here, so not sure if I should send a PR for that or you wanna fix it on your own since it's a tiny fix.

[bonham000]

@cowCrazy thanks for bringing this up, I think your suggestion is a good improvement. Feel free to match a PR for this change. I think it will improve the test — however, many of the tests are not completely foolproof. For instance, this suggestion could still be broken as follows:

componentWillReceiveProps(nextProps) {
   console.log(this.props);
   // nextProps;
}

Feel free to just implement and PR the solution you suggested, however. Thanks for bringing this up!

[u-rogel]

@bonham000 I also noticed it is not completely foolproof, if you want I can try and rise the foolproof level with something like that:

1. For nextProps as a function parameter
2. For nextProps in a console.log call

[bonham000]

@cowCrazy yes — sorry for the delay — feel free to improve the test here, but don't feel obliged.

[bonham000]

Resolved in above-mentioned PR!