From bf6a7cdc090793f45fa292d42ba5c5498f489b6b Mon Sep 17 00:00:00 2001
From: "J.-S. Caux" <J.S.Caux@uva.nl>
Date: Tue, 5 Nov 2019 09:00:30 +0100
Subject: [PATCH] Start repairing tests

---
 submissions/test_views.py | 2 +-
 theses/test_views.py      | 9 +++++++--
 2 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/submissions/test_views.py b/submissions/test_views.py
index 8ad905f92..bc72aeb2f 100644
--- a/submissions/test_views.py
+++ b/submissions/test_views.py
@@ -15,7 +15,7 @@ from .factories import UnassignedSubmissionFactory, EICassignedSubmissionFactory
                        ResubmittedSubmissionFactory, ResubmissionFactory,\
                        PublishedSubmissionFactory, DraftReportFactory,\
                        AcceptedRefereeInvitationFactory
-from .forms import SubmissionIdentifierForm, ReportForm. SubmissionForm
+from .forms import SubmissionIdentifierForm, ReportForm, SubmissionForm
 from .models import Submission, Report, RefereeInvitation
 
 from journals.models import Journal
diff --git a/theses/test_views.py b/theses/test_views.py
index 382afb4b1..1801a47ac 100644
--- a/theses/test_views.py
+++ b/theses/test_views.py
@@ -43,9 +43,14 @@ class TestRequestThesisLink(TestCase):
         self.target = reverse('theses:request_thesislink')
 
     def test_response_when_not_logged_in(self):
-        '''A visitor that is not logged in cannot view this page.'''
+        '''
+        A visitor that is not logged in cannot view this page and is redirected to login.
+        '''
         response = self.client.get(self.target)
-        self.assertEqual(response.status_code, 403)
+        self.assertRedirects(
+            response,
+            expected_url='%s?next=%s' % (
+                reverse('scipost:login'), reverse('theses:request_thesislink')))
 
     def test_response_when_logged_in(self):
         request = RequestFactory().get(self.target)
-- 
GitLab