makeNodeResolver should check error === null instead of falsy

Author: jamestalmage

src/utils.js

@@ -229,13 +229,13 @@
 
           makeNodeResolver:function(deferred){
             return function(err,result){
-              if(err){
-                deferred.reject(err);
+              if(err === null){
+                deferred.resolve(result);
               }
               else {
-                deferred.resolve(result);
+                deferred.reject(err);
               }
-            }
+            };
           },
 
           wait: function(fn, wait) {

tests/unit/utils.spec.js

@@ -198,6 +198,11 @@ describe('$firebaseUtils', function () {
       expect(deferred.reject).toHaveBeenCalledWith(error);
     });
 
+    it('should reject the promise if the first argument is not null', function(){
+      callback(false);
+      expect(deferred.reject).toHaveBeenCalledWith(false);
+    });
+
     it('should resolve the promise if the first argument is falsy', function(){
       var result = {data:'hello world'};
       callback(null,result);