A few tests

This commit is contained in:
Josh Johnson 2017-08-02 14:14:08 +01:00
parent 94e3c97b4e
commit 1013f32ce6
2 changed files with 12 additions and 2 deletions

View file

@ -2737,6 +2737,7 @@ class Choices {
const dropdown = this._getTemplate('dropdown');
const placeholder = this.config.placeholder ?
this.config.placeholderValue ||
this.config.searchPlaceholderValue ||
this.passedElement.getAttribute('placeholder') :
false;
@ -2782,7 +2783,7 @@ class Choices {
// If select one element with a search placeholder value
if (this.config.searchPlaceholderValue) {
input.placeholder = this.config.searchPlaceholderValue;
} else {
} else if (this.placeholder) {
const placeholderItem = this._getTemplate('placeholder', this.placeholder);
this.itemList.appendChild(placeholderItem);
}

View file

@ -523,7 +523,7 @@ describe('Choices', () => {
ctrlKey: false
});
const activeOptions = this.choices.currentState.choices.filter(function (choice) {
const activeOptions = this.choices.currentState.choices.filter(function(choice) {
return choice.active;
});
@ -566,6 +566,15 @@ describe('Choices', () => {
expect(this.choices.currentState.choices[0].value).toEqual('Value 1');
});
it('should set searchPlaceholderValue if set', function() {
const dummyPlaceholder = 'Test placeholder';
this.choices = new Choices(this.input, {
searchPlaceholderValue: dummyPlaceholder
});
expect(this.choices.input.placeholder).toEqual(dummyPlaceholder);
});
});
describe('should accept multiple select inputs', function() {