Handle lazy indeterminate static properties

[taion]

No description provided.

[theKashey]

No tests 😭
Things like this should be covered by tests, to not to be suddenly broken in the future.

[taion]

@theKashey

I'm back with more "fun" on weird React features used in Relay.

Relay now does this: facebook/relay@0573d77

So... we need to also lazily proxy over getDerivedStateFromProps. Sigh.

[taion]

Sorry, tests forthcoming! Wanted to get the PR up to point people at what was happening.

[codecov-io]

Codecov Report

Merging #1014 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1014      +/-   ##
==========================================
+ Coverage   86.52%   86.54%   +0.01%     
==========================================
  Files          31       31              
  Lines         824      825       +1     
  Branches      193      193              
==========================================
+ Hits          713      714       +1     
  Misses         92       92              
  Partials       19       19

Impacted Files	Coverage Δ
src/proxy/createClassProxy.js	97.24% <100%> (+0.01%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c8a36e7...ba10324. Read the comment docs.

[taion]

Updated the test case to cover this.

[theKashey]

👍

[theKashey]

@taion - ok to merge? Is this the only change you need?

[taion]

Yup, this is the only change. I’m not 100% sure that my use of transferStaticProps there is correct, but it’s probably okay. At least, it fixes the problem I see.

[theKashey]

It sounds to be correct. Anyway - I should not break anything.

[theKashey]

v 4.3.1