diff --git a/django_email_learning/migrations/0017_alter_organizationuser_role.py b/django_email_learning/migrations/0017_alter_organizationuser_role.py
new file mode 100644
index 0000000..e5e672d
--- /dev/null
+++ b/django_email_learning/migrations/0017_alter_organizationuser_role.py
@@ -0,0 +1,26 @@
+# Generated by Django 6.0.4 on 2026-04-29 07:18
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("django_email_learning", "0016_alter_jobexecution_job_name_and_more"),
+    ]
+
+    operations = [
+        migrations.AlterField(
+            model_name="organizationuser",
+            name="role",
+            field=models.CharField(
+                choices=[
+                    ("admin", "Admin"),
+                    ("editor", "Editor"),
+                    ("instructor", "Instructor"),
+                    ("viewer", "Viewer"),
+                ],
+                db_index=True,
+                max_length=50,
+            ),
+        ),
+    ]
diff --git a/django_email_learning/models.py b/django_email_learning/models.py
index 5344933..3d27f0a 100644
--- a/django_email_learning/models.py
+++ b/django_email_learning/models.py
@@ -124,6 +124,7 @@ class OrganizationUser(models.Model):
         choices=[
             ("admin", "Admin"),
             ("editor", "Editor"),
+            ("instructor", "Instructor"),
             ("viewer", "Viewer"),
         ],
         db_index=True,
diff --git a/django_email_learning/platform/api/serializers.py b/django_email_learning/platform/api/serializers.py
index 8bb85a9..d613ee4 100644
--- a/django_email_learning/platform/api/serializers.py
+++ b/django_email_learning/platform/api/serializers.py
@@ -430,6 +430,7 @@ class UpdateOrganizationRequest(BaseModel):
 class UserRole(enum.StrEnum):
     ADMIN = "admin"
     EDITOR = "editor"
+    INSTRUCTOR = "instructor"
     VIEWER = "viewer"
 
 
diff --git a/django_email_learning/platform/api/views.py b/django_email_learning/platform/api/views.py
index b10141b..699620a 100644
--- a/django_email_learning/platform/api/views.py
+++ b/django_email_learning/platform/api/views.py
@@ -67,8 +67,10 @@
 
 
 @method_decorator(ensure_csrf_cookie, name="get")
-@method_decorator(accessible_for(roles={"admin", "editor"}), name="post")
-@method_decorator(accessible_for(roles={"admin", "editor", "viewer"}), name="get")
+@method_decorator(accessible_for(roles={"admin", "editor", "instructor"}), name="post")
+@method_decorator(
+    accessible_for(roles={"admin", "editor", "instructor", "viewer"}), name="get"
+)
 class CourseView(View):
     def post(self, request, *args, **kwargs) -> JsonResponse:  # type: ignore[no-untyped-def]
         payload = json.loads(request.body)
@@ -114,8 +116,10 @@ def get(self, request, *args, **kwargs) -> JsonResponse:  # type: ignore[no-unty
         return JsonResponse({"courses": response_list}, status=200)
 
 
-@method_decorator(accessible_for(roles={"admin", "editor"}), name="post")
-@method_decorator(accessible_for(roles={"admin", "editor", "viewer"}), name="get")
+@method_decorator(accessible_for(roles={"admin", "editor", "instructor"}), name="post")
+@method_decorator(
+    accessible_for(roles={"admin", "editor", "viewer", "instructor"}), name="get"
+)
 class CourseContentView(View):
     def post(self, request, *args, **kwargs) -> JsonResponse:  # type: ignore[no-untyped-def]
         payload = json.loads(request.body)
@@ -161,7 +165,7 @@ def get(self, request, *args, **kwargs) -> JsonResponse:  # type: ignore[no-unty
             return JsonResponse({"error": "Course not found"}, status=404)
 
 
-@method_decorator(accessible_for(roles={"admin", "editor"}), name="post")
+@method_decorator(accessible_for(roles={"admin", "editor", "instructor"}), name="post")
 class ReorderCourseContentView(View):
     def post(self, request, *args, **kwargs) -> JsonResponse:  # type: ignore[no-untyped-def]
         payload = json.loads(request.body)
@@ -207,9 +211,13 @@ def post(self, request, *args, **kwargs) -> JsonResponse:  # type: ignore[no-unt
             return JsonResponse({"error": str(e)}, status=409)
 
 
