diff --git a/src/registrar/tests/common.py b/src/registrar/tests/common.py index 8f859b6d1..c6cd8ebfd 100644 --- a/src/registrar/tests/common.py +++ b/src/registrar/tests/common.py @@ -427,14 +427,12 @@ def create_superuser(): def create_user(): User = get_user_model() p = "userpass" - staffuser = User.objects.create_user( + return User.objects.create_user( username="staffuser", email="user@example.com", + is_staff=True, password=p, ) - staffuser.is_staff = True - staffuser.save() - return staffuser def create_ready_domain(): diff --git a/src/registrar/tests/test_admin.py b/src/registrar/tests/test_admin.py index e556adcec..28d407a35 100644 --- a/src/registrar/tests/test_admin.py +++ b/src/registrar/tests/test_admin.py @@ -28,6 +28,7 @@ from .common import ( from django.contrib.sessions.backends.db import SessionStore from django.contrib.auth import get_user_model from unittest.mock import patch +from unittest import skip from django.conf import settings from unittest.mock import MagicMock @@ -40,7 +41,6 @@ logger = logging.getLogger(__name__) class TestDomainAdmin(TestCase): def setUp(self): self.site = AdminSite() - self.factory = RequestFactory() self.admin = DomainAdmin(model=Domain, admin_site=self.site) self.client = Client(HTTP_HOST="localhost:8080") self.superuser = create_superuser() @@ -83,6 +83,10 @@ class TestDomainAdmin(TestCase): self.assertContains(response, "Place hold") self.assertNotContains(response, "Remove hold") + @skip("Waiting on epp lib to implement") + def test_place_and_remove_hold_epp(self): + raise + def tearDown(self): Domain.objects.all().delete() User.objects.all().delete()