fix broken tests. Also drive by fix for the sporadic failure on tests which may…

fix broken tests. Also drive by fix for the sporadic failure on tests which may go to a textbook page. The URL formatting was incorrect in the test

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