added test cases for pipe

Author: NileshPatel17

src/ng-multiselect-dropdown/src/list-filter.pipe.ts

@@ -3,7 +3,7 @@ import { Pipe, PipeTransform } from '@angular/core';
 import { ListItem } from './multiselect.model';
 
 @Pipe({
-    name: 'ng2ListFilter',
+    name: 'multiSelectFilter',
     pure: false
 })
 export class ListFilterPipe implements PipeTransform {
@@ -15,6 +15,10 @@ export class ListFilterPipe implements PipeTransform {
     }
 
     applyFilter(item: ListItem, filter: ListItem): boolean {
-        return !(filter.text && item.text && item.text.toLowerCase().indexOf(filter.text.toLowerCase()) === -1);
+        if (typeof item.text === 'string' && typeof filter.text === 'string') {
+            return !(filter.text && item.text && item.text.toLowerCase().indexOf(filter.text.toLowerCase()) === -1);
+        } else {
+            return !(filter.text && item.text && item.text.toString().toLowerCase().indexOf(filter.text.toString().toLowerCase()) === -1);
+        }
     }
 }

src/ng-multiselect-dropdown/src/multi-select.component.html

@@ -23,7 +23,7 @@
       </li>
     </ul>
     <ul class="item2" [style.maxHeight]="_settings.maxHeight+'px'">
-      <li *ngFor="let item of _data | ng2ListFilter:filter; let i = index;" (click)="onItemClick($event,item)" class="multiselect-item-checkbox">
+      <li *ngFor="let item of _data | multiSelectFilter:filter; let i = index;" (click)="onItemClick($event,item)" class="multiselect-item-checkbox">
         <input type="checkbox" aria-label="multiselect-item" [checked]="isSelected(item)" [disabled]="disabled || (isLimitSelectionReached() && !isSelected(item)) || item.isDisabled" />
         <div>{{item.text}}</div>
       </li>

src/ng-multiselect-dropdown/src/multiselect.model.ts

@@ -24,7 +24,7 @@ export class ListItem {
   isDisabled?: boolean;
 
   public constructor(source: any) {
-    if (typeof source === 'string' || typeof source == 'number') {
+    if (typeof source === 'string' || typeof source === 'number') {
       this.id = this.text = source;
       this.isDisabled = false;
     }

src/ng-multiselect-dropdown/test/list-filter.pipe.spec.ts

@@ -1,3 +1,41 @@
-it('sanity test', () => {
-    expect(1).toBe(1)
+import { ListFilterPipe } from '../src/list-filter.pipe'
+describe('multiSelectFilter', () => {
+    let pipe: ListFilterPipe;
+    beforeEach(() => {
+        pipe = new ListFilterPipe();
+    })
+    it('should create an instance', () => {
+        expect(pipe).toBeTruthy();
+    });
+    it('bad inputs:should return source data if filter data is null', () => {
+        const sourceData = [
+            {
+                id: 1, text: 'Navsari'
+            }, {
+                id: 2, text: 'Pune'
+            }];
+        expect(pipe.transform(sourceData, null)).toEqual(sourceData);
+    })
+    it('transform string data', () => {
+        const sourceData = [
+            {
+                id: 1, text: 'Navsari'
+            }, {
+                id: 2, text: 'Pune'
+            }];
+        const filterData = { id: 1, text: 'Nav' };
+        const expectedData = [{ id: 1, text: 'Navsari' }]
+        expect(pipe.transform(sourceData, filterData)).toEqual(expectedData);
+    })
+    it('transform numbers data', () => {
+        const sourceData = [
+            {
+                id: 1111, text: 1111
+            }, {
+                id: 2222, text: 2222
+            }];
+        const filterData = { id: 1111, text: 1111 };
+        const expectedData = [{ id: 1111, text: 1111 }]
+        expect(pipe.transform(sourceData, filterData)).toEqual(expectedData);
+    })
 })