From 9c4a3dc9c7ec624dd76c1a01df7c6a61ef345b8f Mon Sep 17 00:00:00 2001 From: Renzo Lucioni Date: Thu, 2 Jun 2016 16:23:44 -0400 Subject: [PATCH] Fix program details a11y test --- common/test/acceptance/tests/lms/test_programs.py | 3 --- lms/static/js/spec/learner_dashboard/course_card_view_spec.js | 4 ++-- lms/templates/learner_dashboard/course_card.underscore | 4 ++-- .../learner_dashboard/program_header_view.underscore | 2 +- 4 files changed, 5 insertions(+), 8 deletions(-) diff --git a/common/test/acceptance/tests/lms/test_programs.py b/common/test/acceptance/tests/lms/test_programs.py index 5a29d1b9b9..4695b583fc 100644 --- a/common/test/acceptance/tests/lms/test_programs.py +++ b/common/test/acceptance/tests/lms/test_programs.py @@ -1,6 +1,4 @@ """Acceptance tests for LMS-hosted Programs pages""" -from unittest import skip - from nose.plugins.attrib import attr from ...fixtures.programs import ProgramsFixture, ProgramsConfigMixin @@ -139,7 +137,6 @@ class ProgramListingPageA11yTest(ProgramPageBase): @attr('a11y') -@skip('The tested page is currently disabled. This test will be re-enabled once a11y failures are resolved.') class ProgramDetailsPageA11yTest(ProgramPageBase): """Test program details page accessibility.""" def setUp(self): diff --git a/lms/static/js/spec/learner_dashboard/course_card_view_spec.js b/lms/static/js/spec/learner_dashboard/course_card_view_spec.js index 5fa267b361..b7de0bc487 100644 --- a/lms/static/js/spec/learner_dashboard/course_card_view_spec.js +++ b/lms/static/js/spec/learner_dashboard/course_card_view_spec.js @@ -61,7 +61,7 @@ define([ expect(view.$('.header-img').attr('src')).toEqual(context.run_modes[0].course_image_url); expect(view.$('.course-details .course-title-link').text().trim()).toEqual(context.display_name); expect(view.$('.course-details .course-title-link').attr('href')).toEqual( - context.run_modes[0].marketing_url); + context.run_modes[0].course_url); expect(view.$('.course-details .course-text .course-key').html()).toEqual(context.key); expect(view.$('.course-details .course-text .run-period').html()) .toEqual(context.run_modes[0].start_date + ' - ' + context.run_modes[0].end_date); @@ -71,7 +71,7 @@ define([ expect(view.$('.header-img').attr('src')).toEqual(context.run_modes[0].course_image_url); expect(view.$('.course-details .course-title-link').text().trim()).toEqual(context.display_name); expect(view.$('.course-details .course-title-link').attr('href')).toEqual( - context.run_modes[0].marketing_url); + context.run_modes[0].course_url); expect(view.$('.course-details .course-text .course-key').html()).toEqual(context.key); expect(view.$('.course-details .course-text .run-period').html()).not.toBeDefined(); }); diff --git a/lms/templates/learner_dashboard/course_card.underscore b/lms/templates/learner_dashboard/course_card.underscore index 1f4a923d15..01ba2dc4f4 100644 --- a/lms/templates/learner_dashboard/course_card.underscore +++ b/lms/templates/learner_dashboard/course_card.underscore @@ -1,6 +1,6 @@
- +

- + <%- display_name %>

diff --git a/lms/templates/learner_dashboard/program_header_view.underscore b/lms/templates/learner_dashboard/program_header_view.underscore index a6114b1275..aa9ac727ff 100644 --- a/lms/templates/learner_dashboard/program_header_view.underscore +++ b/lms/templates/learner_dashboard/program_header_view.underscore @@ -5,7 +5,7 @@

<%- name %>

<%- subtitle %>

-<%- gettext('Programs') %> +<%- gettext('Programs') %> <%- StringUtils.interpolate( gettext('{category}\'s program'), {category: category}