Allows state.data to update if props.data reference changes (#59)

Author: bkrem

src/Tree/index.js

@@ -55,7 +55,7 @@ export default class Tree extends React.Component {
 
   componentWillReceiveProps(nextProps) {
     // Clone new data & assign internal properties
-    if (!deepEqual(this.props.data, nextProps.data)) {
+    if (this.props.data !== nextProps.data) {
       this.setState({
         data: this.assignInternalProperties(clone(nextProps.data)),
       });

src/Tree/tests/index.test.js

@@ -79,18 +79,31 @@ describe('<Tree />', () => {
       data: mockData2,
     };
     const renderedComponent = mount(<Tree data={mockData} />);
-
     expect(renderedComponent.instance().assignInternalProperties).toHaveBeenCalledTimes(
       mockDataDepth,
     );
-
     renderedComponent.setProps(nextProps);
-
     expect(renderedComponent.instance().assignInternalProperties).toHaveBeenCalledTimes(
       mockDataDepth + mockData2Depth,
     );
   });
 
+  it("reassigns internal props if `props.data`'s array reference changes", () => {
+    // `assignInternalProperties` recurses by depth: 1 level -> 1 call
+    const mockDataDepth = 2;
+    const nextDataDepth = 2;
+    const nextData = [...mockData];
+    nextData[0].children.push({ name: `${nextData[0].children.length}` });
+    const renderedComponent = mount(<Tree data={mockData} />);
+    expect(renderedComponent.instance().assignInternalProperties).toHaveBeenCalledTimes(
+      mockDataDepth,
+    );
+    renderedComponent.setProps({ data: nextData });
+    expect(renderedComponent.instance().assignInternalProperties).toHaveBeenCalledTimes(
+      mockDataDepth + nextDataDepth,
+    );
+  });
+
   describe('translate', () => {
     it('applies the `translate` prop when specified', () => {
       const fixture = { x: 123, y: 321 };