fix: resolved PR comments

openedx · Oct 11, 2023 · 552b20d · 552b20d
1 parent fcaace3
commit 552b20d
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/lms/djangoapps/certificates/tests/test_webview_views.py b/lms/djangoapps/certificates/tests/test_webview_views.py
@@ -52,8 +52,6 @@
 
 FEATURES_WITH_CERTS_ENABLED = settings.FEATURES.copy()
 FEATURES_WITH_CERTS_ENABLED['CERTIFICATES_HTML_VIEW'] = True
-FEATURES_WITH_BADGES_ENABLED = FEATURES_WITH_CERTS_ENABLED.copy()
-FEATURES_WITH_BADGES_ENABLED['ENABLE_OPENBADGES'] = True
 
 FEATURES_WITH_CERTS_DISABLED = settings.FEATURES.copy()
 FEATURES_WITH_CERTS_DISABLED['CERTIFICATES_HTML_VIEW'] = False
@@ -526,7 +524,6 @@ def test_rendering_course_organization_data(self):
         self.assertContains(response, f'<title>test_organization {self.course.number} Certificate |')
         self.assertContains(response, 'logo_test1.png')
 
-    @override_settings(FEATURES=FEATURES_WITH_BADGES_ENABLED)
     @patch.dict("django.conf.settings.SOCIAL_SHARING_SETTINGS", {
         "CERTIFICATE_TWITTER": True,
         "CERTIFICATE_FACEBOOK": True,

diff --git a/lms/static/js/learner_dashboard/spec/program_card_view_spec.js b/lms/static/js/learner_dashboard/spec/program_card_view_spec.js
@@ -176,4 +176,7 @@ describe('Program card View', () => {
         expect(view.$el.find('.banner-image').attr('srcset')).toEqual('');
     });
 
+    it('should render the subscription badge if subscription is active', () => {
+        expect(view.$('.subscription-badge .badge').html()?.trim()).toEqual('Subscribed');
+    });
 });
diff --git a/lms/static/js/learner_dashboard/spec/program_details_header_spec.js b/lms/static/js/learner_dashboard/spec/program_details_header_spec.js
@@ -82,4 +82,7 @@ describe('Program Details Header View', () => {
             .toEqual(`${context.programData.authoring_organizations[0].name}'s logo`);
     });
 
+    it('should render the subscription badge if subscription is active', () => {
+        expect(view.$('.meta-info .badge').html().trim()).toEqual('Subscribed');
+    });
 });