Merge pull request #6921 from IgniteUI/vmihalkov/groupby

Added a test for GroupBy with Row Pinning

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,63 @@ 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.', () => {
+            // pin 1st and 2nd data row
+            grid.pinRow(fix.componentInstance.data[0]);
+            grid.pinRow(fix.componentInstance.data[1]);
+            fix.detectChanges();
+
+            // group by string column
+            grid.groupBy({
+                fieldName: 'ContactTitle', dir: SortingDirection.Desc, ignoreCase: false
+            });
+            fix.detectChanges();
+
+            expect(grid.pinnedRows.length).toBe(2);
+
+            // verify rows
+            const groupRows = grid.groupsRowList.toArray();
+            const dataRows = grid.dataRowList.toArray();
+
+            expect(groupRows.length).toEqual(2);
+            expect(dataRows.length).toEqual(7);
+            expect(groupRows[0].groupRow.records[0].ID).toEqual('AROUT');
+
+            // pin 4th data row with ID:AROUT
+            grid.pinRow(fix.componentInstance.data[3]);
+            fix.detectChanges();
+
+            expect(grid.pinnedRows.length).toBe(3);
+
+            // 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.', () => {
@@ -281,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');