fix: Add default onAction prop (#382)

* fix: Add default onAction prop

PropTypes define onAction property as optional, however in class its
used unconditionally.

* fix: Add console error spy check

Co-authored-by: Hrusikesh Panda <[email protected]>

Author: kYem

__snapshots__/src/index.test.js.md

@@ -459,6 +459,7 @@ Generated by [AVA](https://ava.li).
       id="rdts"
       inlineSearchInput={false}
       mode="radioSelect"
+      onAction={Function onAction {}}
       onBlur={Function onBlur {}}
       onChange={Function onChange {}}
       onFocus={Function onFocus {}}
@@ -609,6 +610,7 @@ Generated by [AVA](https://ava.li).
       }
       id="rdts"
       inlineSearchInput={false}
+      onAction={Function onAction {}}
       onBlur={Function onBlur {}}
       onChange={Function onChange {}}
       onFocus={Function onFocus {}}

__snapshots__/src/index.test.js.snap

@@ -50,6 +50,7 @@ class DropdownTreeSelect extends Component {
   }
 
   static defaultProps = {
+    onAction: () => {},
     onFocus: () => {},
     onBlur: () => {},
     onChange: () => {},

src/index.js

@@ -107,6 +107,15 @@ test('notifies on action', t => {
   t.true(handler.calledWithExactly(node0, action))
 })
 
+test('notifies on action without onAction handler', t => {
+  const handler = spy(console, 'error')
+  const { tree } = t.context
+  const wrapper = mount(<DropdownTreeSelect id={dropdownId} data={tree} showDropdown="initial" />)
+  wrapper.find('i.fa-ban').simulate('click')
+  t.true(handler.notCalled)
+  handler.restore()
+})
+
 test('notifies on node toggle', t => {
   const handler = spy()
   const { tree } = t.context