Merge pull request #346 from jmmorganMSTS/fix-onKeydown

Support Power Select's onKeydown argument

Author: jelhan

addon/components/bs-form/element/control/power-select-multiple.hbs

@@ -31,7 +31,7 @@
     @onClose={{@onClose}}
     @onFocus={{@onFocus}}
     @onInput={{@onInput}}
-    @onKeyDown={{@onKeyDown}}
+    @onKeydown={{@onKeydown}}
     @onOpen={{@onOpen}}
     @options={{@options}}
     @optionsComponent={{@optionsComponent}}

addon/components/bs-form/element/control/power-select.hbs

@@ -31,7 +31,7 @@
     @onClose={{@onClose}}
     @onFocus={{@onFocus}}
     @onInput={{@onInput}}
-    @onKeyDown={{@onKeyDown}}
+    @onKeydown={{@onKeydown}}
     @onOpen={{@onOpen}}
     @options={{@options}}
     @optionsComponent={{@optionsComponent}}

tests/integration/components/bs-form/element/control/power-select-multiple-test.js

@@ -1,6 +1,12 @@
 import { module, test } from 'qunit';
 import { setupRenderingTest } from 'ember-qunit';
-import { render, click, find, findAll } from '@ember/test-helpers';
+import {
+  render,
+  click,
+  find,
+  findAll,
+  triggerKeyEvent,
+} from '@ember/test-helpers';
 import hbs from 'htmlbars-inline-precompile';
 import { clickTrigger } from 'ember-power-select/test-support/helpers';
 
@@ -132,13 +138,20 @@ module(
     });
 
     test('it passes power-select @options', async function (assert) {
+      let lastKeyPress = null;
+      this.set('onKeydown', function (select, event) {
+        lastKeyPress = event.keyCode;
+        return true;
+      });
       await render(hbs`
     <BsForm as |form|>
       <form.element @controlType="power-select-multiple" @property="prop2" @options={{this.options}} @placeholder="something" as |el|>
-        <el.control @searchEnabled={{false}} @triggerClass="form-control" />
+        <el.control @onKeydown={{this.onKeydown}} @searchEnabled={{false}} @triggerClass="form-control" />
       </form.element>
     </BsForm>`);
       assert.dom('.form-control').exists();
+      await triggerKeyEvent('.form-control', 'keydown', 'X');
+      assert.strictEqual(lastKeyPress, 88);
       await clickTrigger();
       assert.dom('.ember-power-select-search-input').doesNotExist();
     });

tests/integration/components/bs-form/element/control/power-select-test.js

@@ -1,6 +1,12 @@
 import { module, test } from 'qunit';
 import { setupRenderingTest } from 'ember-qunit';
-import { render, click, find, findAll } from '@ember/test-helpers';
+import {
+  render,
+  click,
+  find,
+  findAll,
+  triggerKeyEvent,
+} from '@ember/test-helpers';
 import hbs from 'htmlbars-inline-precompile';
 import { clickTrigger } from 'ember-power-select/test-support/helpers';
 
@@ -112,13 +118,20 @@ module(
     });
 
     test('it passes power-select options', async function (assert) {
+      let lastKeyPress = null;
+      this.set('onKeydown', function (select, event) {
+        lastKeyPress = event.keyCode;
+        return true;
+      });
       await render(hbs`
     <BsForm as |form|>
       <form.element @controlType="power-select" @property="prop2" @options={{this.options}} @placeholder="something" as |el|>
-        <el.control @searchEnabled={{false}} @triggerClass="form-control" />
+        <el.control @onKeydown={{this.onKeydown}} @searchEnabled={{false}} @triggerClass="form-control" />
       </form.element>
     </BsForm>`);
       assert.dom('.form-control').exists();
+      await triggerKeyEvent('.form-control', 'keydown', 'X');
+      assert.strictEqual(lastKeyPress, 88);
       await clickTrigger();
       assert.dom('.ember-power-select-search-input').doesNotExist();
     });