Merge 4d7a6a0 into ee752c6

dowjones · Jul 25, 2020 · 8e01213 · 8e01213
2 parents ee752c6 + 4d7a6a0
commit 8e01213
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 0 deletions.
diff --git a/__snapshots__/src/index.test.js.md b/__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 {}}

diff --git a/__snapshots__/src/index.test.js.snap b/__snapshots__/src/index.test.js.snap
diff --git a/src/index.js b/src/index.js
@@ -50,6 +50,7 @@ class DropdownTreeSelect extends Component {
   }
 
   static defaultProps = {
+    onAction: () => {},
     onFocus: () => {},
     onBlur: () => {},
     onChange: () => {},

diff --git a/src/index.test.js b/src/index.test.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