(PC-1501): Fix redirection to next step

src/components/pages/Offer/RawOffer.js

@@ -182,17 +182,16 @@ class RawOffer extends Component {
       config: { method },
       payload,
     } = action
-    const { history, offer, venue } = this.props
-    const eventOrThing = payload.datum
+
+    const { history } = this.props
+    const offer = payload.datum
 
     if (method === 'PATCH') {
       history.push(`/offres/${offer.id}`)
       return
     }
 
     if (method === 'POST') {
-      const { offers } = eventOrThing || {}
-      const offer = offers && offers.find(o => o.venueId === get(venue, 'id'))
       if (!offer) {
         console.warn(
           'Something wrong with returned data, we should retrieve the created offer here'

src/components/pages/Offer/tests/RawOffer.spec.jsx

@@ -39,6 +39,80 @@ describe('src | components | pages | Offer | RawOffer ', () => {
     })
   })
 
+  describe('handleSuccess', () => {
+    describe('when the offer is successfully modified', () => {
+      it('should redirect to offer page', () => {
+        // given
+        const historyPush = jest.fn()
+        const initialProps = {
+          location: {
+            search: '?lieu=AQ',
+          },
+          match: {
+            params: {},
+          },
+          currentUser: {
+            isAdmin: false,
+          },
+          query: {
+            parse: () => ({ lieu: 'AQ' }),
+          },
+          dispatch: dispatchMock,
+          history: {
+            push: historyPush,
+          },
+        }
+
+        const wrapper = shallow(<RawOffer {...initialProps} />)
+
+        // when
+        let offre = { id: 'SN' }
+        let action = { config: { method: 'PATCH' }, payload: { datum: offre } }
+        wrapper.instance().handleSuccess({}, action)
+
+        // then
+        expect(historyPush).toHaveBeenCalledWith('/offres/SN')
+      })
+    })
+
+    describe('when the offer is successfully modified', () => {
+      it('should redirect to gestion page', () => {
+        // given
+        const historyPush = jest.fn()
+        const initialProps = {
+          location: {
+            search: '?lieu=AQ',
+          },
+          match: {
+            params: {
+              offerId: 'nouveau',
+            },
+          },
+          currentUser: {
+            isAdmin: false,
+          },
+          query: {
+            parse: () => ({ lieu: 'AQ' }),
+          },
+          dispatch: dispatchMock,
+          history: {
+            push: historyPush,
+          },
+        }
+
+        const wrapper = shallow(<RawOffer {...initialProps} />)
+
+        // when
+        let offre = { id: 'SN' }
+        let action = { config: { method: 'POST' }, payload: { datum: offre } }
+        wrapper.instance().handleSuccess({}, action)
+
+        // then
+        expect(historyPush).toHaveBeenCalledWith('/offres/SN?gestion')
+      })
+    })
+  })
+
   describe('render', () => {
     describe('MediationsManager', () => {
       it("should be displayed when it's not a new offer", () => {