enzymejs/enzyme

Why is `.wrap()` not included in the documentation?

Open

#2,249 建立於 2019年10月1日

在 GitHub 查看
 (15 留言) (0 反應) (0 負責人)JavaScript (19,979 star) (2,016 fork)batch import
feature requesthelp wanted

描述

Sorry for not following the template, but it is a simple request.

Why is .wrap() not included in the documentation?

As you can see here #919 at least more 6 people found this method useful. Just for completion it should be in the docs. Even if a note is included against the use of it and a suggestion of how to better write the component.

The author suggested to write the PR, I put myself available to do the same.

My use case

My use case is a HOC <Flow> that receives the content component <Step> as a prop. The component <Container>, which renders this HOC, renders the <Step> conditionally based on the current step of a specific flow. I'm using useMemo() to render this component inside the <Container> and that's why I need .wrap() to wrap the <Step> passed as a prop to <Flow>.

I could use a separate function to conditionally render and return <Step>, but this would obscure the code and have the same behavior: this function would be wrapped in a useMemo() and passed as a prop to <Flow>.

貢獻者指南

Why is `.wrap()` not included in the documentation? · enzymejs/enzyme#2249 | Good First Issue