commencer: fix invalid URL on test procedures

Fix #3693

app/controllers/users/commencer_controller.rb

@@ -42,7 +42,7 @@ module Users
 
     def store_user_location!
       procedure = Procedure.find_by(path: params[:path])
-      store_location_for(:user, commencer_path(path: procedure.path))
+      store_location_for(:user, helpers.procedure_lien(procedure))
     end
   end
 end

spec/controllers/users/commencer_controller_spec.rb

@@ -18,7 +18,7 @@ describe Users::CommencerController, type: :controller do
       end
     end
 
-    context 'when the path is for a non-published procedure' do
+    context 'when the path is for a draft procedure' do
       let(:path) { draft_procedure.path }
 
       it 'redirects with an error message' do
@@ -66,24 +66,50 @@ describe Users::CommencerController, type: :controller do
   end
 
   describe '#sign_in' do
-    subject { get :sign_in, params: { path: published_procedure.path } }
+    context 'for a published procedure' do
+      subject { get :sign_in, params: { path: published_procedure.path } }
 
-    it 'set the path to return after sign-in to the dossier creation path' do
-      subject
-      expect(controller.stored_location_for(:user)).to eq(commencer_path(path: published_procedure.path))
+      it 'set the path to return after sign-in to the procedure start page' do
+        subject
+        expect(controller.stored_location_for(:user)).to eq(commencer_path(path: published_procedure.path))
+      end
+
+      it { expect(subject).to redirect_to(new_user_session_path) }
     end
 
-    it { expect(subject).to redirect_to(new_user_session_path) }
+    context 'for a draft procedure' do
+      subject { get :sign_in, params: { path: draft_procedure.path } }
+
+      it 'set the path to return after sign-in to the draft procedure start page' do
+        subject
+        expect(controller.stored_location_for(:user)).to eq(commencer_test_path(path: draft_procedure.path))
+      end
+
+      it { expect(subject).to redirect_to(new_user_session_path) }
+    end
   end
 
   describe '#sign_up' do
-    subject { get :sign_up, params: { path: published_procedure.path } }
+    context 'for a published procedure' do
+      subject { get :sign_up, params: { path: published_procedure.path } }
 
-    it 'set the path to return after sign-up to the dossier creation path' do
-      subject
-      expect(controller.stored_location_for(:user)).to eq(commencer_path(path: published_procedure.path))
+      it 'set the path to return after sign-up to the procedure start page' do
+        subject
+        expect(controller.stored_location_for(:user)).to eq(commencer_path(path: published_procedure.path))
+      end
+
+      it { expect(subject).to redirect_to(new_user_registration_path) }
     end
 
-    it { expect(subject).to redirect_to(new_user_registration_path) }
+    context 'for a draft procedure' do
+      subject { get :sign_up, params: { path: draft_procedure.path } }
+
+      it 'set the path to return after sign-up to the draft procedure start page' do
+        subject
+        expect(controller.stored_location_for(:user)).to eq(commencer_test_path(path: draft_procedure.path))
+      end
+
+      it { expect(subject).to redirect_to(new_user_registration_path) }
+    end
   end
 end