Codecov Report

❌ Patch coverage is 38.23529% with 21 lines in your changes missing coverage. Please review.
✅ Project coverage is 45.96%. Comparing base (656c090) to head (4cdfc20).

@@            Coverage Diff             @@
##             main    #2595      +/-   ##
==========================================
+ Coverage   45.54%   45.96%   +0.42%     
==========================================
  Files         771      771              
  Lines       43636    43649      +13     
  Branches    11041    11229     +188     
==========================================
+ Hits        19872    20065     +193     
+ Misses      23748    23539     -209     
- Partials       16       45      +29     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Tested w/ ACL editor (which was the reason the check needed to be added in _gotReactComponent). It is a functional component w/ forwardRef which triggers the logic. It's actually not an issue any more (only was w/ my initial iteration which patched the state into the same object as the class component persisted state). But the check and types are more accurate so I'm leaving them. The e2e was failing because the check was looking for componentWillUpdate to exist. There are other example utils that check component.isReactComponent, but the types didn't like that narrowing at all and instanceof Component seems to work.