$firebase: test covarage for config.object/arrayFactory being non-function injectables

Author: jamestalmage

tests/unit/firebase.spec.js

@@ -17,7 +17,7 @@ describe('$firebase', function () {
       $get: function() {
         return function() {};
       }
-    });
+    }).value('NonFunctionFactory','NonFunctionValue');
     inject(function (_$firebase_, _$timeout_, _$rootScope_, $firebaseUtils) {
       $firebase = _$firebase_;
       $timeout = _$timeout_;
@@ -65,14 +65,28 @@ describe('$firebase', function () {
     it('should throw an error if factory name for arrayFactory does not exist', function()  {
       var ref = new Firebase('Mock://');
       expect(function() {
-        $firebase(ref, {arrayFactory: 'notarealarrayfactorymethod'})
+        $firebase(ref, {arrayFactory: 'notarealarrayfactorymethod'}); //injectable by that name doesn't exist.
+      }).toThrowError();
+    });
+
+    it('should throw an error if factory name for arrayFactory exists, but is not a function', function()  {
+      var ref = new Firebase('Mock://');
+      expect(function() {
+        $firebase(ref, {arrayFactory: 'NonFunctionFactory'}); //injectable exists, but is not a function.
       }).toThrowError();
     });
 
     it('should throw an error if factory name for objectFactory does not exist', function()  {
       var ref = new Firebase('Mock://');
       expect(function() {
-        $firebase(ref, {objectFactory: 'notarealobjectfactorymethod'})
+        $firebase(ref, {objectFactory: 'notarealobjectfactorymethod'}); //injectable by that name doesn't exist.
+      }).toThrowError();
+    });
+
+    it('should throw an error if factory name for objectFactory exists, but is not a function', function()  {
+      var ref = new Firebase('Mock://');
+      expect(function() {
+        $firebase(ref, {objectFactory: 'NonFunctionFactory'}); //injectable exists, but is not a function.
       }).toThrowError();
     });
   });