feat: #22 Course page template view (without frontend component)

Author: payamnj

django_email_learning/platform/urls.py

@@ -1,11 +1,12 @@
 from django.urls import path
 from django.views.generic import RedirectView
-from django_email_learning.platform.views import Courses, Organizations
+from django_email_learning.platform.views import CourseView, Courses, Organizations
 
 app_name = "email_learning"
 
 urlpatterns = [
     path("courses/", Courses.as_view(), name="courses_view"),
+    path("courses/<int:course_id>/", CourseView.as_view(), name="course_detail_view"),
     path("organizations/", Organizations.as_view(), name="organizations_view"),
     path(
         "",

django_email_learning/platform/views.py

@@ -3,8 +3,11 @@
 from django.contrib.auth.decorators import login_required
 from django.utils.decorators import method_decorator
 from django.urls import reverse
-from django_email_learning.models import Organization
-from django_email_learning.decorators import is_platform_admin
+from django_email_learning.models import Organization, OrganizationUser, Course
+from django_email_learning.decorators import (
+    is_platform_admin,
+    is_an_organization_member,
+)
 from typing import Dict, Any
 
 
@@ -19,10 +22,19 @@ def get_context_data(self, **kwargs) -> Dict[str, Any]:  # type: ignore[no-untyp
 
     def get_shared_context(self) -> Dict[str, Any]:
         """Get shared context for all platform views"""
+        active_organization_id = self.get_or_set_active_organization()
+        if self.request.user.is_superuser:
+            role = "admin"
+        else:
+            role = OrganizationUser.objects.get(  # type: ignore[misc]
+                user=self.request.user,
+                organization_id=active_organization_id,
+            ).role
         return {
             "api_base_url": reverse("django_email_learning:api:root")[:-1],
             "platform_base_url": reverse("django_email_learning:platform:root")[:-1],
-            "active_organization_id": self.get_or_set_active_organization(),
+            "active_organization_id": active_organization_id,
+            "user_role": role,
             "is_platform_admin": (
                 self.request.user.is_superuser
                 or (
@@ -62,6 +74,18 @@ def get_context_data(self, **kwargs) -> dict:  # type: ignore[no-untyped-def]
         return context
 
 
+@method_decorator(login_required, name="dispatch")
+@method_decorator(is_an_organization_member(), name="dispatch")
+class CourseView(BasePlatformView):
+    template_name = "platform/course.html"
+
+    def get_context_data(self, **kwargs) -> dict:  # type: ignore[no-untyped-def]
+        context = super().get_context_data(**kwargs)
+        course = Course.objects.get(pk=self.kwargs["course_id"])
+        context["page_title"] = course.title
+        return context
+
+
 @method_decorator(login_required, name="dispatch")
 @method_decorator(is_platform_admin(), name="dispatch")
 class Organizations(BasePlatformView):

django_email_learning/templates/platform/base.html

@@ -9,6 +9,7 @@
         localStorage.setItem('activeOrganizationId', '{{ active_organization_id }}');
         localStorage.setItem('apiBaseUrl', '{{ api_base_url }}');
         localStorage.setItem('platformBaseUrl', '{{ platform_base_url }}');
+        localStorage.setItem('userRole', '{{ user_role }}');
         localStorage.setItem('isPlatformAdmin', {{ is_platform_admin|yesno:"true,false" }});
     </script>
     <meta charset="UTF-8" />

django_email_learning/templates/platform/course.html

@@ -0,0 +1,5 @@
+{% extends "platform/base.html" %}
+{% load django_vite %}
+{% block extra_head %}
+    <!-- {% vite_asset 'courses/Courses.jsx' %} -->
+{% endblock %}

tests/api/conftest.py

@@ -0,0 +1,42 @@
+from django.urls import reverse
+import pytest
+
+
+def get_url(course_id: int = 1) -> str:
+    return reverse(
+        "django_email_learning:platform:course_detail_view",
+        kwargs={"course_id": course_id},
+    )
+
+
+def test_anonymous_user_redirects_to_login(anonymous_client):
+    response = anonymous_client.get(get_url())
+    assert response.status_code == 302
+    assert "/login/" in response.url
+
+
+@pytest.mark.parametrize(
+    "client,role",
+    [
+        ("superadmin", "admin"),
+        ("platform_admin", "admin"),
+        ("editor", "editor"),
+        ("viewer", "viewer"),
+    ],
+    indirect=["client"],
+)
+def test_authenticated_user_access_course_detail_view(client, role, course):
+    response = client.get(get_url(course.id))
+    assert response.status_code == 200
+    assert response.context["user_role"] == role
+
+
+def test_context_values(superadmin_client, course):
+    response = superadmin_client.get(get_url(course.id))
+    assert response.status_code == 200
+    assert "api_base_url" in response.context
+    assert "platform_base_url" in response.context
+    assert "active_organization_id" in response.context
+    assert "user_role" in response.context
+    assert response.context["page_title"] == course.title
+    assert response.context["is_platform_admin"] is True

tests/conftest.py

@@ -0,0 +1,39 @@
+from django.urls import reverse
+import pytest
+
+
+def get_url() -> str:
+    return reverse("django_email_learning:platform:courses_view")
+
+
+def test_anonymous_user_redirects_to_login(anonymous_client):
+    response = anonymous_client.get(get_url())
+    assert response.status_code == 302
+    assert "/login/" in response.url
+
+
+@pytest.mark.parametrize(
+    "client,role",
+    [
+        ("superadmin", "admin"),
+        ("platform_admin", "admin"),
+        ("editor", "editor"),
+        ("viewer", "viewer"),
+    ],
+    indirect=["client"],
+)
+def test_authenticated_user_access_courses_view(client, role):
+    response = client.get(get_url())
+    assert response.status_code == 200
+    assert response.context["user_role"] == role
+
+
+def test_context_values(superadmin_client):
+    response = superadmin_client.get(get_url())
+    assert response.status_code == 200
+    assert "api_base_url" in response.context
+    assert "platform_base_url" in response.context
+    assert "active_organization_id" in response.context
+    assert "user_role" in response.context
+    assert response.context["page_title"] == "Courses"
+    assert response.context["is_platform_admin"] is True