fix(#1309): Scope.bindToContext() will be applied on scope if no context is specified, and add specs

DAB0mB · DAB0mB · commit 210f16adac0e · 2016-04-08T10:26:39.000+03:00
diff --git a/.gitignore b/.gitignore
@@ -10,6 +10,7 @@ Thumbs.db
 .build*
 .npm
 node_modules
+npm-debug.log
 
 # IDE's #
 #########
diff --git a/src/modules/core.js b/src/modules/core.js
@@ -125,6 +125,11 @@ angular.module(name, [
 
     // Binds an object or a function to the provided context and digest it once it is invoked
     $$Core.$bindToContext = function(context, fn) {
+      if (_.isFunction(context)) {
+        fn = context;
+        context = this;
+      }
+
       return $$utils.bind(fn, context, this.$$throttledDigest.bind(this));
     };
 
diff --git a/tests/integration/core.spec.js b/tests/integration/core.spec.js
@@ -340,5 +340,42 @@ describe('angular-meteor.core', function() {
         scope.applyMethod('method', ['foo', 'bar'], angular.noop);
       });
     });
+
+    describe('$bindToContext()', function() {
+      var scope;
+
+      beforeEach(function() {
+        scope = $rootScope.$new();
+      });
+
+      afterEach(function() {
+        scope.$destroy();
+      });
+
+      it('should bind a function to scope and digest once invoked', function() {
+        var fn = jasmine.createSpy('function');
+        scope.$digest = jasmine.createSpy('digest');
+
+        var boundFn = scope.$bindToContext(fn);
+        boundFn(1, 2, 3);
+
+        expect(fn).toHaveBeenCalled();
+        expect(fn.calls.mostRecent().object).toEqual(scope);
+        expect(fn.calls.mostRecent().args).toEqual([1, 2, 3]);
+      });
+
+      it('should bind the function to a custom scope if specified', function() {
+        var fn = jasmine.createSpy('function');
+        scope.$digest = jasmine.createSpy('digest');
+
+        var context = {};
+        var boundFn = scope.$bindToContext(context, fn);
+        boundFn(1, 2, 3);
+
+        expect(fn).toHaveBeenCalled();
+        expect(fn.calls.mostRecent().object).toEqual(context);
+        expect(fn.calls.mostRecent().args).toEqual([1, 2, 3]);
+      });
+    });
   });
 });
