♻ permissions refactoring: refactor has_permission methods

1 files changed, 6 insertions, 5 deletions

diff --git a/archaeological_warehouse/tests.py b/archaeological_warehouse/tests.py
index c218b7d0e..96df83024 100644
--- a/archaeological_warehouse/tests.py
+++ b/archaeological_warehouse/tests.py
@@ -20,7 +20,7 @@
 import datetime
 import json
 
-from django.contrib.auth.models import Permission
+from django.contrib.auth.models import Permission, User
 from django.db.utils import IntegrityError
 from django.test.client import Client
 from django.urls import reverse
@@ -560,6 +560,7 @@ class ContainerQATest(FindInit, TestCase):
 
     def setUp(self):
         self.get_default_user()
+        self.alt_username, self.alt_password, self.alt_user = create_user()
         self.main_warehouse = models.Warehouse.objects.create(
             name="Main", warehouse_type=models.WarehouseType.objects.all()[0]
         )
@@ -638,16 +639,16 @@ class ContainerQATest(FindInit, TestCase):
         response = c.get(url)
         self.assertRedirects(response, "/")
         c = Client()
-        c.login(username=self.username, password=self.password)
+        c.login(username=self.alt_username, password=self.alt_password)
         response = c.get(url)
         # change_container permission is needed
-        self.assertEqual(response.status_code, 404)
+        self.assertEqual(response.status_code, 302)
 
         c.logout()
-        self.user.user_permissions.add(
+        self.alt_user.user_permissions.add(
             Permission.objects.get(codename="change_container")
         )
-        c.login(username=self.username, password=self.password)
+        c.login(username=self.alt_username, password=self.alt_password)
         response = c.get(url)
         self.assertEqual(response.status_code, 200)