Fixes non-persistent event when accessing evt downstream (#168)

Author: bkrem

src/Tree/index.js

@@ -259,6 +259,8 @@ export default class Tree extends React.Component {
     const data = clone(this.state.data);
     const matches = this.findNodesById(nodeId, data, []);
     const targetNode = matches[0];
+    // Persist the SyntheticEvent for downstream handling by users.
+    evt.persist();
 
     if (this.props.collapsible && !this.state.isTransitioning) {
       if (targetNode._collapsed) {
@@ -307,6 +309,8 @@ export default class Tree extends React.Component {
       const data = clone(this.state.data);
       const matches = this.findNodesById(nodeId, data, []);
       const targetNode = matches[0];
+      // Persist the SyntheticEvent for downstream handling by users.
+      evt.persist();
       onMouseOver(clone(targetNode), evt);
     }
   };
@@ -324,6 +328,8 @@ export default class Tree extends React.Component {
       const data = clone(this.state.data);
       const matches = this.findNodesById(nodeId, data, []);
       const targetNode = matches[0];
+      // Persist the SyntheticEvent for downstream handling by users.
+      evt.persist();
       onMouseOut(clone(targetNode), evt);
     }
   };

src/Tree/tests/index.test.js

@@ -390,6 +390,19 @@ describe('<Tree />', () => {
         expect.objectContaining(mockEvt),
       );
     });
+
+    it('persists the SynthethicEvent for downstream processing', () => {
+      const persistSpy = jest.fn();
+      const mockEvt = { mock: 'event', persist: persistSpy };
+      const renderedComponent = mount(<Tree data={mockData} onClick={() => {}} />);
+
+      renderedComponent
+        .find(Node)
+        .first()
+        .simulate('click', mockEvt);
+
+      expect(persistSpy).toHaveBeenCalledTimes(1);
+    });
   });
 
   describe('onMouseOver', () => {
@@ -423,6 +436,19 @@ describe('<Tree />', () => {
         expect.objectContaining(mockEvt),
       );
     });
+
+    it('persists the SynthethicEvent for downstream processing if handler is defined', () => {
+      const persistSpy = jest.fn();
+      const mockEvt = { mock: 'event', persist: persistSpy };
+      const renderedComponent = mount(<Tree data={mockData} onMouseOver={() => {}} />);
+
+      renderedComponent
+        .find(Node)
+        .first()
+        .simulate('mouseover', mockEvt);
+
+      expect(persistSpy).toHaveBeenCalledTimes(1);
+    });
   });
 
   describe('onMouseOut', () => {
@@ -456,6 +482,19 @@ describe('<Tree />', () => {
         expect.objectContaining(mockEvt),
       );
     });
+
+    it('persists the SynthethicEvent for downstream processing if handler is defined', () => {
+      const persistSpy = jest.fn();
+      const mockEvt = { mock: 'event', persist: persistSpy };
+      const renderedComponent = mount(<Tree data={mockData} onMouseOut={() => {}} />);
+
+      renderedComponent
+        .find(Node)
+        .first()
+        .simulate('mouseout', mockEvt);
+
+      expect(persistSpy).toHaveBeenCalledTimes(1);
+    });
   });
 
   describe('onUpdate', () => {