additional testing of alternative text search

Author: nlopezcenteno

test/basic.spec.js

@@ -80,24 +80,45 @@ describe('basic suite', function() {
     expect(select.val()).toBe('banana');
   });
 
+  it('should not search a disabled option', function() {
+    select.find('option:eq(4)').prop('disabled', 'disabled');
+    select.selectric('refresh');
+    $('.selectric-input').val('banana').trigger('input');
+    $('.selectric-items').find('.highlighted').lenght;
+    expect($('.selectric-items').find('.highlighted').length).toBe(0);
+  });  
+
   it('should search alternative text', function () {
-    $('.selectric-input').val('alt banana').trigger('input');
+    select.find('option:eq(6)').attr('data-alt', 'alt blackberry');
+    select.selectric('refresh');
+    $('.selectric-input').val('alt blackberry').trigger('input');
     $('.selectric-items').find('.highlighted').click();
-    expect(select.val()).toBe('banana');
+    expect(select.val()).toBe('blackberry');
   });
 
   it('should search alternative text with separator', function () {
-    $('.selectric-input').val('altberry').trigger('input');
+    select.find('option:eq(6)').attr('data-alt', 'alt blackberry | another berry');
+    select.selectric('refresh');
+    $('.selectric-input').val('alt blackberry').trigger('input');
     $('.selectric-items').find('.highlighted').click();
-    expect(select.val()).toBe('bilberry');
+    expect(select.val()).toBe('blackberry');
   });
 
   it('should search alternative text with separator 2', function () {
+    select.find('option:eq(6)').attr('data-alt', 'alt blackberry | another berry');
+    select.selectric('refresh');
     $('.selectric-input').val('another berry').trigger('input');
     $('.selectric-items').find('.highlighted').click();
-    expect(select.val()).toBe('bilberry');
+    expect(select.val()).toBe('blackberry');
   });  
 
+  it('should skip blank alternative text', function () {
+    select.find('option:eq(6)').attr('data-alt', '');
+    select.selectric('refresh');
+    $('.selectric-input').val('a text that does not exist').trigger('input');
+    expect($('.selectric-items').find('.highlighted').length).toBe(0);
+  });    
+
   it('highlight() should return undefined if index is undefined', function () {
     expect(select.data('selectric').highlight(undefined)).toBe(undefined);
   });

test/fixtures/basic.html

@@ -4,8 +4,8 @@
   <option value="loooooooooooooong-option">Loooooooooooooong option</option>
   <option value="apple">Apple</option>
   <option selected value="apricot">Apricot</option>
-  <option class="customOptionClass" value="banana"data-alt="alt banana">Banana</option>
-  <option value="bilberry" data-alt="altberry | another berry">Bilberry</option>
+  <option class="customOptionClass" value="banana">Banana</option>
+  <option value="bilberry">Bilberry</option>
   <option value="blackberry">Blackberry</option>
   <option value="blackcurrant">Blackcurrant</option>
   <option value="blueberry">Blueberry</option>