From aaf13b972424c9307d7f34b328c69c52167b0b35 Mon Sep 17 00:00:00 2001
From: Saleem Latif <saleem_ee@hotmail.com>
Date: Tue, 20 Oct 2015 14:39:02 +0500
Subject: [PATCH] added certificate web view link to Support Dashboard

---
 lms/djangoapps/certificates/api.py            |  2 +-
 .../certificates/tests/test_support_views.py  | 26 +++++++++++++++++++
 2 files changed, 27 insertions(+), 1 deletion(-)

diff --git a/lms/djangoapps/certificates/api.py b/lms/djangoapps/certificates/api.py
index 7b66138a43..2872ea2708 100644
--- a/lms/djangoapps/certificates/api.py
+++ b/lms/djangoapps/certificates/api.py
@@ -71,7 +71,7 @@ def get_certificates_for_user(username):
             # In the future, we can update this to construct a URL to the webview certificate
             # for courses that have this feature enabled.
             "download_url": (
-                cert.download_url
+                cert.download_url or get_certificate_url(cert.user.id, cert.course_id)
                 if cert.status == CertificateStatuses.downloadable
                 else None
             ),
diff --git a/lms/djangoapps/certificates/tests/test_support_views.py b/lms/djangoapps/certificates/tests/test_support_views.py
index 47a6774cc0..ccb7ddf19d 100644
--- a/lms/djangoapps/certificates/tests/test_support_views.py
+++ b/lms/djangoapps/certificates/tests/test_support_views.py
@@ -5,8 +5,10 @@ Tests for certificate app views used by the support team.
 import json
 
 import ddt
+from django.conf import settings
 from django.core.urlresolvers import reverse
 from django.test import TestCase
+from django.test.utils import override_settings
 
 from opaque_keys.edx.keys import CourseKey
 from student.tests.factories import UserFactory
@@ -16,6 +18,9 @@ from xmodule.modulestore.tests.django_utils import ModuleStoreTestCase
 from xmodule.modulestore.tests.factories import CourseFactory
 from certificates.models import GeneratedCertificate, CertificateStatuses
 
+FEATURES_WITH_CERTS_ENABLED = settings.FEATURES.copy()
+FEATURES_WITH_CERTS_ENABLED['CERTIFICATES_HTML_VIEW'] = True
+
 
 class CertificateSupportTestCase(TestCase):
     """
@@ -132,6 +137,27 @@ class CertificateSearchTests(CertificateSupportTestCase):
         self.assertEqual(retrieved_cert["grade"], unicode(self.CERT_GRADE))
         self.assertEqual(retrieved_cert["status"], self.CERT_STATUS)
         self.assertEqual(retrieved_cert["type"], self.CERT_MODE)
+        self.assertEqual(retrieved_cert["download_url"], self.CERT_DOWNLOAD_URL)
+
+    @override_settings(FEATURES=FEATURES_WITH_CERTS_ENABLED)
+    def test_download_link(self):
+        self.cert.download_url = ''
+        self.cert.save()
+
+        response = self._search(self.STUDENT_USERNAME)
+        self.assertEqual(response.status_code, 200)
+        results = json.loads(response.content)
+
+        self.assertEqual(len(results), 1)
+        retrieved_cert = results[0]
+
+        self.assertEqual(
+            retrieved_cert["download_url"],
+            reverse(
+                'certificates:html_view',
+                kwargs={"user_id": self.student.id, "course_id": self.CERT_COURSE_KEY}  # pylint: disable=no-member
+            )
+        )
 
     def _search(self, query):
         """Execute a search and return the response. """