Support redirect to previous location after login, test

src/actions/setLastLocationAction.js

@@ -0,0 +1,5 @@
+import { SET_LAST_LOCATION } from '../types'
+
+export let setLastLocation = props => dispatch => {
+  dispatch({ type: SET_LAST_LOCATION, payload: props.location.pathname })
+}

src/containers/LogIn.js

@@ -17,7 +17,7 @@ export class LogIn extends Component {
     await this.props
       .postLogInInfo({ email, password })
       .then(() => {
-        this.props.history.push('/')
+        this.props.history.push(this.props.lastLocation)
         iziToast.show({
           theme: 'light',
           title: 'Success',
@@ -92,7 +92,7 @@ export class LogIn extends Component {
   }
 }
 
-const mapStateToProps = store => ({ loggedInUser: store.loggedInUser })
+const mapStateToProps = store => ({ loggedInUser: store.loggedInUser, lastLocation: store.lastLocation })
 export default connect(
   mapStateToProps,
   { postLogInInfo }

src/containers/ProjectsList.js

@@ -2,6 +2,7 @@ import React, { Component, Fragment } from 'react'
 import { Header, Card, Grid } from 'semantic-ui-react'
 import { connect } from 'react-redux'
 import { fetchProjects } from '../actions/getProjectsAction'
+import { setLastLocation } from '../actions/setLastLocationAction'
 import { Link } from 'react-router-dom'
 import Select from 'react-select'
 import Project from '../components/Project'
@@ -37,7 +38,8 @@ export class ProjectsList extends Component {
     } else {
       this.paginateProjects(this.props.projects)
     }
-  };
+    this.props.setLastLocation(this.props)
+  }
 
   componentWillReceiveProps (nextProps) {
     if (
@@ -223,5 +225,5 @@ export class ProjectsList extends Component {
 const mapStateToProps = store => ({ projects: store.projects, error: store.error })
 export default connect(
   mapStateToProps,
-  { fetchProjects }
+  { fetchProjects, setLastLocation }
 )(ProjectsList)

src/containers/UsersList.js

@@ -4,6 +4,7 @@ import Paginate from '../components/Paginate'
 import PaginationLinks from '../components/PaginationLinks'
 import { connect } from 'react-redux'
 import { fetchUsers } from '../actions/getUsersAction'
+import { setLastLocation } from '../actions/setLastLocationAction'
 import User from '../components/User'
 import '../assets/UsersList.css'
 export class UsersList extends Component {
@@ -25,6 +26,7 @@ export class UsersList extends Component {
     } else {
       this.normalizeUsers(this.props.users)
     }
+    this.props.setLastLocation(this.props)
   }
 
   componentWillReceiveProps (nextProps) {
@@ -105,5 +107,5 @@ export class UsersList extends Component {
 const mapStateToProps = store => ({ users: store.users })
 export default connect(
   mapStateToProps,
-  { fetchUsers }
+  { fetchUsers, setLastLocation }
 )(UsersList)

src/index.js

@@ -18,9 +18,14 @@ render(
       <Navbar />
       <Switch>
         <Route path='/' exact component={Homepage} />
-        <Container className='main-content'>
+        <Container>
           <Route path='/users' component={UsersList} />
-          <Route path='/login' component={LogIn} />
+          <Route
+            path='/login'
+            render={props => {
+              return <LogIn {...props} lastLocation={props} />
+            }}
+          />
           <Route path='/signup' component={SignUp} />
           <Route path='/projects' component={ProjectsList} />
         </Container>

src/reducers/initialState.js

@@ -3,5 +3,6 @@ export default {
   loggedInUser: {},
   signedUpUser: {},
   projects: [],
-  error: []
+  error: [],
+  lastLocation: ''
 }

src/reducers/lastLocationReducer.js

@@ -0,0 +1,13 @@
+import { SET_LAST_LOCATION } from '../types'
+import initialState from './initialState'
+
+const lastLocationReducer = (state = initialState.lastLocation, action) => {
+  switch (action.type) {
+    case SET_LAST_LOCATION:
+      return action.payload
+    default:
+      return state
+  }
+}
+
+export default lastLocationReducer

src/store/index.js

@@ -5,14 +5,16 @@ import projects from '../reducers/projectsReducer'
 import loggedInUser from '../reducers/loggedInUserReducer'
 import signedUpUser from '../reducers/signedUpUserReducer'
 import error from '../reducers/errorReducer'
+import lastLocation from '../reducers/lastLocationReducer'
 
 const composeEnhancers = window.__REDUX_DEVTOOLS_EXTENSION_COMPOSE__ || compose
 const rootReducer = combineReducers({
   users,
   projects,
   loggedInUser,
   signedUpUser,
-  error
+  error,
+  lastLocation
 })
 
 export default createStore(

src/tests/actions/setLastLocationAction.test.js

@@ -0,0 +1,20 @@
+import thunk from 'redux-thunk'
+import configureMockStore from 'redux-mock-store'
+import { SET_LAST_LOCATION } from '../../types'
+import { setLastLocation } from '../../actions/setLastLocationAction'
+
+const middlewares = [thunk]
+const mockStore = configureMockStore(middlewares)
+let store
+
+describe('setLastLocation action', () => {
+  beforeEach(() => {
+    store = mockStore({})
+  })
+
+  it('sets last location to /users', () => {
+    let props = { location: { pathname: '/users' } }
+    store.dispatch(setLastLocation(props))
+    expect(store.getActions()).toEqual([{ type: SET_LAST_LOCATION, payload: '/users' }])
+  })
+})

src/tests/containers/ProjectsList.test.js

@@ -18,7 +18,8 @@ describe('ProjectsList', () => {
         }, 300)
       }),
     filteredProjectsList: null,
-    error: false
+    error: false,
+    setLastLocation: () => {}
   }
   wrapper = mount(
     <StaticRouter context={context}>
@@ -51,6 +52,7 @@ describe('ProjectsList', () => {
       <ProjectsList
         projects={paginatedProjectsFixture}
         fetchProjects={() => {}}
+        setLastLocation={() => {}}
       />
     )
     wrapper.setState(
@@ -89,15 +91,15 @@ describe('ProjectsList', () => {
   it("shouldn't render a Project component without projects", () => {
     const wrapper = mount(
       <StaticRouter context={context}>
-        <ProjectsList projects={{ 1: [] }} fetchProjects={() => {}} />
+        <ProjectsList projects={{ 1: [] }} fetchProjects={() => {}} setLastLocation={() => {}} />
       </StaticRouter>
     )
     expect(wrapper.find('Project')).toHaveLength(0)
   })
 
   it('should test componentWillReceiveProps', () => {
     const wrapper = shallow(
-      <ProjectsList projects={[]} fetchProjects={() => {}} />
+      <ProjectsList projects={[]} fetchProjects={() => {}} setLastLocation={() => {}} />
     )
     wrapper.setProps({ projects: [{ id: 1, languages: [] }] })
     expect(wrapper.instance().state.projects).toEqual({
@@ -107,7 +109,7 @@ describe('ProjectsList', () => {
 
   it('should call normalizeFilteredProjects', () => {
     const wrapper = shallow(
-      <ProjectsList projects={[]} fetchProjects={() => {}} />
+      <ProjectsList projects={[]} fetchProjects={() => {}} setLastLocation={() => {}} />
     )
     wrapper.setProps({ projects: projectsFixture })
     expect(wrapper.instance().state.projects).toEqual(
@@ -131,7 +133,7 @@ describe('ProjectsList', () => {
 
   it('adds error to the state if fetchProjects fails', async () => {
     const wrapper = shallow(
-      <ProjectsList projects={[]} error={[]} fetchProjects={() => {}} />
+      <ProjectsList projects={[]} error={[]} fetchProjects={() => {}} setLastLocation={() => {}} />
     )
     await wrapper.instance().componentDidMount()
     expect(wrapper.state().error).toEqual(true)

src/tests/containers/UserList.test.js

@@ -18,6 +18,7 @@ describe('UsersList', () => {
             }, 300)
           })
         }
+        setLastLocation={() => {}}
       />
     </StaticRouter>
   )
@@ -66,20 +67,20 @@ describe('UsersList', () => {
   it("shouldn't render a Project component without users", () => {
     const wrapper = mount(
       <StaticRouter context={context}>
-        <UsersList users={[]} fetchUsers={() => {}} />
+        <UsersList users={[]} fetchUsers={() => {}} setLastLocation={() => {}} />
       </StaticRouter>
     )
     expect(wrapper.find('User')).toHaveLength(0)
   })
 
   it('should test componentWillReceiveProps', () => {
-    const wrapper = shallow(<UsersList users={[]} fetchUsers={() => {}} />)
+    const wrapper = shallow(<UsersList users={[]} fetchUsers={() => {}} setLastLocation={() => {}} />)
     wrapper.setProps({ users: ['something'] })
     expect(wrapper.instance().state.users).toEqual({ '1': ['something'] })
   })
 
   it('should test componentWillReceiveProps', () => {
-    const wrapper = shallow(<UsersList users={usersFixture} fetchUsers={() => {}} />)
+    const wrapper = shallow(<UsersList users={usersFixture} fetchUsers={() => {}} setLastLocation={() => {}} />)
     wrapper.setProps(usersFixture)
     expect(wrapper.instance().state.users[1][0]).toEqual(usersFixture[0])
   })

src/tests/reducers/lastLocationReducer.test.js

@@ -0,0 +1,17 @@
+import lastLocationReducer from '../../reducers/lastLocationReducer'
+import { SET_LAST_LOCATION } from '../../types'
+
+describe('reduces a user', () => {
+  it('defaults to empty projects if none are passed in', () => {
+    expect(lastLocationReducer(undefined, {})).toEqual('')
+  })
+
+  it('reduces the signed in user', () => {
+    expect(
+      lastLocationReducer([], {
+        type: SET_LAST_LOCATION,
+        payload: { lastLocation: '/users' }
+      })
+    ).toEqual({ lastLocation: '/users' })
+  })
+})

src/tests/store/error.test.js

@@ -7,12 +7,6 @@ describe('Store', () => {
       message: 'Network Error'
     })
 
-    expect(store.getState()).toEqual({
-      users: [],
-      projects: [],
-      loggedInUser: {},
-      signedUpUser: {},
-      error: ['Network Error']
-    })
+    expect(store.getState().error).toEqual(['Network Error'])
   })
 })

src/tests/store/store.test.js

@@ -12,7 +12,8 @@ describe('Store', () => {
       loggedInUser: {},
       signedUpUser: {},
       error: [],
-      projects: []
+      projects: [],
+      lastLocation: ''
     })
   })
 })

src/types/index.js

@@ -3,3 +3,4 @@ export const GET_PROJECTS = 'GET_PROJECTS'
 export const POST_LOGIN_INFO = 'POST_LOGIN_INFO'
 export const POST_SIGNUP_INFO = 'POST_SIGNUP_INFO'
 export const FETCH_PROJECTS_FAILURE = 'FETCH_PROJECTS_FAILURE'
+export const SET_LAST_LOCATION = 'SET_LAST_LOCATION'