Shallow renderer does not support React.memo

[ljharb]

Do you want to request a feature or report a bug?
bug

What is the current behavior?
In normal rendering, you can memoize a class-based component: https://jsfiddle.net/586ea3cx/

With the shallow renderer, it seems like you can't: https://jsfiddle.net/odj217Lv/1/

This might be blocking enzymejs/enzyme#1914, in which tests are failing with Cannot call a class as a function. (it's tough to repro stuff with the shallow renderer)

[Yurickh]

I would like to point out that React.memo being used with classes is not documented behaviour and might not be part of React's public facing API.

You can check that all references to React.memo in the docs say it is meant to be used with functional components here and here.

[ljharb]

Indeed; but it doesn’t throw when you try, and it seems to work fine in actual React - I’d expect it to work, or not work, the same everywhere.

[gaearon]

memo() can be used with any component type. Happy to take a PR that adds support for it!

[aweary]

This doesn't seem to be isolated to class components, it fails with any component using memo(): https://jsfiddle.net/f0j1e5tw/6/

[ljharb]

The relevant tests in the open enzyme PR only fail for class components, fwiw.

[gaearon]

Fixed in 16.8.5.