Merge pull request #2044 from edx/fix/cdodge/fix-test-on-css-class

fix broken tests. Also drive by fix for the sporadic failure on tests wh...

cms/djangoapps/contentstore/tests/test_contentstore.py

@@ -646,7 +646,7 @@ class ContentStoreTest(ModuleStoreTestCase):
         resp = self.client.get(reverse('index'))
         self.assertContains(
             resp,
-            '<h1 class="title-1">My Courses</h1>',
+            '<h1 class="page-header">My Courses</h1>',
             status_code=200,
             html=True
         )

cms/djangoapps/contentstore/tests/test_i18n.py

@@ -48,7 +48,7 @@ class InternationalizationTest(ModuleStoreTestCase):
 
         resp = self.client.get(reverse('index'))
         self.assertContains(resp,
-                            '<h1 class="title-1">My Courses</h1>',
+                            '<h1 class="page-header">My Courses</h1>',
                             status_code=200,
                             html=True)
 
@@ -63,7 +63,7 @@ class InternationalizationTest(ModuleStoreTestCase):
                                )
 
         self.assertContains(resp,
-                            '<h1 class="title-1">My Courses</h1>',
+                            '<h1 class="page-header">My Courses</h1>',
                             status_code=200,
                             html=True)
 

lms/djangoapps/courseware/tests/tests.py

@@ -631,8 +631,8 @@ class TestViewAuth(LoginEnrollmentTestCase):
             urls = reverse_urls(['info', 'progress'], course)
             urls.extend([
                 reverse('book', kwargs={'course_id': course.id,
-                                        'book_index': book.title})
-                for book in course.textbooks
+                                        'book_index': index})
+                for index, book in enumerate(course.textbooks)
             ])
             return urls
 
@@ -643,8 +643,6 @@ class TestViewAuth(LoginEnrollmentTestCase):
             """
             urls = reverse_urls(['about_course'], course)
             urls.append(reverse('courses'))
-            # Need separate test for change_enrollment, since it's a POST view
-            #urls.append(reverse('change_enrollment'))
 
             return urls