test(tree-base.spec): Updating tree-base unit tests.

Also, setting up tree-base to support future performance improvement.

Author: Portugal, Marcelo

src/features/tree-base/js/tree-base.js

@@ -715,7 +715,7 @@
         };
 
         rowHeaderColumnDef.visible = grid.options.treeRowHeaderAlwaysVisible;
-        grid.addRowHeaderColumn( rowHeaderColumnDef, -100 );
+        grid.addRowHeaderColumn(rowHeaderColumnDef, -100, true);
       },
 
 

src/features/tree-base/test/tree-base.spec.js

@@ -1,26 +1,28 @@
 describe('ui.grid.treeBase uiGridTreeBaseService', function () {
-  var uiGridTreeBaseService;
-  var uiGridTreeBaseConstants;
-  var gridClassFactory;
-  var grid;
-  var $rootScope;
-  var $scope;
-  var GridRow;
-  var gridUtil;
-  var uiGridConstants;
-
-  beforeEach(module('ui.grid.treeBase'));
-
-  beforeEach(inject(function (_uiGridTreeBaseService_,_gridClassFactory_, $templateCache, _uiGridTreeBaseConstants_,
-                              _$rootScope_, _GridRow_, _gridUtil_, _uiGridConstants_) {
-    uiGridTreeBaseService = _uiGridTreeBaseService_;
-    uiGridTreeBaseConstants = _uiGridTreeBaseConstants_;
-    gridClassFactory = _gridClassFactory_;
-    $rootScope = _$rootScope_;
+  var uiGridTreeBaseService,
+      uiGridTreeBaseConstants,
+      gridClassFactory,
+      grid,
+      $rootScope,
+      $scope,
+      GridRow,
+      gridUtil,
+      uiGridConstants;
+
+  beforeEach(function() {
+    module('ui.grid.treeBase');
+
+    inject(function (_uiGridTreeBaseService_,_gridClassFactory_, $templateCache, _uiGridTreeBaseConstants_,
+      _$rootScope_, _GridRow_, _gridUtil_, _uiGridConstants_) {
+      uiGridTreeBaseService = _uiGridTreeBaseService_;
+      uiGridTreeBaseConstants = _uiGridTreeBaseConstants_;
+      gridClassFactory = _gridClassFactory_;
+      $rootScope = _$rootScope_;
+      GridRow = _GridRow_;
+      gridUtil = _gridUtil_;
+      uiGridConstants = _uiGridConstants_;
+    });
     $scope = $rootScope.$new();
-    GridRow = _GridRow_;
-    gridUtil = _gridUtil_;
-    uiGridConstants = _uiGridConstants_;
 
     grid = gridClassFactory.createGrid({});
     grid.options.columnDefs = [
@@ -30,6 +32,7 @@ describe('ui.grid.treeBase uiGridTreeBaseService', function () {
       {field: 'col3'}
     ];
 
+    spyOn(grid, 'addRowHeaderColumn').and.callThrough();
     uiGridTreeBaseService.initializeGrid(grid, $scope);
     $scope.$apply();
 
@@ -44,24 +47,11 @@ describe('ui.grid.treeBase uiGridTreeBaseService', function () {
     data[7].$$treeLevel = 0;
     data[9].$$treeLevel = 1;
 
-//    data = [
-//      { col0: 'a_0', col1: 0, col2: 'c_0', col3: 0, '$$treeLevel': 0 } ,
-//      { col0: 'a_0', col1: 0, col2: 'c_1', col3: 1, '$$treeLevel': 1 },
-//      { col0: 'a_0', col1: 1, col2: 'c_2', col3: 2 },
-//      { col0: 'a_0', col1: 1, col2: 'c_3', col3: 3, '$$treeLevel': 1 },
-//      { col0: 'a_1', col1: 2, col2: 'c_4', col3: 4, '$$treeLevel': 2 },
-//      { col0: 'a_1', col1: 2, col2: 'c_5', col3: 5 },
-//      { col0: 'a_1', col1: 3, col2: 'c_6', col3: 6 },
-//      { col0: 'a_1', col1: 3, col2: 'c_7', col3: 7, '$$treeLevel': 0 },
-//      { col0: 'a_2', col1: 4, col2: 'c_8', col3: 8 },
-//      { col0: 'a_2', col1: 4, col2: 'c_9', col3: 9, '$$treeLevel': 1 }
-//    ];
-
     grid.options.data = data;
 
     grid.buildColumns();
     grid.modifyRows(grid.options.data);
-  }));
+  });
 
 
   describe( 'initializeGrid and defaultGridOptions', function() {
@@ -90,6 +80,9 @@ describe('ui.grid.treeBase uiGridTreeBaseService', function () {
         enableExpandAll: true
       });
     });
+    it('should call addRowHeaderColumn', function() {
+      expect(grid.addRowHeaderColumn).toHaveBeenCalledWith(jasmine.any(Object), -100, true);
+    });
   });