Skip to content
This repository has been archived by the owner on Jan 2, 2020. It is now read-only.

Commit

Permalink
Merge pull request #1028 from pixelated/forgot-password-page
Browse files Browse the repository at this point in the history
Adds admin code form and logic for next step of recovery form
  • Loading branch information
anikarni authored Mar 28, 2017
2 parents a9462a5 + c7d135c commit 444b399
Show file tree
Hide file tree
Showing 10 changed files with 226 additions and 12 deletions.
13 changes: 12 additions & 1 deletion web-ui/app/locales/en_US/translation.json
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,18 @@
"tags": "Tags"
},
"account-recovery": {
"page-title": "Pixelated Account Recovery"
"page-title": "Pixelated Account Recovery",
"admin-form": {
"title": "Contact your account administrator and ask for their part of the recovery code",
"tip1": "The safest way to do this is in person.",
"tip2": "You can call or text if you need to.",
"tip3": "Don't ever ask for it via email.",
"input-label": "type here admin's backup code",
"button": "Next"
},
"user-form": {
"title": "Remember your backup account?"
}
},
"backup-account": {
"page-title": "Pixelated Backup Account",
Expand Down
45 changes: 45 additions & 0 deletions web-ui/src/account_recovery/forms/admin_recovery_code_form.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,45 @@
/*
* Copyright (c) 2017 ThoughtWorks, Inc.
*
* Pixelated is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as published by
* the Free Software Foundation, either version 3 of the License, or
* (at your option) any later version.
*
* Pixelated is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU Affero General Public License for more details.
*
* You should have received a copy of the GNU Affero General Public License
* along with Pixelated. If not, see <http://www.gnu.org/licenses/>.
*/

import React from 'react';
import { translate } from 'react-i18next';

import InputField from 'src/common/input_field/input_field';
import SubmitButton from 'src/common/submit_button/submit_button';

import './admin_recovery_code_form.scss';


export const AdminRecoveryCodeForm = ({ t, next }) => (
<form className='admin-code-form' onSubmit={next}>
<h1>{t('account-recovery.admin-form.title')}</h1>
<ul>
<li>{t('account-recovery.admin-form.tip1')}</li>
<li>{t('account-recovery.admin-form.tip2')}</li>
<li>{t('account-recovery.admin-form.tip3')}</li>
</ul>
<InputField name='admin-code' label={t('account-recovery.admin-form.input-label')} />
<SubmitButton buttonText={t('account-recovery.admin-form.button')} />
</form>
);

AdminRecoveryCodeForm.propTypes = {
t: React.PropTypes.func.isRequired,
next: React.PropTypes.func.isRequired
};

export default translate('', { wait: true })(AdminRecoveryCodeForm);
Empty file.
38 changes: 38 additions & 0 deletions web-ui/src/account_recovery/forms/admin_recovery_code_form.spec.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,38 @@
import { shallow } from 'enzyme';
import expect from 'expect';
import React from 'react';
import { AdminRecoveryCodeForm } from 'src/account_recovery/forms/admin_recovery_code_form';

describe('AdminRecoveryCodeForm', () => {
let adminRecoveryCodeForm;
let mockNext;

beforeEach(() => {
const mockTranslations = key => key;
mockNext = expect.createSpy();
adminRecoveryCodeForm = shallow(
<AdminRecoveryCodeForm t={mockTranslations} next={mockNext} />
);
});

it('renders title for admin recovery code', () => {
expect(adminRecoveryCodeForm.find('h1').text()).toEqual('account-recovery.admin-form.title');
});

it('renders tips for retrieving recovery code', () => {
expect(adminRecoveryCodeForm.find('li').length).toEqual(3);
});

it('renders input field for admin code', () => {
expect(adminRecoveryCodeForm.find('InputField').props().name).toEqual('admin-code');
});

it('renders button for next step', () => {
expect(adminRecoveryCodeForm.find('SubmitButton').props().buttonText).toEqual('account-recovery.admin-form.button');
});

it('submits form to next step', () => {
adminRecoveryCodeForm.find('form').simulate('submit');
expect(mockNext).toHaveBeenCalled();
});
});
31 changes: 31 additions & 0 deletions web-ui/src/account_recovery/forms/user_recovery_code_form.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,31 @@
/*
* Copyright (c) 2017 ThoughtWorks, Inc.
*
* Pixelated is free software: you can redistribute it and/or modify
* it under the terms of the GNU Affero General Public License as published by
* the Free Software Foundation, either version 3 of the License, or
* (at your option) any later version.
*
* Pixelated is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU Affero General Public License for more details.
*
* You should have received a copy of the GNU Affero General Public License
* along with Pixelated. If not, see <http://www.gnu.org/licenses/>.
*/

import React from 'react';
import { translate } from 'react-i18next';

export const UserRecoveryCodeForm = ({ t }) => (
<form>
<h1>{t('account-recovery.user-form.title')}</h1>
</form>
);

UserRecoveryCodeForm.propTypes = {
t: React.PropTypes.func.isRequired
};

export default translate('', { wait: true })(UserRecoveryCodeForm);
19 changes: 19 additions & 0 deletions web-ui/src/account_recovery/forms/user_recovery_code_form.spec.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
import { shallow } from 'enzyme';
import expect from 'expect';
import React from 'react';
import { UserRecoveryCodeForm } from 'src/account_recovery/forms/user_recovery_code_form';

describe('UserRecoveryCodeForm', () => {
let userRecoveryCodeForm;

beforeEach(() => {
const mockTranslations = key => key;
userRecoveryCodeForm = shallow(
<UserRecoveryCodeForm t={mockTranslations} />
);
});

it('renders title for user recovery code', () => {
expect(userRecoveryCodeForm.find('h1').text()).toEqual('account-recovery.user-form.title');
});
});
49 changes: 39 additions & 10 deletions web-ui/src/account_recovery/page.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,22 +18,51 @@
import React from 'react';
import { translate } from 'react-i18next';
import DocumentTitle from 'react-document-title';
import Footer from 'src/common/footer/footer';
import Header from 'src/common/header/header';
import AdminRecoveryCodeForm from 'src/account_recovery/forms/admin_recovery_code_form';
import UserRecoveryCodeForm from 'src/account_recovery/forms/user_recovery_code_form';
import Footer from 'src/common/footer/footer';

import 'font-awesome/scss/font-awesome.scss';
import './page.scss';


export const Page = ({ t }) => (
<DocumentTitle title={t('account-recovery.page-title')}>
<div className='page'>
<Header />
<section />
<Footer />
</div>
</DocumentTitle>
);
export class Page extends React.Component {

constructor(props) {
super(props);
this.state = { step: 0 };
}

nextStep = (event) => {
event.preventDefault();
this.setState({ step: this.state.step + 1 });
}

steps = {
0: <AdminRecoveryCodeForm next={this.nextStep} />,
1: <UserRecoveryCodeForm />
}

mainContent = () => this.steps[this.state.step];

render() {
const t = this.props.t;
return (
<DocumentTitle title={t('account-recovery.page-title')}>
<div className='page'>
<Header />
<section>
<div className='container'>
{this.mainContent()}
</div>
</section>
<Footer />
</div>
</DocumentTitle>
);
}
}

Page.propTypes = {
t: React.PropTypes.func.isRequired
Expand Down
2 changes: 2 additions & 0 deletions web-ui/src/account_recovery/page.scss
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,8 @@ a {
background: $white;
margin: 3% auto;
box-shadow: 0 2px 3px 0 $shadow;
width: 84%;
padding: 6% 5%;
}

.page {
Expand Down
32 changes: 32 additions & 0 deletions web-ui/src/account_recovery/page.spec.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,12 @@
import { shallow } from 'enzyme';
import expect from 'expect';
import React from 'react';

import { Page } from 'src/account_recovery/page';
import Header from 'src/common/header/header';
import AdminRecoveryCodeForm from 'src/account_recovery/forms/admin_recovery_code_form';
import UserRecoveryCodeForm from 'src/account_recovery/forms/user_recovery_code_form';
import Footer from 'src/common/footer/footer';

describe('Account Recovery Page', () => {
let page;
Expand All @@ -14,4 +19,31 @@ describe('Account Recovery Page', () => {
it('renders account recovery page title', () => {
expect(page.props().title).toEqual('account-recovery.page-title');
});

it('renders header', () => {
expect(page.find(Header).length).toEqual(1);
});

it('renders footer', () => {
expect(page.find(Footer).length).toEqual(1);
});

context('main content', () => {
let pageInstance;

beforeEach(() => {
pageInstance = page.instance();
});

it('renders admin recovery code form as default form', () => {
expect(page.find(AdminRecoveryCodeForm).length).toEqual(1);
});

it('renders user recovery code form when admin code submitted', () => {
pageInstance.nextStep({ preventDefault: () => {} });

expect(page.find(AdminRecoveryCodeForm).length).toEqual(0);
expect(page.find(UserRecoveryCodeForm).length).toEqual(1);
});
});
});
9 changes: 8 additions & 1 deletion web-ui/test/integration/translations.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,15 @@ import testI18n from './i18n';

describe('Translations', () => {
context('Account Recovery Page', () => {
it('translates all key', () => {
it('translates all keys on first step', () => {
const app = mount(<App i18n={testI18n} child={<AccountRecoveryPage />} />);
expect(app.text()).toNotContain('untranslated', 'Unstranslated message found in the text: ' + app.text());
});

it('translates all keys on second step', () => {
const app = mount(<App i18n={testI18n} child={<AccountRecoveryPage />} />);
app.find('form.admin-code-form').simulate('submit');

expect(app.text()).toNotContain('untranslated', 'Unstranslated message found in the text: ' + app.text());
});
});
Expand Down

0 comments on commit 444b399

Please sign in to comment.