fix(auth): return promise reject on request error

nuxt-community · Apr 1, 2018 · f2883c6 · f2883c6
1 parent ffd44de
commit f2883c6
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 10 deletions.
diff --git a/examples/demo/store/index.js b/examples/demo/store/index.js
diff --git a/lib/auth/auth.js b/lib/auth/auth.js
@@ -89,9 +89,10 @@ export default class Auth {
       return this.fetchUserOnce()
     }
 
-    return Promise.resolve(this.strategy.mounted(...arguments)).catch(error =>
+    return Promise.resolve(this.strategy.mounted(...arguments)).catch(error => {
       this.callOnError(error, { method: 'mounted' })
-    )
+      return Promise.reject(error)
+    })
   }
 
   loginWith (name, ...args) {
@@ -103,19 +104,21 @@ export default class Auth {
       return Promise.resolve()
     }
 
-    return this.wrapLogin(this.strategy.login(...arguments)).catch(error =>
+    return this.wrapLogin(this.strategy.login(...arguments)).catch(error => {
       this.callOnError(error, { method: 'login' })
-    )
+      return Promise.reject(error)
+    })
   }
 
   fetchUser () {
     if (!this.strategy.fetchUser) {
       return Promise.resolve()
     }
 
-    return Promise.resolve(this.strategy.fetchUser(...arguments)).catch(error =>
+    return Promise.resolve(this.strategy.fetchUser(...arguments)).catch(error => {
       this.callOnError(error, { method: 'fetchUser' })
-    )
+      return Promise.reject(error)
+    })
   }
 
   logout () {
@@ -124,9 +127,10 @@ export default class Auth {
       return Promise.resolve()
     }
 
-    return Promise.resolve(this.strategy.logout(...arguments)).catch(error =>
+    return Promise.resolve(this.strategy.logout(...arguments)).catch(error => {
       this.callOnError(error, { method: 'logout' })
-    )
+      return Promise.reject(error)
+    })
   }
 
   reset () {
@@ -136,9 +140,10 @@ export default class Auth {
       return Promise.resolve()
     }
 
-    return Promise.resolve(this.strategy.reset(...arguments)).catch(error =>
+    return Promise.resolve(this.strategy.reset(...arguments)).catch(error => {
       this.callOnError(error, { method: 'reset' })
-    )
+      return Promise.reject(error)
+    })
   }
 
   // ---------------------------------------------------------------