From 57f3134cd88c20057d3c8e547d8744d05435239f Mon Sep 17 00:00:00 2001
From: SleepWalker <mybox@udf.su>
Date: Sun, 22 May 2016 21:58:43 +0300
Subject: [PATCH] #97: frontend for repeat activation functionality

---
 src/components/auth/PanelTransition.jsx       |  2 +-
 src/components/auth/actions.js                | 12 ++-
 .../ResendActivation.intl.json                |  5 +
 .../resendActivation/ResendActivation.jsx     | 16 ++++
 .../resendActivation/ResendActivationBody.jsx | 40 ++++++++
 .../resendActivation/resendActivation.scss    |  8 ++
 src/i18n/en.json                              |  3 +
 src/i18n/ru.json                              |  3 +
 src/routes.js                                 |  2 +
 src/services/authFlow/ActivationState.js      |  5 +
 src/services/authFlow/AuthFlow.js             |  5 +
 .../authFlow/ResendActivationState.js         | 24 +++++
 .../services/authFlow/ActivationState.test.js |  9 ++
 .../authFlow/RecoverPasswordState.test.js     | 11 +++
 .../authFlow/ResendActivationState.test.js    | 95 +++++++++++++++++++
 15 files changed, 238 insertions(+), 2 deletions(-)
 create mode 100644 src/components/auth/resendActivation/ResendActivation.intl.json
 create mode 100644 src/components/auth/resendActivation/ResendActivation.jsx
 create mode 100644 src/components/auth/resendActivation/ResendActivationBody.jsx
 create mode 100644 src/components/auth/resendActivation/resendActivation.scss
 create mode 100644 src/services/authFlow/ResendActivationState.js
 create mode 100644 tests/services/authFlow/ResendActivationState.test.js

diff --git a/src/components/auth/PanelTransition.jsx b/src/components/auth/PanelTransition.jsx
index 661fbba..190595c 100644
--- a/src/components/auth/PanelTransition.jsx
+++ b/src/components/auth/PanelTransition.jsx
@@ -31,7 +31,7 @@ const changeContextSpringConfig = {stiffness: 500, damping: 20, precision: 0.5};
  */
 const contexts = [
     ['login', 'password', 'forgotPassword', 'recoverPassword'],
-    ['register', 'activation'],
+    ['register', 'activation', 'resendActivation'],
     ['changePassword'],
     ['permissions']
 ];
diff --git a/src/components/auth/actions.js b/src/components/auth/actions.js
index 4e59a52..39c758b 100644
--- a/src/components/auth/actions.js
+++ b/src/components/auth/actions.js
@@ -126,7 +126,17 @@ export function activate({key = ''}) {
 
             return dispatch(authenticate(resp.jwt));
         })
-        .catch(validationErrorsHandler(dispatch, '/reactivate'))
+        .catch(validationErrorsHandler(dispatch, '/resend-activation'))
+    );
+}
+
+export function resendActivation({email = ''}) {
+    return wrapInLoader((dispatch) =>
+        request.post(
+            '/api/signup/repeat-message',
+            {email}
+        )
+        .catch(validationErrorsHandler(dispatch))
     );
 }
 
