Support Suspense

[maclockard]

Current behavior

I'm currently getting the following error using a Suspense component in my tests.

Enzyme Internal Error: unknown node with tag 13

Looking at the current react work tags, this is for the suspense component: https://github.com/facebook/react/blob/v16.6.0/packages/shared/ReactWorkTags.js#L44

Looking at the current implementation of detectFiberTag it doesn't look for Suspense components: https://github.com/airbnb/enzyme/blob/enzyme%403.7.0/packages/enzyme-adapter-react-16/src/detectFiberTags.js#L42

I believe the exception is coming here: https://github.com/airbnb/enzyme/blob/enzyme%403.7.0/packages/enzyme-adapter-react-16/src/ReactSixteenAdapter.js#L103 after it hits the default for the switch.

Expected behavior

Can handle suspense components

API

• shallow
• mount
• render

Version

library	version
enzyme	3.7.0
react	16.6.3
react-dom	16.6.3
react-test-renderer	16.6.3
adapter (below)	1.7.0

Adapter

• enzyme-adapter-react-16
• enzyme-adapter-react-16.3
• enzyme-adapter-react-16.2
• enzyme-adapter-react-16.1
• enzyme-adapter-react-15
• enzyme-adapter-react-15.4
• enzyme-adapter-react-14
• enzyme-adapter-react-13
• enzyme-adapter-react-helper
• others ( )

[ljharb]

Indeed, Suspense is not yet supported.

[missbruni]

Hey, any idea when Suspense will be supported ?

[rodoabad]

@maclockard are you also using React.lazy?

[maclockard]

No, currently just Suspense. Will probably start using lazy as well soon though, but its lower priority to me.

[KrisB1022]

Any update on this? I'm using Suspense with lazy. Just converted over to using, and getting this error. Wasn't sure it there was something being looked at. Thanks in advance.

[anupammaurya]

no luck found.. on this simulate event handling....

[chenesan]

Hello @ljharb I'd like to work on this to make React.Suspense and React.lazy work. Will look into this in the next few days. Since these featrues only works after 16.6, we also have to copy current enzyme-adapter-react-16 as enzyme-adapter-react-16.5 package right?

[ljharb]

@chenesan for now, just implement it in the 16 adapter; we can iterate in that in the PR.