[@emotion/jest] Pass through includeStyles option in createEnzymeSerializer
#2715
Conversation
🦋 Changeset detectedLatest commit: a3250b0 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit a3250b0:
|
Codecov Report
|
| @@ -66,11 +66,13 @@ const wrappedEnzymeSerializer = { | |||
|
|
|||
| export function createEnzymeSerializer({ | |||
| classNameReplacer, | |||
| DOMElements = true | |||
| DOMElements = true, | |||
| includeStyles = true | |||
There was a problem hiding this comment.
We might also need to add this in the .d.ts file
There was a problem hiding this comment.
createEnzymeSerializer uses the same CreateSerializerOptions interface as createSerializer (updated with includeStyles in #2709). In enzyme.d.ts, imported from index.d.ts. Or are you referring to a different d.ts file?
There was a problem hiding this comment.
No, this is fine. I was reviewing from mobile and I've assumed that there is another type for this one.
What: Add the missing
includeStylesoption tocreateEnzymeSerializerWhy: The option was added to
createSerializerin #2709, but was not added tocreateEnzymeSerializerHow: The option is accepted and passed through to
createEmotionSerializer