diff --git a/src/components/auth/resendActivation/ResendActivation.intl.json b/src/components/auth/resendActivation/ResendActivation.intl.json
new file mode 100644
index 0000000..140dce7
--- /dev/null
+++ b/src/components/auth/resendActivation/ResendActivation.intl.json
@@ -0,0 +1,5 @@
+{
+    "title": "Did not received an E-mail",
+    "specifyYourEmail": "Please, enter an E-mail you've registered with and we will send you new activation code.",
+    "sendNewEmail": "Send new E-mail"
+}
diff --git a/src/components/auth/resendActivation/ResendActivation.jsx b/src/components/auth/resendActivation/ResendActivation.jsx
new file mode 100644
index 0000000..b14a1dc
--- /dev/null
+++ b/src/components/auth/resendActivation/ResendActivation.jsx
@@ -0,0 +1,16 @@
+import React from 'react';
+
+import { Button } from 'components/ui/form';
+import AuthTitle from 'components/auth/AuthTitle';
+
+import messages from './ResendActivation.intl.json';
+import Body from './ResendActivationBody';
+
+export default function ResendActivation() {
+    return {
+        Title: () => <AuthTitle title={messages.title} />,
+        Body,
+        Footer: () => <Button color="blue" label={messages.sendNewEmail} type="submit" />,
+        Links: () => null
+    };
+}
diff --git a/src/components/auth/resendActivation/ResendActivationBody.jsx b/src/components/auth/resendActivation/ResendActivationBody.jsx
new file mode 100644
index 0000000..4fa5f59
--- /dev/null
+++ b/src/components/auth/resendActivation/ResendActivationBody.jsx
@@ -0,0 +1,40 @@
+import React from 'react';
+
+import { FormattedMessage as Message } from 'react-intl';
+
+import { Input } from 'components/ui/form';
+import registerMessages from 'components/auth/register/Register.intl.json';
+
+import BaseAuthBody from 'components/auth/BaseAuthBody';
+import styles from './resendActivation.scss';
+import messages from './ResendActivation.intl.json';
+
+export default class ResendActivation extends BaseAuthBody {
+    static displayName = 'ResendActivation';
+    static panelId = 'resendActivation';
+    static hasGoBack = true;
+
+    autoFocusField = 'email';
+
+    render() {
+        return (
+            <div>
+                {this.renderErrors()}
+
+                <div className={styles.description}>
+                    <Message {...messages.specifyYourEmail} />
+                </div>
+
+                <div className={styles.formRow}>
+                    <Input {...this.bindField('email')}
+                        icon="envelope"
+                        color="blue"
+                        type="email"
+                        required
+                        placeholder={registerMessages.yourEmail}
+                    />
+                </div>
+            </div>
+        );
+    }
+}
diff --git a/src/components/auth/resendActivation/resendActivation.scss b/src/components/auth/resendActivation/resendActivation.scss
new file mode 100644
index 0000000..3ef7d57
--- /dev/null
+++ b/src/components/auth/resendActivation/resendActivation.scss
@@ -0,0 +1,8 @@
+@import '~components/ui/fonts.scss';
+
+.description {
+    font-family: $font-family-title;
+    margin: 5px 0 19px;
+    line-height: 1.4;
+    font-size: 16px;
+}
diff --git a/src/i18n/en.json b/src/i18n/en.json
index 0dfb641..bb5f336 100644
--- a/src/i18n/en.json
+++ b/src/i18n/en.json
@@ -57,6 +57,9 @@
     "components.auth.register.termsOfService": "terms of service",
     "components.auth.register.yourEmail": "Your E-mail",
     "components.auth.register.yourNickname": "Your nickname",
+    "components.auth.resendActivation.sendNewEmail": "Send new E-mail",
+    "components.auth.resendActivation.specifyYourEmail": "Please, enter an E-mail you've registered with and we will send you new activation code.",
+    "components.auth.resendActivation.title": "Did not received an E-mail",
     "components.contact.email": "E-mail",
     "components.contact.message": "Message",
     "components.contact.send": "Send",
diff --git a/src/i18n/ru.json b/src/i18n/ru.json
index 501fc8a..ccbc343 100644
--- a/src/i18n/ru.json
+++ b/src/i18n/ru.json
@@ -57,6 +57,9 @@
     "components.auth.register.termsOfService": "правилами сервиса",
     "components.auth.register.yourEmail": "Ваш E-mail",
     "components.auth.register.yourNickname": "Желаемый ник",
+    "components.auth.resendActivation.sendNewEmail": "Отправить новое письмо",
+    "components.auth.resendActivation.specifyYourEmail": "Укажите здесь ваш регистрационный E-mail адрес и мы вышлем на него новое письмо с кодом активации аккаунта",
+    "components.auth.resendActivation.title": "Не получил письмо",
     "components.contact.email": "E-mail",
     "components.contact.message": "Сообщение",
     "components.contact.send": "Отправить",
