Ensure onToggle is called when on is controlled but setOnState is called

src/__tests__/index.js

@@ -159,13 +159,25 @@ test('onToggle gets called in controlled prop scenario', () => {
   expect(spy).not.toHaveBeenCalled()
   wrapper.setProps({on: true})
   expect(spy).toHaveBeenCalled()
+  expect(spy.mock.calls.length).toBe(1)
 })
 
 test('onToggle gets called with fresh state in controlled prop scenario', () => {
   const spy = jest.fn()
   const {wrapper} = setup({on: false, onToggle: spy})
   wrapper.setProps({on: true})
   expect(spy).toHaveBeenLastCalledWith(true, expect.anything())
+  expect(spy.mock.calls.length).toBe(1)
+})
+
+test('onToggle gets called when setOnState is called in controlled prop scenario', () => {
+  const spy = jest.fn()
+  const {getTogglerProps} = setup({on: false, onToggle: spy})
+  const {onClick} = getTogglerProps()
+  const fakeEvent = {target: null}
+  onClick(fakeEvent)
+  expect(spy).toHaveBeenLastCalledWith(true, expect.anything())
+  expect(spy.mock.calls.length).toBe(1)
 })
 
 function setup({children = () => <div />, ...props} = {}) {

src/index.js

@@ -69,6 +69,9 @@ class Toggle extends Component {
   }
 
   setOnState = (state = !this.getOn()) => {
+    if (this.getOn() !== state) {
+      this.props.onToggle(state, this.getTogglerStateAndHelpers())
+    }
     this.setState({on: state})
   }