-@method_decorator(accessible_for(roles={"admin", "editor", "viewer"}), name="get")
-@method_decorator(accessible_for(roles={"admin", "editor"}), name="delete")
-@method_decorator(accessible_for(roles={"admin", "editor"}), name="post")
+@method_decorator(
+    accessible_for(roles={"admin", "editor", "instructor", "viewer"}), name="get"
+)
+@method_decorator(
+    accessible_for(roles={"admin", "editor", "instructor"}), name="delete"
+)
+@method_decorator(accessible_for(roles={"admin", "editor", "instructor"}), name="post")
 class SingleCourseContentView(View):
     def get(self, request, *args, **kwargs) -> JsonResponse:  # type: ignore[no-untyped-def]
         try:
@@ -344,9 +352,13 @@ def _update_course_content_atomic(
         )
 
 
-@method_decorator(accessible_for(roles={"admin", "editor"}), name="post")
-@method_decorator(accessible_for(roles={"admin", "editor"}), name="delete")
-@method_decorator(accessible_for(roles={"admin", "editor", "viewer"}), name="get")
+@method_decorator(accessible_for(roles={"admin", "editor", "instructor"}), name="post")
+@method_decorator(
+    accessible_for(roles={"admin", "editor", "instructor"}), name="delete"
+)
+@method_decorator(
+    accessible_for(roles={"admin", "editor", "instructor", "viewer"}), name="get"
+)
 class SingleCourseView(View):
     def get(self, request, *args, **kwargs) -> JsonResponse:  # type: ignore[no-untyped-def]
         try:
@@ -496,8 +508,10 @@ def post(self, request, *args, **kwargs):  # type: ignore[no-untyped-def]
             return JsonResponse({"error": "Failed to enroll users"}, status=500)
 
 
-@method_decorator(accessible_for(roles={"admin", "editor"}), name="post")
-@method_decorator(accessible_for(roles={"admin", "editor", "viewer"}), name="get")
+@method_decorator(accessible_for(roles={"admin", "editor", "instructor"}), name="post")
+@method_decorator(
+    accessible_for(roles={"admin", "editor", "instructor", "viewer"}), name="get"
+)
 class ImapConnectionView(View):
     def get(self, request, *args, **kwargs) -> JsonResponse:  # type: ignore[no-untyped-def]
         response_list = []
@@ -661,7 +675,9 @@ def post(self, request, *args, **kwargs) -> JsonResponse:  # type: ignore[no-unt
 
 @method_decorator(is_platform_admin(), name="post")
 @method_decorator(is_platform_admin(), name="delete")
-@method_decorator(accessible_for(roles={"admin", "editor", "viewer"}), name="get")
+@method_decorator(
+    accessible_for(roles={"admin", "editor", "instructor", "viewer"}), name="get"
+)
 class SingleOrganizationView(View):
     def post(self, request, *args, **kwargs) -> JsonResponse:  # type: ignore[no-untyped-def]
         try:
@@ -771,7 +787,7 @@ def post(self, request, *args, **kwargs) -> JsonResponse:  # type: ignore[no-unt
             return JsonResponse({"error": str(e)}, status=409)
 
 
-@method_decorator(accessible_for(roles={"admin", "editor"}), name="post")
+@method_decorator(accessible_for(roles={"admin", "editor", "instructor"}), name="post")
 class SendLessonToPlatformUser(View):
     def post(self, request, *args, **kwargs) -> JsonResponse:  # type: ignore[no-untyped-def]
         if not request.user.email:
@@ -802,8 +818,10 @@ def post(self, request, *args, **kwargs) -> JsonResponse:  # type: ignore[no-unt
         )
 
 
-@method_decorator(accessible_for(roles={"admin", "editor"}), name="post")
-@method_decorator(accessible_for(roles={"admin", "editor"}), name="delete")
+@method_decorator(accessible_for(roles={"admin", "editor", "instructor"}), name="post")
+@method_decorator(
+    accessible_for(roles={"admin", "editor", "instructor"}), name="delete"
+)
 class FileView(View):
     def post(self, request, *args, **kwargs) -> JsonResponse:  # type: ignore[no-untyped-def]
         uploaded_file = request.FILES.get("file")
@@ -891,7 +909,7 @@ def post(self, request, *args, **kwargs) -> JsonResponse:  # type: ignore[no-unt
         return JsonResponse(response_serializer.model_dump(), status=200)
 
 
