new user indicies reload test and fix for it

Author: dhilt

src/ui-scroll.js

@@ -111,7 +111,9 @@ angular.module('ui.scroll', [])
           // need to postpone min/maxIndexUser processing if the view is empty
           if(datasource.hasOwnProperty(propName) && !buffer.length) {
             getter = datasource[propName];
-            onRenderHandlers.push(() => datasource[propName] = getter);
+            if(Number.isInteger(getter)) {
+              onRenderHandlers.push(() => datasource[propName] = getter);
+            }
           }
         }
 

test/UserIndiciesSpec.js

@@ -110,20 +110,20 @@ describe('uiScroll user min/max indicies.', () => {
       );
     });
 
-    // it('should persist user minIndex after reload', () => {
-    //   inject(function(myInfiniteDatasource) {
-    //     myInfiniteDatasource.minIndex = userMinIndex;
-    //   });
-    //   runTest(scrollSettings,
-    //     (viewport, scope) => {
-    //       scope.adapter.reload();
-    //       const topPaddingElement = angular.element(viewport.children()[0]);
-    //       const virtualItemsAmount = (-1) * userMinIndex - bufferSize + 1;
-    //       expect(topPaddingElement.height()).toBe(itemHeight * virtualItemsAmount);
-    //       expect(viewport.scrollTop()).toBe(itemHeight * ((-1) * userMinIndex + 1));
-    //     }
-    //   );
-    // });
+    it('should persist user minIndex after reload', () => {
+      inject(function(myInfiniteDatasource) {
+        myInfiniteDatasource.minIndex = userMinIndex;
+      });
+      runTest(scrollSettings,
+        (viewport, scope) => {
+          scope.adapter.reload();
+          const topPaddingElement = angular.element(viewport.children()[0]);
+          const virtualItemsAmount = (-1) * userMinIndex - bufferSize + 1;
+          expect(topPaddingElement.height()).toBe(itemHeight * virtualItemsAmount);
+          expect(viewport.scrollTop()).toBe(itemHeight * ((-1) * userMinIndex + 1));
+        }
+      );
+    });
 
   });