confirmPasswordReset action (#74)

* Adds `confirmPasswordReset` action
* Adds tests for `confirmPasswordReset` action
* Fixes typo in `test` npm script

Author: Ryan J. Peterson

package.json

@@ -10,7 +10,7 @@
     "clean": "rimraf dist",
     "lint": "eslint src/** test/**",
     "lint:fix": "npm run lint -- --fix",
-    "test": "mocha -R spec --compilers js:babel-core/register ./tests/setup.js ./test/**/*.spec.js",
+    "test": "mocha -R spec --compilers js:babel-core/register ./tests/setup.js ./tests/**/*.spec.js",
     "test:cov": "istanbul cover ./node_modules/mocha/bin/_mocha -- ./tests/** --recursive --report lcov --compilers js:babel-register --require babel-polyfill",
     "codecov": "cat coverage/*/lcov.info | codecov",
     "build:commonjs": "cross-env BABEL_ENV=commonjs babel src --out-dir lib",

src/actions/auth.js

@@ -388,6 +388,46 @@ export const resetPassword = (dispatch, firebase, email) => {
     })
 }
 
+/**
+ * @description Confirm the password reset with code and password
+ * @param {Function} dispatch - Action dispatch function
+ * @param {Object} firebase - Internal firebase object
+ * @param {String} code - Email confirmation reset code
+ * @param {String} password - Password to set it to
+ * @return {Promise}
+ * @private
+ */
+export const confirmPasswordReset = (dispatch, firebase, code, password) => {
+  dispatchLoginError(dispatch, null)
+  return firebase.auth()
+    .confirmPasswordReset(code, password)
+    .catch((err) => {
+      if (err) {
+        switch (err.code) {
+          case 'auth/expired-action-code':
+            dispatchLoginError(dispatch, new Error('The action code has expired.'))
+            break
+          case 'auth/invalid-action-code':
+            dispatchLoginError(dispatch, new Error('The action code is invalid.'))
+            break
+          case 'auth/user-disabled':
+            dispatchLoginError(dispatch, new Error('The user is disabled.'))
+            break
+          case 'auth/user-not-found':
+            dispatchLoginError(dispatch, new Error('The user is not found.'))
+            break
+          case 'auth/weak-password':
+            dispatchLoginError(dispatch, new Error('The password is not strong enough.'))
+            break
+          default:
+            dispatchLoginError(dispatch, err)
+        }
+        return Promise.reject(err)
+      }
+    })
+}
+
+
 export default {
   dispatchLoginError,
   dispatchUnauthorizedError,
@@ -399,5 +439,6 @@ export default {
   login,
   logout,
   createUser,
-  resetPassword
+  resetPassword,
+  confirmPasswordReset
 }

src/compose.js

@@ -141,6 +141,9 @@ export default (fbConfig, otherConfig) => next =>
     const resetPassword = (credentials) =>
       authActions.resetPassword(dispatch, firebase, credentials)
 
+    const confirmPasswordReset = (code, password) =>
+      authActions.confirmPasswordReset(dispatch, firebase, code, password)
+
     firebase.helpers = {
       ref: path => Firebase.database().ref(path),
       set,
@@ -155,6 +158,7 @@ export default (fbConfig, otherConfig) => next =>
       deleteFile,
       createUser,
       resetPassword,
+      confirmPasswordReset,
       watchEvent,
       unWatchEvent,
       storage: () => Firebase.storage()

tests/unit/actions/auth.spec.js

@@ -10,7 +10,8 @@ import {
   login,
   logout,
   createUser,
-  resetPassword
+  resetPassword,
+  confirmPasswordReset,
 } from '../../../src/actions/auth'
 import { promisesForPopulate } from '../../../src/utils/populate'
 
@@ -69,7 +70,11 @@ const fakeFirebase = {
         ? Promise.reject({code: 'auth/user-not-found'})
         : email === 'error2'
           ? Promise.reject({code: 'asdfasdf'})
-          : Promise.resolve({some: 'val'})
+          : Promise.resolve({some: 'val'}),
+    confirmPasswordReset: (code, password) =>
+      password === 'error'
+        ? Promise.reject({code: code})
+        : Promise.resolve()
   })
 }
 
@@ -283,4 +288,19 @@ describe('Actions: Auth', () => {
         })
     })
   })
+
+  describe('confirmPasswordReset', () => {
+    it('resets password for real user', () => {
+      return confirmPasswordReset(dispatch, fakeFirebase, 'test', 'test')
+        .then((err) => {
+          expect(err).to.be.undefined
+        })
+    })
+    it('dispatches for all other errors', () => {
+      return confirmPasswordReset(dispatch, fakeFirebase, 'auth/user-not-found', 'error')
+        .catch((err) => {
+          expect(err.code).to.be.a.string
+        })
+    })
+  })
 })

tests/unit/compose.spec.js

@@ -108,6 +108,15 @@ describe('Compose', () => {
         expect(err).to.be.an.object
       }
     })
+
+    describe('confirmPasswordReset', () => {
+      try {
+        helpers.confirmPasswordReset({ code: 'test', password: 'test' })
+      } catch (err) {
+        expect(err).to.be.an.object
+      }
+    })
+
     describe('storage', () => {
       try {
         helpers.storage()