squash!: Refactor test to not use contentstore imports

Author: rodmgwgu

openedx/core/djangoapps/content_tagging/rest_api/v1/tests/test_views.py

@@ -13,6 +13,7 @@
 import ddt
 from django.contrib.auth import get_user_model
 from django.core.files.uploadedfile import SimpleUploadedFile
+from django.urls import reverse
 from edx_django_utils.cache import RequestCache
 from opaque_keys.edx.locator import BlockUsageLocator, CourseLocator, LibraryCollectionLocator, LibraryContainerLocator
 from openedx_authz.constants.roles import COURSE_STAFF
@@ -24,7 +25,6 @@
 from rest_framework.test import APITestCase, APIClient
 
 
-from cms.djangoapps.contentstore.api.tests.base import BaseCourseViewTest
 from common.djangoapps.student.auth import add_users, update_org_role
 from common.djangoapps.student.roles import (
     CourseInstructorRole,
@@ -34,8 +34,10 @@
     OrgLibraryUserRole,
     OrgStaffRole
 )
-from common.djangoapps.student.tests.factories import UserFactory
+from common.djangoapps.student.tests.factories import StaffFactory, UserFactory
 from openedx.core.djangoapps.authz.tests.mixins import CourseAuthzTestMixin
+from xmodule.modulestore.tests.django_utils import SharedModuleStoreTestCase
+from xmodule.modulestore.tests.factories import CourseFactory
 from openedx.core.djangoapps.content_libraries.api import AccessLevel, create_library, set_library_user_permissions
 from openedx.core.djangoapps.content_tagging import api as tagging_api
 from openedx.core.djangoapps.content_tagging.models import TaxonomyOrg
@@ -2056,51 +2058,52 @@ def test_export_course_invalid_id(self) -> None:
         assert response.status_code == status.HTTP_403_FORBIDDEN
 
 @skip_unless_cms
-class TestContentObjectChildrenExportViewWithAuthz(CourseAuthzTestMixin, BaseCourseViewTest):
+class TestContentObjectChildrenExportViewWithAuthz(CourseAuthzTestMixin, SharedModuleStoreTestCase, APITestCase):
     """
     Tests Tags Export in Course authorization using openedx-authz.
     """
 
-    view_name = 'content_tagging:taxonomy-object-tag-export'
-    course_key_arg_name = 'context_id'
     authz_roles_to_assign = [COURSE_STAFF.external_key]
 
+    @classmethod
+    def setUpClass(cls):
+        super().setUpClass()
+        cls.password = 'test'
+        cls.course = CourseFactory.create()
+        cls.course_key = cls.course.id
+        cls.staff = StaffFactory(course_key=cls.course_key, password=cls.password)
+
+    def get_url(self, course_key):
+        return reverse('content_tagging:taxonomy-object-tag-export', kwargs={'context_id': course_key})
+
     def test_authorized_user_can_access(self):
         """User with COURSE_STAFF role can access."""
-        self.authorized_client.login(username=self.authorized_user.username, password=self.password)
         resp = self.authorized_client.get(self.get_url(self.course_key))
         self.assertEqual(resp.status_code, status.HTTP_200_OK)
 
     def test_unauthorized_user_cannot_access(self):
         """User without role cannot access."""
-        self.unauthorized_client.login(username=self.unauthorized_user.username, password=self.password)
         resp = self.unauthorized_client.get(self.get_url(self.course_key))
         self.assertEqual(resp.status_code, status.HTTP_403_FORBIDDEN)
 
     def test_role_scoped_to_course(self):
         """Authorization should only apply to the assigned course."""
         other_course = self.store.create_course("OtherOrg", "OtherCourse", "Run", self.staff.id)
 
-        self.authorized_client.login(username=self.authorized_user.username, password=self.password)
         resp = self.authorized_client.get(self.get_url(other_course.id))
         self.assertEqual(resp.status_code, status.HTTP_403_FORBIDDEN)
 
     def test_staff_user_allowed_via_legacy(self):
-        """
-        Staff users should still pass through legacy fallback.
-        """
-        self.client.login(username=self.staff.username, password=self.password)
-
+        """Staff users should still pass through legacy fallback."""
+        self.client.force_authenticate(user=self.staff)
         resp = self.client.get(self.get_url(self.course_key))
         self.assertEqual(resp.status_code, status.HTTP_200_OK)
 
     def test_superuser_allowed(self):
         """Superusers should always be allowed."""
-        superuser = UserFactory(is_superuser=True, username='superuser', password=self.password)
-
+        superuser = UserFactory(is_superuser=True)
         client = APIClient()
-        client.login(username=superuser.username, password=self.password)
-
+        client.force_authenticate(user=superuser)
         resp = client.get(self.get_url(self.course_key))
         self.assertEqual(resp.status_code, status.HTTP_200_OK)