Only refresh once during an enableAll call, imakewebthings#454

Previously, enableAll would invoke enable on each Waypoint, but
enable calls refresh on the context. This meant n-refreshes where
n is the number of active Waypoints.

Author: imakewebthings

package.json

@@ -28,7 +28,7 @@
     "gulp-tap": "^0.1.1",
     "gulp-uglify": "^0.3.1",
     "merge-stream": "^0.1.1",
-    "testem": "^0.6.24"
+    "testem": "0.6.24"
   },
   "license": "MIT"
 }

src/waypoint.js

@@ -115,7 +115,11 @@
   /* Public */
   /* http://imakewebthings.com/waypoints/api/enable-all */
   Waypoint.enableAll = function() {
-    Waypoint.invokeAll('enable')
+    Waypoint.Context.refreshAll()
+    for (var waypointKey in allWaypoints) {
+      allWaypoints[waypointKey].enabled = true
+    }
+    return this
   }
 
   /* Public */

test/waypoint-spec.js

@@ -527,20 +527,29 @@ window.jQuery.each(Waypoint.adapters, function(i, adapter) {
       })
 
       describe('Waypoint.enableAll()', function() {
-        it('calls enable on all waypoints', function() {
-          var secondWaypoint = new Waypoint({
+        var secondWaypoint
+
+        beforeEach(function() {
+          secondWaypoint = new Waypoint({
             element: $('#same1')[0],
             handler: function() {}
           })
           waypoint = new Waypoint({
             element: $('#same1')[0],
             handler: function() {}
           })
-          spyOn(secondWaypoint, 'enable').andCallThrough()
-          spyOn(waypoint, 'enable').andCallThrough()
+          Waypoint.disableAll()
+          spyOn(Waypoint.Context, 'refreshAll').andCallThrough()
           Waypoint.enableAll()
-          expect(secondWaypoint.enable).toHaveBeenCalled()
-          expect(waypoint.enable).toHaveBeenCalled()
+        })
+
+        it('sets enabled on all waypoints', function() {
+          expect(secondWaypoint.enabled).toBeTruthy()
+          expect(waypoint.enabled).toBeTruthy()
+        })
+
+        it('refreshes all contexts', function() {
+          expect(Waypoint.Context.refreshAll).toHaveBeenCalled()
         })
       })
     })