From b741ce786b9f567431fc7352b524172d3ed8f79a Mon Sep 17 00:00:00 2001
From: Chris Dodge <cdodge@edx.org>
Date: Tue, 16 Jun 2015 16:22:56 -0400
Subject: [PATCH] fix python unit tests

---
 common/djangoapps/terrain/steps.py          | 2 +-
 lms/djangoapps/branding/tests/test_page.py  | 6 +++---
 lms/static/js/discovery/result_list_view.js | 2 +-
 lms/templates/courseware/courses.html       | 4 ++--
 4 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/common/djangoapps/terrain/steps.py b/common/djangoapps/terrain/steps.py
index 616991c58d..b44cc2d030 100644
--- a/common/djangoapps/terrain/steps.py
+++ b/common/djangoapps/terrain/steps.py
@@ -65,7 +65,7 @@ def i_should_be_on_the_dashboard(step):
 @step(u'I (?:visit|access|open) the courses page$')
 def i_am_on_the_courses_page(step):
     world.visit('/courses')
-    assert world.is_css_present('section.courses')
+    assert world.is_css_present('div.courses')
 
 
 @step(u'I press the "([^"]*)" button$')
diff --git a/lms/djangoapps/branding/tests/test_page.py b/lms/djangoapps/branding/tests/test_page.py
index ac4357e4f0..4b3c725a8c 100644
--- a/lms/djangoapps/branding/tests/test_page.py
+++ b/lms/djangoapps/branding/tests/test_page.py
@@ -220,7 +220,7 @@ class IndexPageCourseCardsSortingTests(ModuleStoreTestCase):
         self.assertNotIn('<aside aria-label="Refine your search" class="search-facets phone-menu">', response.content)
 
         # make sure we have the special css class on the section
-        self.assertIn('<section class="courses no-course-discovery">', response.content)
+        self.assertIn('<div class="courses no-course-discovery"', response.content)
 
     @patch('student.views.render_to_response', RENDER_MOCK)
     @patch('courseware.views.render_to_response', RENDER_MOCK)
@@ -239,10 +239,10 @@ class IndexPageCourseCardsSortingTests(ModuleStoreTestCase):
         response = self.client.get(reverse('branding.views.courses'))
         self.assertEqual(response.status_code, 200)
 
-        # assert that the course discovery UI is not present
+        # assert that the course discovery UI is present
         self.assertIn('Search for a course', response.content)
         self.assertIn('<aside aria-label="Refine your search" class="search-facets phone-menu">', response.content)
-        self.assertIn('<section class="courses">', response.content)
+        self.assertIn('<div class="courses"', response.content)
 
     @patch('student.views.render_to_response', RENDER_MOCK)
     @patch('courseware.views.render_to_response', RENDER_MOCK)
diff --git a/lms/static/js/discovery/result_list_view.js b/lms/static/js/discovery/result_list_view.js
index ef0b7cc096..1a45575cec 100644
--- a/lms/static/js/discovery/result_list_view.js
+++ b/lms/static/js/discovery/result_list_view.js
@@ -11,7 +11,7 @@ define([
 
     return Backbone.View.extend({
 
-        el: 'section.courses',
+        el: 'div.courses',
         $window: $(window),
         $document: $(document),
 
diff --git a/lms/templates/courseware/courses.html b/lms/templates/courseware/courses.html
index c4b4d1bb37..671fe360d9 100644
--- a/lms/templates/courseware/courses.html
+++ b/lms/templates/courseware/courses.html
@@ -78,7 +78,7 @@
     </div>
     % endif
 
-    <section class="courses ${'' if course_discovery_enabled else ' no-course-discovery'}" role="region" aria-label="${_('List of Courses')}">
+    <div class="courses${'' if course_discovery_enabled else ' no-course-discovery'}" role="region" aria-label="${_('List of Courses')}">
       <ul class="courses-listing">
         %for course in courses:
         <li class="courses-listing-item">
@@ -86,7 +86,7 @@
         </li>
         %endfor
       </ul>
-    </section>
+    </div>
 
 
     % if course_discovery_enabled: