From 923889645b48feceee1086e5002143631b7bcb1e Mon Sep 17 00:00:00 2001 From: Thais Siqueira Date: Wed, 15 Mar 2017 17:01:04 -0300 Subject: [#924] Updates recovery code on backup account submit with @anikarni --- web-ui/src/backup_account/backup_account.js | 2 + .../backup_account/backup_email/backup_email.js | 18 ++++- .../backup_email/backup_email.spec.js | 88 +++++++++++++++------- 3 files changed, 76 insertions(+), 32 deletions(-) (limited to 'web-ui/src') diff --git a/web-ui/src/backup_account/backup_account.js b/web-ui/src/backup_account/backup_account.js index ac218a39..19b7c19c 100644 --- a/web-ui/src/backup_account/backup_account.js +++ b/web-ui/src/backup_account/backup_account.js @@ -22,6 +22,8 @@ import a11y from 'react-a11y'; import App from 'src/common/app'; import PageWrapper from './page'; +require('es6-promise').polyfill(); + if (process.env.NODE_ENV === 'development') a11y(React); render( diff --git a/web-ui/src/backup_account/backup_email/backup_email.js b/web-ui/src/backup_account/backup_email/backup_email.js index 05cba35e..32d17935 100644 --- a/web-ui/src/backup_account/backup_email/backup_email.js +++ b/web-ui/src/backup_account/backup_email/backup_email.js @@ -15,11 +15,13 @@ * along with Pixelated. If not, see . */ +import 'isomorphic-fetch'; import React from 'react'; import { translate } from 'react-i18next'; import SubmitButton from 'src/common/submit_button/submit_button'; import InputField from 'src/common/input_field/input_field'; import validator from 'validator'; +import browser from 'helpers/browser'; import './backup_email.scss'; @@ -42,9 +44,19 @@ export class BackupEmail extends React.Component { submitHandler = (event) => { event.preventDefault(); - if (typeof this.props.onSubmit === 'function') { - this.props.onSubmit(); - } + + fetch('/backup-account', { + credentials: 'same-origin', + method: 'POST', + headers: { + 'Content-Type': 'application/json' + }, + body: JSON.stringify({ + csrftoken: [browser.getCookie('XSRF-TOKEN')] + }) + }); + + this.props.onSubmit(); } render() { diff --git a/web-ui/src/backup_account/backup_email/backup_email.spec.js b/web-ui/src/backup_account/backup_email/backup_email.spec.js index 8732003b..66dc8df6 100644 --- a/web-ui/src/backup_account/backup_email/backup_email.spec.js +++ b/web-ui/src/backup_account/backup_email/backup_email.spec.js @@ -1,90 +1,120 @@ import { shallow } from 'enzyme'; import expect from 'expect'; import React from 'react'; +import fetchMock from 'fetch-mock'; import { BackupEmail } from 'src/backup_account/backup_email/backup_email'; +import browser from 'helpers/browser'; describe('BackupEmail', () => { - let page; - let mockTranslations; + let backupEmail; + let mockOnSubmit; beforeEach(() => { - mockTranslations = key => key; - page = shallow(); + mockOnSubmit = expect.createSpy(); + const mockTranslations = key => key; + backupEmail = shallow(); }); it('renders backup email title', () => { - expect(page.find('h1').text()).toEqual('backup-account.backup-email.title'); + expect(backupEmail.find('h1').text()).toEqual('backup-account.backup-email.title'); }); it('renders backup account email input field', () => { - expect(page.find('InputField').props().name).toEqual('email'); + expect(backupEmail.find('InputField').props().name).toEqual('email'); }); it('renders backup account submit button', () => { - expect(page.find('SubmitButton').props().buttonText).toEqual('backup-account.backup-email.button'); - }); - - it('form submit should call parameter custom submit', () => { - const mockOnSubmit = expect.createSpy(); - const event = { preventDefault() {} }; - page = shallow(); - - page.instance().submitHandler(event); - expect(mockOnSubmit).toHaveBeenCalled(); + expect(backupEmail.find('SubmitButton').props().buttonText).toEqual('backup-account.backup-email.button'); }); describe('Email validation', () => { - let pageInstance; + let backupEmailInstance; beforeEach(() => { - pageInstance = page.instance(); + backupEmailInstance = backupEmail.instance(); }); it('verify initial state', () => { - expect(pageInstance.state.error).toEqual(''); - expect(page.find('SubmitButton').props().disabled).toEqual(true); + expect(backupEmailInstance.state.error).toEqual(''); + expect(backupEmail.find('SubmitButton').props().disabled).toBe(true); }); context('with invalid email', () => { beforeEach(() => { - pageInstance.validateEmail({ target: { value: 'test' } }); + backupEmailInstance.validateEmail({ target: { value: 'test' } }); }); it('sets error in state', () => { - expect(pageInstance.state.error).toEqual('backup-account.backup-email.error.invalid-email'); + expect(backupEmailInstance.state.error).toEqual('backup-account.backup-email.error.invalid-email'); }); it('disables submit button', () => { - expect(page.find('SubmitButton').props().disabled).toEqual(true); + expect(backupEmail.find('SubmitButton').props().disabled).toBe(true); }); }); context('with valid email', () => { beforeEach(() => { - pageInstance.validateEmail({ target: { value: 'test@test.com' } }); + backupEmailInstance.validateEmail({ target: { value: 'test@test.com' } }); }); it('does not set error in state', () => { - expect(pageInstance.state.error).toEqual(''); + expect(backupEmailInstance.state.error).toEqual(''); }); it('submit button is enabled', () => { - expect(page.find('SubmitButton').props().disabled).toEqual(false); + expect(backupEmail.find('SubmitButton').props().disabled).toBe(false); }); }); context('with empty email', () => { beforeEach(() => { - pageInstance.validateEmail({ target: { value: '' } }); + backupEmailInstance.validateEmail({ target: { value: '' } }); }); it('not set error in state', () => { - expect(pageInstance.state.error).toEqual(''); + expect(backupEmailInstance.state.error).toEqual(''); }); it('disables submit button', () => { - expect(page.find('SubmitButton').props().disabled).toEqual(true); + expect(backupEmail.find('SubmitButton').props().disabled).toBe(true); }); }); }); + + describe('Submit', () => { + let preventDefaultSpy; + + beforeEach(() => { + preventDefaultSpy = expect.createSpy(); + expect.spyOn(browser, 'getCookie').andReturn('abc123'); + + fetchMock.post('/backup-account', 204); + backupEmail.find('form').simulate('submit', { preventDefault: preventDefaultSpy }); + }); + + it('posts backup email', () => { + expect(fetchMock.called('/backup-account')).toBe(true, 'Backup account POST was not called'); + }); + + it('sends csrftoken as content', () => { + expect(fetchMock.lastOptions('/backup-account').body).toContain('"csrftoken":["abc123"]'); + }); + + it('sends content-type header', () => { + expect(fetchMock.lastOptions('/backup-account').headers['Content-Type']).toEqual('application/json'); + }); + + it('sends same origin headers', () => { + expect(fetchMock.lastOptions('/backup-account').credentials).toEqual('same-origin'); + }); + + it('prevents default call to refresh page', () => { + expect(preventDefaultSpy).toHaveBeenCalled(); + }); + + it('calls onSubmit from props', () => { + expect(mockOnSubmit).toHaveBeenCalled(); + }); + }); }); -- cgit v1.2.3