infernojs / inferno / 7173

100%

master: 93%

push

travis-ci

inferno-clone-vnode doesn't include children in cloned node (#1394)

Fixes #1393.

inferno-clone-vnode does not currently clone existing children of a
vnode. Instead, we are required to specify the children to include.
While this works, it seems redundant and does not behave in the same
way as React.

This patch will allow the children of a node to automatically be cloned if no children
props are specified in the second parameter and no children are
specified in the third parameter. Furthermore, this patch will flatten
the list of children. This will allow all children to be accounted for.
For example:

```
const itemList = [{name: 'a'}, {name: 'b'}];
<Wrapper>
    <StaticItem1 />
    <StaticItem2 />
    {itemList.map(x => <StaticItem name={x.name} />)}
</Wrapper>
```

Signed-off-by: Will Johnson <wjohnson@whamcloud.com>

2300 of 2679 branches covered (85.85%)

15 of 15 new or added lines in 1 file covered. (100.0%)

3517 of 3707 relevant lines covered (94.87%)

1880.63 hits per line