search

adriantoine / enzyme-to-json

Snapshot test your Enzyme wrappers
MIT License
947 stars 64 forks source link

handle deep mounted memo components properly #171

Closed nathanmarks closed 4 years ago

nathanmarks commented 4 years ago

Similar to this PR: https://github.com/adriantoine/enzyme-to-json/pull/167

We're not currently handling .memo components with deep mount serialization the same way we handle other components.

This addresses that. This results in no diff for deep snapshots when wrapping components in memo.

codecov[bot] commented 4 years ago

Codecov Report

Merging #171 into master will increase coverage by 0.05%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #171      +/-   ##
==========================================
+ Coverage   97.19%   97.24%   +0.05%     
==========================================
  Files           6        6              
  Lines         107      109       +2     
  Branches       41       42       +1     
==========================================
+ Hits          104      106       +2     
  Misses          3        3
Impacted Files Coverage Δ
src/mount.js 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update a4f83d3...33a39f0. Read the comment docs.

VincentLanglet commented 4 years ago

Thanks !

VincentLanglet commented 4 years ago

Released as 3.6.0