Merge remote-tracking branch 'origin/master' into vmihalkov/groupby

Author: ChronosSF

projects/igniteui-angular/src/lib/grids/grid/row-pinning.spec.ts

@@ -30,7 +30,7 @@ describe('Row Pinning #grid', () => {
                 IgxGridModule
             ]
         })
-        .compileComponents();
+            .compileComponents();
     }));
 
     describe('', () => {
@@ -71,7 +71,7 @@ describe('Row Pinning #grid', () => {
 
             // 2 records pinned + 2px border
             expect(grid.pinnedRowHeight).toBe(2 * grid.renderedRowHeight + 2);
-            const expectedHeight = parseInt(grid.height, 10) - grid.pinnedRowHeight - 18 -  grid.theadRow.nativeElement.offsetHeight;
+            const expectedHeight = parseInt(grid.height, 10) - grid.pinnedRowHeight - 18 - grid.theadRow.nativeElement.offsetHeight;
             expect(grid.calcHeight - expectedHeight).toBeLessThanOrEqual(1);
         });
 
@@ -110,7 +110,7 @@ describe('Row Pinning #grid', () => {
 
             // 2 records pinned + 2px border
             expect(grid.pinnedRowHeight).toBe(2 * grid.renderedRowHeight + 2);
-            const expectedHeight = parseInt(grid.height, 10) - grid.pinnedRowHeight - 18 -  grid.theadRow.nativeElement.offsetHeight;
+            const expectedHeight = parseInt(grid.height, 10) - grid.pinnedRowHeight - 18 - grid.theadRow.nativeElement.offsetHeight;
             expect(grid.calcHeight - expectedHeight).toBeLessThanOrEqual(1);
         });
 
@@ -195,31 +195,31 @@ describe('Row Pinning #grid', () => {
         });
 
         it('should pin/unpin via row pinned setter.', () => {
-              // pin 2nd row
-              let row = grid.getRowByIndex(1);
-              row.pinned = true;
-              fix.detectChanges();
-
-              expect(grid.pinnedRows.length).toBe(1);
-              let pinRowContainer = fix.debugElement.queryAll(By.css(FIXED_ROW_CONTAINER));
-              expect(pinRowContainer.length).toBe(1);
-              expect(pinRowContainer[0].children.length).toBe(1);
-              expect(pinRowContainer[0].children[0].context.rowID).toBe(fix.componentInstance.data[1]);
-
-              expect(grid.getRowByIndex(0).rowID).toBe(fix.componentInstance.data[1]);
-              expect(grid.getRowByIndex(1).rowID).toBe(fix.componentInstance.data[0]);
-
-              // unpin
-              row = grid.getRowByIndex(0);
-              row.pinned = false;
-              fix.detectChanges();
-
-              expect(grid.pinnedRows.length).toBe(0);
-              pinRowContainer = fix.debugElement.queryAll(By.css(FIXED_ROW_CONTAINER));
-              expect(pinRowContainer.length).toBe(0);
-
-              expect(grid.getRowByIndex(0).rowID).toBe(fix.componentInstance.data[0]);
-              expect(grid.getRowByIndex(1).rowID).toBe(fix.componentInstance.data[1]);
+            // pin 2nd row
+            let row = grid.getRowByIndex(1);
+            row.pinned = true;
+            fix.detectChanges();
+
+            expect(grid.pinnedRows.length).toBe(1);
+            let pinRowContainer = fix.debugElement.queryAll(By.css(FIXED_ROW_CONTAINER));
+            expect(pinRowContainer.length).toBe(1);
+            expect(pinRowContainer[0].children.length).toBe(1);
+            expect(pinRowContainer[0].children[0].context.rowID).toBe(fix.componentInstance.data[1]);
+
+            expect(grid.getRowByIndex(0).rowID).toBe(fix.componentInstance.data[1]);
+            expect(grid.getRowByIndex(1).rowID).toBe(fix.componentInstance.data[0]);
+
+            // unpin
+            row = grid.getRowByIndex(0);
+            row.pinned = false;
+            fix.detectChanges();
+
+            expect(grid.pinnedRows.length).toBe(0);
+            pinRowContainer = fix.debugElement.queryAll(By.css(FIXED_ROW_CONTAINER));
+            expect(pinRowContainer.length).toBe(0);
+
+            expect(grid.getRowByIndex(0).rowID).toBe(fix.componentInstance.data[0]);
+            expect(grid.getRowByIndex(1).rowID).toBe(fix.componentInstance.data[1]);
         });
 
         it('should pin rows when columns are grouped.', () => {
@@ -253,6 +253,26 @@ describe('Row Pinning #grid', () => {
             // make sure the pinned row is out of the first groupBy group
             expect(groupRows[0].groupRow.records[0].ID).toEqual('BLAUS');
         });
+
+        it('should apply sorting to both pinned and unpinned rows.', () => {
+            grid.getRowByIndex(1).pin();
+            grid.getRowByIndex(5).pin();
+            fix.detectChanges();
+
+            expect(grid.getRowByIndex(0).rowID).toBe(fix.componentInstance.data[1]);
+            expect(grid.getRowByIndex(1).rowID).toBe(fix.componentInstance.data[5]);
+
+            grid.sort({ fieldName: 'ID', dir: SortingDirection.Desc, ignoreCase: false });
+            fix.detectChanges();
+
+            // check pinned rows data is sorted
+            expect(grid.getRowByIndex(0).rowID).toBe(fix.componentInstance.data[5]);
+            expect(grid.getRowByIndex(1).rowID).toBe(fix.componentInstance.data[1]);
+
+            // check unpinned rows data is sorted
+            const lastIndex = fix.componentInstance.data.length - 1;
+            expect(grid.getRowByIndex(2).rowID).toBe(fix.componentInstance.data[lastIndex]);
+        });
     });
 
     describe(' Editing ', () => {
@@ -293,7 +313,7 @@ describe('Row Pinning #grid', () => {
 
         it('should allow pinning added row.', () => {
 
-            grid.addRow({ 'ID': 'Test', 'CompanyName': 'Test'});
+            grid.addRow({ 'ID': 'Test', 'CompanyName': 'Test' });
             fix.detectChanges();
 
             grid.pinRow('Test');