Add certificate download button to course view (#157)

* Initial plan

* Add certificate download button to course view

Co-authored-by: alexeygrigorev <875246+alexeygrigorev@users.noreply.github.com>

* Address code review feedback: add is_authenticated check and test for unauthenticated users

Co-authored-by: alexeygrigorev <875246+alexeygrigorev@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: alexeygrigorev <875246+alexeygrigorev@users.noreply.github.com>

Author: Copilot

courses/templates/courses/course.html

@@ -23,6 +23,9 @@ <h2>{{ course.title }}</h2>
     {% if is_authenticated %}
       <a class="btn btn-info" href="{% url 'enrollment' course.slug %}" role="button">Edit course profile</a>
     {% endif %}
+    {% if is_authenticated and certificate_url %}
+      <a class="btn btn-success" href="{{ certificate_url }}" target="_blank" role="button"><i class="fas fa-certificate"></i> Download Certificate</a>
+    {% endif %}
   </p>
   {% if user.is_staff %}
   <p>

courses/tests/test_course.py

@@ -545,6 +545,53 @@ def test_course_view_without_completed_projects(self):
         self.assertFalse(response.context["has_completed_projects"])
         self.assertNotContains(response, "See all submitted projects")
 
+    def test_course_view_with_certificate(self):
+        """Test that the course view shows the certificate download button when a certificate is available"""
+        # Set a certificate URL for the enrollment
+        self.enrollment.certificate_url = "https://example.com/certificate.pdf"
+        self.enrollment.save()
+
+        self.client.login(**credentials)
+        response = self.client.get(
+            reverse("course", args=[self.course.slug])
+        )
+        self.assertEqual(response.status_code, 200)
+        self.assertTemplateUsed(response, "courses/course.html")
+        self.assertEqual(response.context["certificate_url"], "https://example.com/certificate.pdf")
+        self.assertContains(response, "Download Certificate")
+        self.assertContains(response, 'href="https://example.com/certificate.pdf"')
+
+    def test_course_view_without_certificate(self):
+        """Test that the course view doesn't show the certificate download button when no certificate is available"""
+        # Ensure no certificate URL is set
+        self.enrollment.certificate_url = None
+        self.enrollment.save()
+
+        self.client.login(**credentials)
+        response = self.client.get(
+            reverse("course", args=[self.course.slug])
+        )
+        self.assertEqual(response.status_code, 200)
+        self.assertTemplateUsed(response, "courses/course.html")
+        self.assertIsNone(response.context["certificate_url"])
+        self.assertNotContains(response, "Download Certificate")
+
+    def test_course_view_certificate_not_shown_when_not_authenticated(self):
+        """Test that the certificate button is not shown to unauthenticated users even if certificate exists"""
+        # Set a certificate URL for the enrollment
+        self.enrollment.certificate_url = "https://example.com/certificate.pdf"
+        self.enrollment.save()
+
+        # Don't login - access as unauthenticated user
+        response = self.client.get(
+            reverse("course", args=[self.course.slug])
+        )
+        self.assertEqual(response.status_code, 200)
+        self.assertTemplateUsed(response, "courses/course.html")
+        # certificate_url should be None for unauthenticated users
+        self.assertIsNone(response.context["certificate_url"])
+        self.assertNotContains(response, "Download Certificate")
+
     def test_list_all_submissions_view(self):
         """Test the list all submissions view shows submissions in correct order"""
         self.client.login(**credentials)

courses/views/course.py

@@ -176,13 +176,15 @@ def course_view(request: HttpRequest, course_slug: str) -> HttpResponse:
             has_completed_projects = True
 
     total_score = None
+    certificate_url = None
     if user.is_authenticated:
         try:
             enrollment = Enrollment.objects.get(
                 student=user,
                 course=course,
             )
             total_score = enrollment.total_score
+            certificate_url = enrollment.certificate_url
         except Enrollment.DoesNotExist:
             pass
 
@@ -193,6 +195,7 @@ def course_view(request: HttpRequest, course_slug: str) -> HttpResponse:
         "has_completed_projects": has_completed_projects,
         "is_authenticated": user.is_authenticated,
         "total_score": total_score,
+        "certificate_url": certificate_url,
     }
 
     return render(request, "courses/course.html", context)