feature(read-book): prevent redirect on update profile failure

client/__tests__/actions/userActions.spec.js

@@ -41,7 +41,7 @@ describe('user Actions', () => {
       });
       const expectedActions = [];
       const store = mockStore({});
-      return store.dispatch(updateProfile({})).then(() => {
+      return store.dispatch(updateProfile({})).catch(() => {
         expect(store.getActions()).toEqual(expectedActions);
         expect(Notify.error).toHaveBeenCalled();
       });

client/__tests__/components/Dashboard/PdfViewer.spec.jsx

@@ -54,6 +54,16 @@ describe('PdfViewer Component', () => {
     expect(wrapper.state().page).toBe(1);
   });
 
+  it('should call onDocumentError if file load encounters an error', () => {
+    const wrapper = setUp();
+    const onPageCompleteSpy = jest.spyOn(
+      wrapper.instance(), 'onDocumentError'
+    );
+    wrapper.instance().onDocumentError(new Error('failed to load pdf'));
+    expect(onPageCompleteSpy).toHaveBeenCalledTimes(1);
+    expect(wrapper.state().fileError).toBe(true);
+  });
+
   it('should call setPage when page is manually inserted', () => {
     const wrapper = setUp();
     const setPageSpy = jest.spyOn(

client/__tests__/components/Dashboard/updateProfile.spec.jsx

@@ -8,7 +8,7 @@ import { mockStoreData } from '../../__mocks__/mockData';
 
 let props = {
   ...mockStoreData.authReducer,
-  updateProfile: jest.fn(),
+  updateProfile: jest.fn(() => Promise.resolve(1)),
 };
 
 const middleware = [thunk];

client/actions/updateProfile.js

@@ -22,7 +22,10 @@ const updateProfile = profile => dispatch => (
       Notify.success(response.data.message);
       return dispatch(loginUser(response.data));
     })
-    .catch(error => reportNetworkError(error))
+    .catch((error) => {
+      reportNetworkError(error);
+      return Promise.reject(error);
+    })
 );
 
 export default updateProfile;

client/components/Dashboard/UpdateProfile.jsx

@@ -46,8 +46,10 @@ export class UpdateProfile extends Component {
    */
   handleSubmit = (event) => {
     event.preventDefault();
-    this.props.updateProfile(this.state);
-    this.setState(() => ({ redirect: true }));
+    this.props.updateProfile(this.state)
+      .then(() => this.setState(() => ({ redirect: true })))
+      .catch(() => {})
+    ;
   }
 
   /**