-@method_decorator(accessible_for(roles={"admin", "editor", "viewer"}), name="get")
+@method_decorator(accessible_for(roles={"admin", "instructor"}), name="get")
 class LearnersView(PaginatedApiMixin, View):
     def get_query_set(self, request: Any) -> models.QuerySet:
         organization_id = self.kwargs["organization_id"]
@@ -913,7 +931,9 @@ def get_item_serializer_class(self) -> Any:
         return serializers.LearnerResponse
 
 
-@method_decorator(accessible_for(roles={"admin", "editor", "viewer"}), name="get")
+@method_decorator(
+    accessible_for(roles={"admin", "editor", "viewer", "instructor"}), name="get"
+)
 class SingleLearnerView(View):
     def get(self, request, *args, **kwargs) -> JsonResponse:  # type: ignore[no-untyped-def]
         try:
@@ -954,7 +974,7 @@ def get(self, request, *args, **kwargs) -> JsonResponse:  # type: ignore[no-unty
             return JsonResponse({"error": "An internal error occurred."}, status=500)
 
 
-@method_decorator(accessible_for(roles={"admin"}), name="post")
+@method_decorator(accessible_for(roles={"admin", "instructor"}), name="post")
 class EnrollmentsView(PaginatedApiMixin, View):
     def post(self, request, *args, **kwargs) -> JsonResponse:  # type: ignore[no-untyped-def]
         payload = json.loads(request.body)
@@ -998,7 +1018,9 @@ def post(self, request, *args, **kwargs) -> JsonResponse:  # type: ignore[no-unt
             return JsonResponse({"error": e.json()}, status=400)
 
 
-@method_decorator(accessible_for(roles={"admin", "editor", "viewer"}), name="get")
+@method_decorator(
+    accessible_for(roles={"admin", "editor", "instructor", "viewer"}), name="get"
+)
 class EnrollmentView(View):
     def get(self, request, *args, **kwargs) -> JsonResponse:  # type: ignore[no-untyped-def]
         try:
@@ -1015,7 +1037,9 @@ def get(self, request, *args, **kwargs) -> JsonResponse:  # type: ignore[no-unty
             return JsonResponse({"error": e.json()}, status=400)
 
 
-@method_decorator(accessible_for(roles={"admin", "editor", "viewer"}), name="get")
+@method_decorator(
+    accessible_for(roles={"admin", "editor", "instructor", "viewer"}), name="get"
+)
 class EnrollmentsStatisticsView(View):
     def get(self, request, *args, **kwargs) -> JsonResponse:  # type: ignore[no-untyped-def]
         course_id = kwargs["course_id"]
diff --git a/django_email_learning/platform/views.py b/django_email_learning/platform/views.py
index 64cc691..3f7dcc6 100644
--- a/django_email_learning/platform/views.py
+++ b/django_email_learning/platform/views.py
@@ -68,6 +68,14 @@ def get_shared_context(self) -> Dict[str, Any]:
                         and getattr(self.request.user, "has_platform_admin_role", False)
                     )
                 ),
