Issue with multiple children about react-tree-walker HOT 8 CLOSED

@ctrlplusb The issue happens when you have multiple react components that are siblings, so if you updated your tree to be:

const tree = (
      <OnlyChildren something={1}>
        <OnlyChildren something={2}>
          <div>Hello world!</div>
        </OnlyChildren>
        <OnlyChildren something={3}>
          <div>Hello world!</div>
        </OnlyChildren>
      </OnlyChildren>
    )

you would expect actual to be [1, 2, 3], when in fact the children are never visited, resulting in actual being only [1]. See screenshot:

from react-tree-walker.

Hey @grxy

Awesome, thanks so much for picking this up. I'll try review and merge within the next 24 hours. 👍

from react-tree-walker.

@ctrlplusb No prob, dude. I actually noticed the issue when writing unit tests for my code while using this.

P.S. Great work breaking it out from react-apollo!

from react-tree-walker.

Hi @grxy 👋

Apologies for the late one on this.

I have struggled to recreate your initial issue. I have added another branch containing a test that I was initially expecting to fail as it doesn't have your fix applied. The test passes though.

Do you mind having a look to see if I have configured the test correctly?

Thanks!

from react-tree-walker.

Perfect thanks!

from react-tree-walker.

@ctrlplusb Any chance you'll be able to merge my PR and publish 2.1.2 soon?

from react-tree-walker.

All done and published. Thanks for being persistent with me!

from react-tree-walker.

@ctrlplusb Great! Thank you!

from react-tree-walker.