added test without permissions. fixed other tests

smash/web/tests/view/test_visit.py

@@ -101,7 +101,28 @@ class VisitViewTests(LoggedInTestCase):
         self.assertTrue(new_visit.is_finished)
         self.assertEqual(2, Visit.objects.count())
 
+    def test_unfinish_visit_without_permissions(self):
+        visit = create_visit()
+
+        self.assertFalse(visit.is_finished)
+
+        response = self.client.get(reverse('web.views.visit_mark', args=[visit.id, "finished"]))
+        self.assertEqual(response.status_code, 302)
+
+        new_visit = Visit.objects.get(id=visit.id)
+        self.assertTrue(new_visit.is_finished)
+        self.assertEqual(2, Visit.objects.count())
+
+        response = self.client.get(reverse('web.views.visit_unfinish', args=[visit.id]), follow=True)
+        messages = list(get_messages(response.wsgi_request))
+        self.assertEqual(len(messages), 1)
+        self.assertEqual(str(messages[0]), 'You are not authorized to view this page or perform this action. Request permissions to the system administrator.')
+        new_visit = Visit.objects.get(id=visit.id)
+        self.assertEqual(2, Visit.objects.count())
+        self.assertTrue(new_visit.is_finished)
+    
     def test_unfinish_visit_with_future_visit_without_appointments(self):
+        self.login_as_admin()
         visit = create_visit()
 
         self.assertFalse(visit.is_finished)
@@ -122,6 +143,7 @@ class VisitViewTests(LoggedInTestCase):
         self.assertFalse(new_visit.is_finished)
 
     def test_unfinish_visit_with_future_visit_with_appointments(self):
+        self.login_as_admin()
         visit = create_visit()
 
         self.assertFalse(visit.is_finished)
@@ -143,6 +165,7 @@ class VisitViewTests(LoggedInTestCase):
         self.assertTrue(original_visit.is_finished)
 
     def test_unfinish_visit_with_two_future_visits(self):
+        self.login_as_admin()
         visit = create_visit()
         self.assertFalse(visit.is_finished)
         response = self.client.get(reverse('web.views.visit_mark', args=[visit.id, "finished"]))
@@ -169,6 +192,7 @@ class VisitViewTests(LoggedInTestCase):
         self.assertTrue(original_visit.is_finished)
 
     def test_unfinish_visit_which_is_not_finished(self):
+        self.login_as_admin()
         visit = create_visit()
 
         self.assertFalse(visit.is_finished)