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.

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()
  })
  })
  })