diff --git a/src/routes.js b/src/routes.js
index 31fe3da..2fb7a3e 100644
--- a/src/routes.js
+++ b/src/routes.js
@@ -17,6 +17,7 @@ import Register from 'components/auth/register/Register';
 import Login from 'components/auth/login/Login';
 import Permissions from 'components/auth/permissions/Permissions';
 import Activation from 'components/auth/activation/Activation';
+import ResendActivation from 'components/auth/resendActivation/ResendActivation';
 import Password from 'components/auth/password/Password';
 import ChangePassword from 'components/auth/changePassword/ChangePassword';
 import ForgotPassword from 'components/auth/forgotPassword/ForgotPassword';
@@ -55,6 +56,7 @@ export default function routesFactory(store) {
                 <Route path="/password" components={new Password()} {...startAuthFlow} />
                 <Route path="/register" components={new Register()} {...startAuthFlow} />
                 <Route path="/activation" components={new Activation()} {...startAuthFlow} />
+                <Route path="/resend-activation" components={new ResendActivation()} {...startAuthFlow} />
                 <Route path="/oauth/permissions" components={new Permissions()} {...startAuthFlow} />
                 <Route path="/oauth/finish" component={Finish} {...startAuthFlow} />
                 <Route path="/change-password" components={new ChangePassword()} {...startAuthFlow} />
diff --git a/src/services/authFlow/ActivationState.js b/src/services/authFlow/ActivationState.js
index ca35865..a894649 100644
--- a/src/services/authFlow/ActivationState.js
+++ b/src/services/authFlow/ActivationState.js
@@ -1,5 +1,6 @@
 import AbstractState from './AbstractState';
 import CompleteState from './CompleteState';
+import ResendActivationState from './ResendActivationState';
 
 export default class ActivationState extends AbstractState {
     enter(context) {
@@ -16,4 +17,8 @@ export default class ActivationState extends AbstractState {
         context.run('activate', payload)
             .then(() => context.setState(new CompleteState()));
     }
+
+    reject(context) {
+        context.setState(new ResendActivationState());
+    }
 }
diff --git a/src/services/authFlow/AuthFlow.js b/src/services/authFlow/AuthFlow.js
index 239d2fd..d39f559 100644
--- a/src/services/authFlow/AuthFlow.js
+++ b/src/services/authFlow/AuthFlow.js
@@ -5,6 +5,7 @@ import LoginState from './LoginState';
 import OAuthState from './OAuthState';
 import ForgotPasswordState from './ForgotPasswordState';
 import RecoverPasswordState from './RecoverPasswordState';
+import ResendActivationState from './ResendActivationState';
 
 // TODO: a way to unload service (when we are on account page)
 
@@ -99,6 +100,10 @@ export default class AuthFlow {
                 this.setState(new ForgotPasswordState());
                 break;
 
+            case '/resend-activation':
+                this.setState(new ResendActivationState());
+                break;
+
             case '/':
             case '/login':
             case '/password':
diff --git a/src/services/authFlow/ResendActivationState.js b/src/services/authFlow/ResendActivationState.js
new file mode 100644
index 0000000..4383200
--- /dev/null
+++ b/src/services/authFlow/ResendActivationState.js
@@ -0,0 +1,24 @@
+import AbstractState from './AbstractState';
+import CompleteState from './CompleteState';
+import ActivationState from './ActivationState';
+
+export default class ResendActivationState extends AbstractState {
+    enter(context) {
+        const {user} = context.getState();
+
+        if (user.isActive) {
+            context.setState(new CompleteState());
+        } else {
+            context.navigate('/repeat-message');
+        }
+    }
+
+    resolve(context, payload) {
+        context.run('resendActivation', payload)
+            .then(() => context.setState(new CompleteState()));
+    }
+
+    goBack(context) {
+        context.setState(new ActivationState());
+    }
+}
diff --git a/tests/services/authFlow/ActivationState.test.js b/tests/services/authFlow/ActivationState.test.js
index 2238441..a6e922f 100644
--- a/tests/services/authFlow/ActivationState.test.js
+++ b/tests/services/authFlow/ActivationState.test.js
@@ -1,5 +1,6 @@
 import ActivationState from 'services/authFlow/ActivationState';
 import CompleteState from 'services/authFlow/CompleteState';
+import ResendActivationState from 'services/authFlow/ResendActivationState';
 
 import { bootstrap, expectState, expectNavigate, expectRun } from './helpers';
 
@@ -83,4 +84,12 @@ describe('ActivationState', () => {
             return promise.catch(mock.verify.bind(mock));
         });
     });
+
+    describe('#reject', () => {
+        it('should transition to resend-activation', () => {
+            expectState(mock, ResendActivationState);
+
+            state.reject(context);
+        });
+    });
 });
