Fix initial implementation to actually render the unsubscribe page

Signed-off-by: Rémy Coutable <remy@rymai.me>

1 files changed, 10 insertions, 4 deletions

diff --git a/spec/controllers/sent_notifications_controller_spec.rb b/spec/controllers/sent_notifications_controller_spec.rb
index 4e75372ffb2..191e290a118 100644
--- a/spec/controllers/sent_notifications_controller_spec.rb
+++ b/spec/controllers/sent_notifications_controller_spec.rb
@@ -41,7 +41,7 @@ describe SentNotificationsController, type: :controller do
         end
 
         it 'redirects to the login page' do
-          expect(response).to redirect_to(new_user_session_path)
+          expect(response).to render_template :unsubscribe
         end
       end
     end
@@ -83,19 +83,25 @@ describe SentNotificationsController, type: :controller do
       end
 
       context 'when the force param is not passed' do
+        let(:merge_request) do
+          create(:merge_request, source_project: project, author: user) do |merge_request|
+            merge_request.subscriptions.create(user: user, subscribed: true)
+          end
+        end
+        let(:sent_notification) { create(:sent_notification, noteable: merge_request, recipient: user) }
         before { get(:unsubscribe, id: sent_notification.reply_key) }
 
         it 'unsubscribes the user' do
-          expect(issue.subscribed?(user)).to be_falsey
+          expect(merge_request.subscribed?(user)).to be_falsey
         end
 
         it 'sets the flash message' do
           expect(controller).to set_flash[:notice].to(/unsubscribed/).now
         end
 
-        it 'redirects to the issue page' do
+        it 'redirects to the merge request page' do
           expect(response).
-            to redirect_to(namespace_project_issue_path(project.namespace, project, issue))
+            to redirect_to(namespace_project_merge_request_path(project.namespace, project, merge_request))
         end
       end
     end