Exclude forward ref when using deep mount

[mmarcuccio]

When using deep mount on a component wrapped with a forwardRef, currently enzyme-to-json includes the forwardRef node in the JSON.

Running the test included here without the added check for forwardRef results in the following snapshot:

<ForwardRef
  falsyValue={false}
  value="hi mum"
>
  <div
    className="basic-class undefined"
    onClick={[Function]}
  >
    <div
      className="group"
      id="group-id"
    >
      <span />
      <span
        className="empty"
      />
    </div>
  </div>
</ForwardRef>

Since the deep mode should only include DOM nodes, this PR adds a check to exclude the forwardRef node but include the output of the wrapped component.

[codecov]

Codecov Report

Merging #167 into master will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #167   +/-   ##
=======================================
  Coverage   97.19%   97.19%           
=======================================
  Files           6        6           
  Lines         107      107           
  Branches       41       41           
=======================================
  Hits          104      104           
  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 fd3b939...356c30c. Read the comment docs.

[mmarcuccio]

Hi @adriantoine. Just a friendly ping - I was wondering if you had a chance to review this.

[VincentLanglet]

Hi @adriantoine. Just a friendly ping - I was wondering if you had a chance to review this.

If not, ping me, I'll do it.

My only concern is that this PR will break existing snapshot...

[mmarcuccio]

Hey @VincentLanglet, just following up on this. Yes, this PR would definitely break existing snapshots. It would be limited to those using mount with deep mode on components using forward refs. However, from reading the documentation, this new behavior is the expected behavior of deep mode.

Would it be worth making an option to opt out of this change via configuration?

[VincentLanglet]

However, from reading the documentation, this new behavior is the expected behavior of deep mode.

Sorry for the delay.
You convince me.

[VincentLanglet]

@mmarcuccio 3.5.0 is released.