diff --git a/tests/services/authFlow/RecoverPasswordState.test.js b/tests/services/authFlow/RecoverPasswordState.test.js
index da6da92..5e42b0c 100644
--- a/tests/services/authFlow/RecoverPasswordState.test.js
+++ b/tests/services/authFlow/RecoverPasswordState.test.js
@@ -84,6 +84,17 @@ describe('RecoverPasswordState', () => {
 
             return promise;
         });
+
+        it('should NOT transition to complete state on fail', () => {
+            const promise = Promise.reject();
+
+            mock.expects('run').returns(promise);
+            mock.expects('setState').never();
+
+            state.resolve(context);
+
+            return promise.catch(mock.verify.bind(mock));
+        });
     });
 
     describe('#reject', () => {
diff --git a/tests/services/authFlow/ResendActivationState.test.js b/tests/services/authFlow/ResendActivationState.test.js
new file mode 100644
index 0000000..714a7af
--- /dev/null
+++ b/tests/services/authFlow/ResendActivationState.test.js
@@ -0,0 +1,95 @@
+import ResendActivationState from 'services/authFlow/ResendActivationState';
+import CompleteState from 'services/authFlow/CompleteState';
+import ActivationState from 'services/authFlow/ActivationState';
+
+import { bootstrap, expectState, expectNavigate, expectRun } from './helpers';
+
+describe('ResendActivationState', () => {
+    let state;
+    let context;
+    let mock;
+
+    beforeEach(() => {
+        state = new ResendActivationState();
+
+        const data = bootstrap();
+        context = data.context;
+        mock = data.mock;
+    });
+
+    afterEach(() => {
+        mock.verify();
+    });
+
+    describe('#enter', () => {
+        it('should navigate to /resend-activation', () => {
+            context.getState.returns({
+                user: {
+                    isGuest: false,
+                    isActive: false
+                }
+            });
+
+            expectNavigate(mock, '/resend-activation');
+
+            state.enter(context);
+        });
+
+        it('should transition to complete state if account activated', () => {
+            context.getState.returns({
+                user: {
+                    isGuest: false,
+                    isActive: true
+                }
+            });
+
+            expectState(mock, CompleteState);
+
+            state.enter(context);
+        });
+    });
+
+    describe('#resolve', () => {
+        it('should call resendActivation with payload', () => {
+            const payload = {email: 'foo@bar.com'};
+
+            expectRun(
+                mock,
+                'resendActivation',
+                sinon.match.same(payload)
+            ).returns({then() {}});
+
+            state.resolve(context, payload);
+        });
+
+        it('should transition to complete state on success', () => {
+            const promise = Promise.resolve();
+
+            mock.expects('run').returns(promise);
+            expectState(mock, CompleteState);
+
+            state.resolve(context);
+
+            return promise;
+        });
+
+        it('should NOT transition to complete state on fail', () => {
+            const promise = Promise.reject();
+
+            mock.expects('run').returns(promise);
+            mock.expects('setState').never();
+
+            state.resolve(context);
+
+            return promise.catch(mock.verify.bind(mock));
+        });
+    });
+
+    describe('#goBack', () => {
+        it('should transition to resend-activation', () => {
+            expectState(mock, ActivationState);
+
+            state.goBack(context);
+        });
+    });
+});