Ensures onClick cb fires on non-collapsible tree (#25)

* Fix for #24

* Added unit test for #24

Author: justinsisley

src/Tree/index.js

@@ -185,14 +185,17 @@ export default class Tree extends React.Component {
    * @return {void}
    */
   handleNodeToggle(nodeId) {
+    const data = clone(this.state.data);
+    const matches = this.findNodesById(nodeId, data, []);
+    const targetNode = matches[0];
+
     if (this.props.collapsible) {
-      const data = clone(this.state.data);
-      const matches = this.findNodesById(nodeId, data, []);
-      const targetNode = matches[0];
       targetNode._collapsed
         ? this.expandNode(targetNode)
         : this.collapseNode(targetNode);
       this.setState({ data }, () => this.handleOnClickCb(targetNode));
+    } else {
+      this.handleOnClickCb(targetNode);
     }
   }
 

src/Tree/tests/index.test.js

@@ -211,6 +211,22 @@ describe('<Tree />', () => {
   });
 
 
+  it('calls the onClick callback when `props.collapsible` is false', () => {
+    const onClickSpy = jest.fn();
+    const renderedComponent = mount(
+      <Tree
+        data={mockData}
+        collapsible={false}
+        onClick={onClickSpy}
+      />
+    );
+
+    renderedComponent.find(Node).first().simulate('click');
+
+    expect(onClickSpy).toHaveBeenCalledTimes(1);
+  });
+
+
   it('clones the clicked node\'s data & passes it to the onClick callback if defined', () => {
     const onClickSpy = jest.fn();
     const renderedComponent = mount(