Merge pull request #34 from ifwe/angular-inject

Add angular wrapper helpers to disable batching

Author: djvirgen

lib/wrappers/angular.js

@@ -13,11 +13,20 @@ function angularWrapper(angular, Monocle) {
         this._base = '/';
         this._timeout = 30000;
         this._headers = {};
+        this._enableBatching = true;
 
         this.setBase = function(base) {
             this._base = base;
         };
 
+        this.enableBatching = function() {
+            this._enableBatching = true;
+        };
+
+        this.disableBatching = function() {
+            this._enableBatching = false;
+        };
+
         this.setTimeout = function(timeout) {
             this._timeout = parseInt(timeout, 10) || 30000;
         };
@@ -34,6 +43,9 @@ function angularWrapper(angular, Monocle) {
 
             var monocle = new Monocle(angularAdapter, $q);
             monocle.setBase(this._base);
+            if (!this._enableBatching) {
+              monocle.disableBatching();
+            }
 
             // Wrap all promises in Angular promises
             ['get', 'post', 'put', 'patch', 'delete', 'options'].forEach(function(method) {

monocle-client-angular-min.js

@@ -3519,11 +3519,20 @@ function angularWrapper(angular, Monocle) {
         this._base = '/';
         this._timeout = 30000;
         this._headers = {};
+        this._enableBatching = true;
 
         this.setBase = function(base) {
             this._base = base;
         };
 
+        this.enableBatching = function() {
+            this._enableBatching = true;
+        };
+
+        this.disableBatching = function() {
+            this._enableBatching = false;
+        };
+
         this.setTimeout = function(timeout) {
             this._timeout = parseInt(timeout, 10) || 30000;
         };
@@ -3540,6 +3549,9 @@ function angularWrapper(angular, Monocle) {
 
             var monocle = new Monocle(angularAdapter, $q);
             monocle.setBase(this._base);
+            if (!this._enableBatching) {
+              monocle.disableBatching();
+            }
 
             // Wrap all promises in Angular promises
             ['get', 'post', 'put', 'patch', 'delete', 'options'].forEach(function(method) {

monocle-client-angular.js

@@ -19,6 +19,8 @@ describe('Angular Wrapper', function() {
 
         this.Monocle = sinon.spy();
         this.Monocle.prototype.setBase = sinon.spy();
+        this.Monocle.prototype.disableBatching = sinon.spy();
+        this.Monocle.prototype.enableBatching = sinon.spy();
 
         wrapper(this.angular, this.Monocle);
     });
@@ -60,5 +62,49 @@ describe('Angular Wrapper', function() {
             var api = provider.$get($http, $q, $window);
             this.Monocle.prototype.setBase.calledWith('/test').should.be.true;
         });
+
+        it('does not disable batching by default', function() {
+            var $http = {};
+            var $q = {};
+            var $window = {};
+            var provider = new (this.providers.monocle)();
+            var api = provider.$get($http, $q, $window);
+            this.Monocle.prototype.enableBatching.called.should.be.false;
+            this.Monocle.prototype.disableBatching.called.should.be.false;
+        });
+
+        it('can en batching', function() {
+            var $http = {};
+            var $q = {};
+            var $window = {};
+            var provider = new (this.providers.monocle)();
+            provider.disableBatching();
+            var api = provider.$get($http, $q, $window);
+            this.Monocle.prototype.enableBatching.called.should.be.false;
+            this.Monocle.prototype.disableBatching.called.should.be.true;
+        });
+
+        it('can disable batching', function() {
+            var $http = {};
+            var $q = {};
+            var $window = {};
+            var provider = new (this.providers.monocle)();
+            provider.disableBatching();
+            var api = provider.$get($http, $q, $window);
+            this.Monocle.prototype.enableBatching.called.should.be.false;
+            this.Monocle.prototype.disableBatching.called.should.be.true;
+        });
+
+        it('can disable and re-enable batching', function() {
+            var $http = {};
+            var $q = {};
+            var $window = {};
+            var provider = new (this.providers.monocle)();
+            provider.disableBatching();
+            provider.enableBatching();
+            var api = provider.$get($http, $q, $window);
+            this.Monocle.prototype.enableBatching.called.should.be.false;
+            this.Monocle.prototype.disableBatching.called.should.be.false;
+        });
     });
 });