fix(#20): Silent error in waitForUser() (#23)

Urigo · Apr 22, 2016 · e0292c0 · e0292c0
1 parent d482b90
commit e0292c0
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/src/angular-meteor-auth.js b/src/angular-meteor-auth.js
@@ -16,8 +16,9 @@ angular.module(name, [
  */
 .factory('$$Auth', [
   '$Mixer',
+  '$log',
 
-function($Mixer) {
+function($Mixer, $log) {
   const Accounts = (Package['accounts-base'] || {}).Accounts;
 
   if (!Accounts) {
@@ -104,7 +105,9 @@ function($Mixer) {
   // Silent error
   $$Auth.$waitForUser = function() {
     // Silent error
-    return this.$awaitUser().catch();
+    return this.$awaitUser().catch((err) => {
+      $log.debug(`user login has failed (${err})`);
+    });
   };
 
   // No validation

diff --git a/tests/integration/auth.spec.js b/tests/integration/auth.spec.js
@@ -217,7 +217,11 @@ describe('angular-meteor.auth', function() {
       });
 
       it('should call $awaitUser() and ignore error', function(done) {
-        scope.$waitForUser().catch(done).finally(done);
+        scope.$waitForUser()
+          .then(angular.noop, done.fail)
+          .catch(done.fail)
+          .finally(done);
+
         scope.$$afterFlush('$$throttledDigest');
       });
     });