🛠 add onInit option for binding initial events

fixes race condition on prefill tests

Author: desandro

.gitignore

@@ -1,3 +1,2 @@
-infinte-scroll.tmproj
-.idea
-
+bower_components/
+node_modules/

js/core.js

@@ -64,6 +64,7 @@ proto.create = function() {
   }
   this.updateGetAbsolutePath();
   this.log( 'initialized', [ this.element.className ] );
+  this.callOnInit();
   // create features
   for ( var method in InfiniteScroll.create ) {
     InfiniteScroll.create[ method ].call( this );
@@ -74,6 +75,14 @@ proto.option = function( opts ) {
   utils.extend( this.options, opts );
 };
 
+// call onInit option, used for binding events on init
+proto.callOnInit = function() {
+  var onInit = this.options.onInit;
+  if ( onInit ) {
+    onInit.call( this, this );
+  }
+};
+
 // ----- events ----- //
 
 proto.dispatchEvent = function( type, event, args ) {

test/.jshintrc

@@ -4,7 +4,7 @@
   "unused": true,
   "undef": true,
   "globals": {
-    "Episcroll": false,
+    "InfiniteScroll": false,
     "QUnit": false
   }
-}
+}

test/unit/prefill-element.js

@@ -1,9 +1,7 @@
 QUnit.test( 'prefill element', function( assert ) {
 
   // expected load count, each post is 200px tall
-  var expLoadCount = 3
-
-  // assert.ok( true );
+  var expLoadCount = 3;
 
   var done = assert.async( expLoadCount );
 
@@ -16,15 +14,18 @@ QUnit.test( 'prefill element', function( assert ) {
     elementScroll: true,
     history: false,
     scrollThreshold: false,
+    onInit: function() {
+      this.on( 'append', onAppend );
+    },
   });
 
-  infScroll.on( 'append', function() {
+  function onAppend() {
     assert.ok( true, 'prefill element appended post ' + infScroll.loadCount );
     if ( infScroll.loadCount == expLoadCount ) {
       assert.equal( infScroll.loadCount, expLoadCount,
         expLoadCount + ' pages appended' );
     }
     done();
-  });
+  }
 
 });

test/unit/prefill-window.js

@@ -2,7 +2,6 @@ QUnit.test( 'prefill window', function( assert ) {
 
   // expected load count, each post is 200px tall
   var expLoadCount = Math.ceil( window.innerHeight / 200 );
-
   var done = assert.async( expLoadCount );
 
   var infScroll = new InfiniteScroll( '.demo--prefill-window', {
@@ -13,15 +12,18 @@ QUnit.test( 'prefill window', function( assert ) {
     prefill: true,
     history: false,
     scrollThreshold: false,
+    onInit: function() {
+      this.on( 'append', onAppend );
+    },
   });
 
-  infScroll.on( 'append', function() {
+  function onAppend() {
     assert.ok( true, 'prefill window appended post ' + infScroll.loadCount );
     if ( infScroll.loadCount == expLoadCount ) {
       assert.equal( infScroll.loadCount, expLoadCount,
         expLoadCount + ' pages appended' );
     }
     done();
-  });
+  }
 
 });