+                "isInstructor": (
+                    self.request.user.is_superuser
+                    or (
+                        OrganizationUser.objects.filter(
+                            user=self.request.user, role="instructor"
+                        ).exists()  # type: ignore[misc]
+                    )
+                ),
                 "aiTextEditingModel": AI_CONFIGURATIONS.get("TEXT_EDITING_MODEL"),
                 "customLogo": {
                     "horizontalLight": DJANGO_EMAIL_LEARNING_SETTINGS.get("LOGO", {})
@@ -478,7 +486,16 @@ def get_locale_messages(self) -> Dict[str, str]:
             "role": _("Role"),
             "admin": _("Admin"),
             "editor": _("Editor"),
+            "instructor": _("Instructor"),
             "viewer": _("Viewer"),
+            "viewer_role_description": _("Read-only access to course content."),
+            "editor_role_description": _("Can create and edit course content."),
+            "instructor_role_description": _(
+                "All editor permissions, plus access to learners and assignment review and approval."
+            ),
+            "admin_role_description": _(
+                "Full access, including creating users and organizations."
+            ),
             "email": _("Email"),
             "delete_user_with_email": _("Deleting USER_EMAIL"),
             "user_delete_confirmation": _(
@@ -494,7 +511,7 @@ def get_locale_messages(self) -> Dict[str, str]:
 
 
 @method_decorator(login_required, name="dispatch")
-@method_decorator(is_an_organization_member(only_admin=True), name="dispatch")
+@method_decorator(is_an_organization_member(), name="dispatch")
 class Learners(BasePlatformView):
     template_name = "platform/learners.html"
 
diff --git a/frontend/platform/learners/Learners.jsx b/frontend/platform/learners/Learners.jsx
index 2c3cfed..575269f 100644
--- a/frontend/platform/learners/Learners.jsx
+++ b/frontend/platform/learners/Learners.jsx
@@ -202,7 +202,7 @@ function Learners(initialQs="") {
           </Box>
           <Box sx={{ p: 2 }}>
           <Suspense fallback={<Box sx={{ p: 2 }}><LinearProgress /></Box>}>
-            <EnrollentList enrollments={enrollments} selectHandler={showEnrollmentStatus} />
+            { enrollments && <EnrollentList enrollments={enrollments} selectHandler={showEnrollmentStatus} /> }
           </Suspense>
           </Box>
         </Box>
diff --git a/frontend/platform/organization/components/UserForm.jsx b/frontend/platform/organization/components/UserForm.jsx
index 8a650a6..fe661d6 100644
--- a/frontend/platform/organization/components/UserForm.jsx
+++ b/frontend/platform/organization/components/UserForm.jsx
@@ -17,6 +17,15 @@ const UserForm = ({ onClose, organizationId, refreshUsers, user = null }) => {
     const [role, setRole] = useState(user ? user.role : 'viewer');
     const [error, setError] = useState('');
 
+    const roleDescriptionByRole = {
+        viewer: localeMessages["viewer_role_description"],
+        editor: localeMessages["editor_role_description"],
+        instructor: localeMessages["instructor_role_description"],
+        admin: localeMessages["admin_role_description"],
+    };
+
+    const selectedRoleDescription = roleDescriptionByRole[role] || '';
+
     const createUser = (id) => {
         fetch(`${apiBaseUrl}/organizations/${organizationId}/users/`, {
             method: 'POST',
@@ -123,9 +132,15 @@ const UserForm = ({ onClose, organizationId, refreshUsers, user = null }) => {
                 >
                     <MenuItem value="viewer">{localeMessages["viewer"]}</MenuItem>
                     <MenuItem value="editor">{localeMessages["editor"]}</MenuItem>
+                    <MenuItem value="instructor">{localeMessages["instructor"]}</MenuItem>
                     <MenuItem value="admin">{localeMessages["admin"]}</MenuItem>
                 </Select>
             </FormControl>
+            {selectedRoleDescription && (
+                <Typography variant="body2" color="text.secondary">
+                    {selectedRoleDescription}
+                </Typography>
+            )}
             {error && <Typography color="error">{error}</Typography>}
             <Button type="submit" variant="contained" color="secondary">
                 {  user ? localeMessages["edit_user"] : localeMessages["add_user"]}
diff --git a/frontend/src/components/MenuBar.jsx b/frontend/src/components/MenuBar.jsx
index 9cf6ec6..41e51bf 100644
--- a/frontend/src/components/MenuBar.jsx
+++ b/frontend/src/components/MenuBar.jsx
@@ -56,7 +56,7 @@ function MenuBar({activeOrganizationId, changeOrganizationCallback, showOrganiza
     const [menuOpen, setMenuOpen] = useState(false)
     const [organizations, setOrganizations] = useState([])
     const [deliverContentsJobStatus, setDeliverContentsJobStatus] = useState(null)
-    const { localeMessages, isPlatformAdmin, isOrganizationAdmin, direction, apiBaseUrl, platformBaseUrl, sidebarCustomComponent, customLogo } = useAppContext();
+    const { localeMessages, isPlatformAdmin, isOrganizationAdmin, isInstructor, direction, apiBaseUrl, platformBaseUrl, sidebarCustomComponent, customLogo } = useAppContext();
 
     const theme = useTheme();
     const isMdUpScreen = useMediaQuery(theme.breakpoints.up('md'));
@@ -137,7 +137,7 @@ function MenuBar({activeOrganizationId, changeOrganizationCallback, showOrganiza
     }
 
     pages.push({ name: localeMessages["course_management"], icon: <SchoolOutlinedIcon fontSize="small" />, href: platformBaseUrl + '/courses/' });
-    if (isOrganizationAdmin || isPlatformAdmin) {
+    if (isOrganizationAdmin || isPlatformAdmin || isInstructor) {
         pages.push({ name: localeMessages["learners"], icon: <PeopleOutlinedIcon fontSize="small" />, href: platformBaseUrl + '/learners/' });
     }
     // pages.push({ name: 'Analytics', icon: <BarChartIcon fontSize="small" />, href: platformBaseUrl + '/analytics/' });
diff --git a/tests/conftest.py b/tests/conftest.py
index 4515d96..a50cf94 100644
--- a/tests/conftest.py
+++ b/tests/conftest.py
@@ -51,6 +51,12 @@ def users(db):
         email="orgadmin@example.com",
         password="orgadminpass",
     )
+    instructor_user = User(
+        id=6,
+        username="instructor",
+        email="instructor@example.com",
+        password="instructorpass",
+    )
     User.objects.bulk_create(
         [
             superadmin,
@@ -58,11 +64,15 @@ def users(db):
             platform_admin_user,
             viewer_user,
             organization_admin_user,
+            instructor_user,
         ]
     )
     group = Group.objects.get(name=PLATFORM_ADMIN_GROUP_NAME)
     platform_admin_user.groups.add(group)
     editor = OrganizationUser(user=editor_user, organization_id=1, role="editor")
+    instructor = OrganizationUser(
+        user=instructor_user, organization_id=1, role="instructor"
+    )
     platform_admin = OrganizationUser(
         user=platform_admin_user, organization_id=1, role="admin"
     )
@@ -71,11 +81,12 @@ def users(db):
     )
     viewer = OrganizationUser(user=viewer_user, organization_id=1, role="viewer")
     OrganizationUser.objects.bulk_create(
-        [editor, platform_admin, organization_admin, viewer]
+        [editor, instructor, platform_admin, organization_admin, viewer]
     )
     return {
         "superadmin": superadmin,
         "editor_user": editor_user,
+        "instructor_user": instructor_user,
         "platform_admin": platform_admin_user,
         "organization_admin": organization_admin_user,
         "viewer_user": viewer_user,
@@ -101,6 +112,13 @@ def editor_client(users):
     return client
 
 
+@pytest.fixture()
+def instructor_client(users):
+    client = Client()
+    client.force_login(users["instructor_user"])
+    return client
+
+
 @pytest.fixture()
 def platform_admin_client(users):
     client = Client()
@@ -127,6 +145,7 @@ def client(
     request,
     superadmin_client,
     editor_client,
+    instructor_client,
     platform_admin_client,
     viewer_client,
     anonymous_client,
@@ -140,6 +159,7 @@ def _get_client(role_name):
             "platform_admin": platform_admin_client,
             "viewer": viewer_client,
             "org_admin": org_admin_client,
+            "instructor": instructor_client,
         }
         return role_map.get(role_name)
 
diff --git a/tests/oauth_integrations/test_views.py b/tests/oauth_integrations/test_views.py
index a3ad27b..9caac7a 100644
--- a/tests/oauth_integrations/test_views.py
+++ b/tests/oauth_integrations/test_views.py
@@ -9,7 +9,7 @@
     Group,
     User,
 )
-
+import pytest
 
 SESSIONS_URL = reverse("django_email_learning:oauth_integrations:sessions_view")
 REDIRECT_URL = reverse("django_email_learning:oauth_integrations:redirect_view")
@@ -66,8 +66,11 @@ def test_create_session_unauthorized(anonymous_client, course):
     assert response.json() == {"error": "Unauthorized"}
 
 
-def test_create_session_forbidden_for_editor(editor_client, course):
-    response = editor_client.post(
+@pytest.mark.parametrize(
+    "client", ["editor", "viewer", "instructor"], indirect=["client"]
+)
+def test_create_session_forbidden_for_editor(client, course):
+    response = client.post(
         SESSIONS_URL,
         json.dumps(oauth_payload(course.id)),
         content_type="application/json",
@@ -282,8 +285,11 @@ def test_enroll_users_unauthorized(db, anonymous_client):
     assert response.json() == {"error": "Unauthorized"}
 
 
-def test_enroll_users_forbidden_for_editor(db, editor_client):
-    response = editor_client.post(
+@pytest.mark.parametrize(
+    "client", ["editor", "viewer", "instructor"], indirect=["client"]
+)
+def test_enroll_users_forbidden_for_editor(db, client):
+    response = client.post(
         oauth_enroll_users_url("any-session"),
         json.dumps({"groups": ["all"]}),
         content_type="application/json",
diff --git a/tests/platform/api/test_views/test_api_key_view.py b/tests/platform/api/test_views/test_api_key_view.py
index 42725a8..7aa7962 100644
--- a/tests/platform/api/test_views/test_api_key_view.py
+++ b/tests/platform/api/test_views/test_api_key_view.py
@@ -1,4 +1,5 @@
 from django.urls import reverse
+import pytest
 
 URL = reverse("django_email_learning:api_platform:api_key_view")
 
@@ -21,8 +22,11 @@ def test_create_api_key(superadmin_client):
     assert any(api_key["key"] == created_key for api_key in api_keys)
 
 
-def test_organization_user_cannot_create_api_key(editor_client):
-    response = editor_client.post(URL)
+@pytest.mark.parametrize(
+    "client", ["editor", "viewer", "instructor"], indirect=["client"]
+)
+def test_organization_user_cannot_create_api_key(client):
+    response = client.post(URL)
     assert response.status_code == 403
 
 
diff --git a/tests/platform/api/test_views/test_course_view.py b/tests/platform/api/test_views/test_course_view.py
index 816869e..8920f0d 100644
--- a/tests/platform/api/test_views/test_course_view.py
+++ b/tests/platform/api/test_views/test_course_view.py
@@ -423,20 +423,21 @@ def test_viewer_not_allowed_to_delete_course(sample_course, viewer_client):
     assert delete_response.status_code == 403
 
 
-def test_editor_can_delete_course(sample_course, editor_client):
+@pytest.mark.parametrize("client", ["editor", "instructor"], indirect=["client"])
+def test_editor_can_delete_course(sample_course, client):
     # Check that we have one course before the delete
-    courses = editor_client.get(get_url(1))
+    courses = client.get(get_url(1))
     assert len(courses.json().get("courses")) == 1
 
     url = reverse(
         "django_email_learning:api_platform:single_course_view",
         kwargs={"organization_id": 1, "course_id": sample_course["id"]},
     )
-    delete_response = editor_client.delete(url)
+    delete_response = client.delete(url)
     assert delete_response.status_code == 200
 
     # Check that we don't have any course after the delete
-    courses = editor_client.get(get_url(1))
+    courses = client.get(get_url(1))
     assert len(courses.json().get("courses")) == 0
 
 
diff --git a/tests/platform/api/test_views/test_learners_view.py b/tests/platform/api/test_views/test_learners_view.py
index 8cc7b6f..8cb566d 100644
--- a/tests/platform/api/test_views/test_learners_view.py
+++ b/tests/platform/api/test_views/test_learners_view.py
@@ -40,17 +40,6 @@ def test_learners_view_pagination(page, has_more, superadmin_client, learners_fa
     assert data["has_more"] is has_more
 
 
-def test_learner_view_accesible_for_platform_viewer(viewer_client, learners_factory):
-    learners_factory(5)
-
-    response = viewer_client.get(URL)
-
-    assert response.status_code == 200
-    data = response.json()
-    assert data["count"] == 5
-    assert len(data["items"]) == 5
-
-
 def test_learner_view_not_accesible_for_no_role(anonymous_client, learners_factory):
     learners_factory(5)
 
diff --git a/tests/platform/test_views/test_learners_view.py b/tests/platform/test_views/test_learners_view.py
index 2d5c845..5e31e32 100644
--- a/tests/platform/test_views/test_learners_view.py
+++ b/tests/platform/test_views/test_learners_view.py
@@ -1,15 +1,16 @@
 from django.urls import reverse
+import pytest
 
 URL = reverse("django_email_learning:platform:learners_view")
 
 
-def test_learners_accesible_to_org_admin(org_admin_client):
-    response = org_admin_client.get(URL)
+@pytest.mark.parametrize("client", ["org_admin", "instructor"], indirect=["client"])
+def test_learners_accesible_to_org_admin_and_instructor(client):
+    response = client.get(URL)
     assert response.status_code == 200
 
 
-def test_learners_inaccessible_to_editor_and_viewer(editor_client, viewer_client):
-    response = editor_client.get(URL)
-    assert response.status_code == 403
-    response = viewer_client.get(URL)
-    assert response.status_code == 403
+def test_learners_view_redirects_anonymous(anonymous_client):
+    response = anonymous_client.get(URL)
+    assert response.status_code == 302
+    assert response.url.startswith("/accounts/login/")