Skip to content
This repository was archived by the owner on Oct 2, 2019. It is now read-only.

Commit 627e51e

Browse files
Jefiozieaaronroberson
authored andcommitted
fix(ResetSearchInput) default is set to true, tests edited. (#1773)
1 parent 238d990 commit 627e51e

File tree

2 files changed

+16
-16
lines changed

2 files changed

+16
-16
lines changed

src/common.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ var uis = angular.module('ui.select', [])
102102
skipFocusser: false,
103103
dropdownPosition: 'auto',
104104
removeSelected: true,
105-
resetSearchInput: false,
105+
resetSearchInput: true,
106106
generateId: function() {
107107
return latestId++;
108108
},

test/select.spec.js

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -2759,18 +2759,18 @@ describe('ui-select tests', function() {
27592759
});
27602760

27612761
describe('resetSearchInput option multiple', function () {
2762-
it('should be false by default', function () {
2763-
expect(createUiSelectMultiple().scope().$select.resetSearchInput).toBe(false);
2762+
it('should be true by default', function () {
2763+
expect(createUiSelectMultiple().scope().$select.resetSearchInput).toBe(true);
27642764
});
27652765

2766-
it('should be true when set.', function () {
2767-
expect(createUiSelectMultiple({ resetSearchInput: true }).scope().$select.resetSearchInput).toBe(true);
2766+
it('should be false when set.', function () {
2767+
expect(createUiSelectMultiple({ resetSearchInput: false }).scope().$select.resetSearchInput).toBe(false);
27682768
});
27692769
});
27702770

27712771
describe('Reset the search value', function () {
27722772
it('should clear the search input when resetSearchInput is true', function () {
2773-
var el = createUiSelectMultiple({ resetSearchInput: true });
2773+
var el = createUiSelectMultiple();
27742774
$(el).scope().$select.search = 'idontexist';
27752775
$(el).scope().$select.select('idontexist');
27762776
expect($(el).scope().$select.search).toEqual('');
@@ -2783,11 +2783,11 @@ describe('ui-select tests', function() {
27832783
expect($(el).scope().$select.search).toEqual('idontexist');
27842784
});
27852785

2786-
it('should not clear the search input when resetSearchInput is default set', function () {
2786+
it('should clear the search input when resetSearchInput is default set', function () {
27872787
var el = createUiSelectMultiple();
27882788
$(el).scope().$select.search = 'idontexist';
27892789
$(el).scope().$select.select('idontexist');
2790-
expect($(el).scope().$select.search).toEqual('idontexist');
2790+
expect($(el).scope().$select.search).toEqual('');
27912791
});
27922792
});
27932793
});
@@ -2859,38 +2859,38 @@ describe('ui-select tests', function() {
28592859
});
28602860

28612861
describe('resetSearchInput option', function () {
2862-
it('should be false by default', function () {
2863-
expect(createUiSelect().scope().$select.resetSearchInput).toBe(false);
2862+
it('should be true by default', function () {
2863+
expect(createUiSelect().scope().$select.resetSearchInput).toBe(true);
28642864
});
28652865

2866-
it('should be overridden by inline option reset-search-input=true', function () {
2867-
expect(createUiSelect({ resetSearchInput: true }).scope().$select.resetSearchInput).toBe(true);
2866+
it('should be overridden by inline option reset-search-input=false', function () {
2867+
expect(createUiSelect({ resetSearchInput: false }).scope().$select.resetSearchInput).toBe(false);
28682868
});
28692869

28702870
describe('Reset the search value', function () {
28712871
it('should clear the search input when resetSearchInput is true', function () {
2872-
var control = createUiSelect({ resetSearchInput: true });
2872+
var control = createUiSelect();
28732873
setSearchText(control, 'idontexist');
28742874
clickMatch(control);
28752875
expect(control.scope().$select.search).toEqual('');
28762876
});
28772877

28782878
it('should not clear the search input', function () {
2879-
var control = createUiSelect();
2879+
var control = createUiSelect({ resetSearchInput: false });
28802880
setSearchText(control, 'idontexist');
28812881
clickMatch(control);
28822882
expect(control.scope().$select.search).toEqual('idontexist');
28832883
});
28842884

28852885
it('should clear the search input when resetSearchInput is true and closeOnSelect is true', function () {
2886-
var control = createUiSelect({ resetSearchInput: true, closeOnSelect: true });
2886+
var control = createUiSelect({ closeOnSelect: true });
28872887
setSearchText(control, 'idontexist');
28882888
clickMatch(control);
28892889
expect(control.scope().$select.search).toEqual('');
28902890
});
28912891

28922892
it('should clear the search input when resetSearchInput is true and closeOnSelect is false', function () {
2893-
var control = createUiSelect({ resetSearchInput: true, closeOnSelect: false });
2893+
var control = createUiSelect({ closeOnSelect: false });
28942894
setSearchText(control, 'idontexist');
28952895
clickMatch(control);
28962896
expect(control.scope().$select.search).toEqual('');

0 commit comments

Comments
 (0)