[fixed] OverlayTrigger passes onClick prop to child

src/OverlayTrigger.js

@@ -90,14 +90,16 @@ const OverlayTrigger = React.createClass({
  },
 
  render() {
+ let child = React.Children.only(this.props.children);
  if (this.props.trigger === 'manual') {
- return React.Children.only(this.props.children);
+ return child;
  }
 
  let props = {};
 
+ props.onClick = createChainedFunction(child.props.onClick, this.props.onClick);
  if (isOneOf('click', this.props.trigger)) {
- props.onClick = createChainedFunction(this.toggle, this.props.onClick);
+ props.onClick = createChainedFunction(this.toggle, props.onClick);
  }
 
  if (isOneOf('hover', this.props.trigger)) {
@@ -111,7 +113,7 @@ const OverlayTrigger = React.createClass({
  }
 
  return cloneElement(
- React.Children.only(this.props.children),
+ child,
  props
  );
  },

test/OverlayTriggerSpec.js

@@ -0,0 +1,30 @@
+import React from 'react';
+import ReactTestUtils from 'react/lib/ReactTestUtils';
+import OverlayTrigger from '../src/OverlayTrigger';
+
+describe('OverlayTrigger', function() {
+ it('Should create OverlayTrigger element', function() {
+ let instance = ReactTestUtils.renderIntoDocument(
+ <OverlayTrigger overlay={<div>test</div>}>
+ <button>button</button>
+ </OverlayTrigger>
+ );
+ let overlayTrigger = instance.getDOMNode();
+ assert.equal(overlayTrigger.nodeName, 'BUTTON');
+ });
+
+ it('Should pass OverlayTrigger onClick prop to child', function() {
+ let called = false;
+ let callback = function() {
+ called = true;
+ };
+ let instance = ReactTestUtils.renderIntoDocument(
+ <OverlayTrigger overlay={<div>test</div>} onClick={callback}>
+ <button>button</button>
+ </OverlayTrigger>
+ );
+ let overlayTrigger = instance.getDOMNode();
+ ReactTestUtils.Simulate.click(overlayTrigger);
+ assert.equal(called, true);
+ });
+});