pylint/pep8 cleanup.

cleanup.

cms/djangoapps/contentstore/tests/test_utils.py

@@ -5,8 +5,6 @@ import collections
 import copy
 from django.test import TestCase
 from django.test.utils import override_settings
-from xmodule.modulestore.tests.factories import CourseFactory
-from xmodule.modulestore.tests.django_utils import ModuleStoreTestCase
 
 
 class LMSLinksTestCase(TestCase):
@@ -78,6 +76,7 @@ class LMSLinksTestCase(TestCase):
             "//localhost:8000/courses/mitX/101/test/jump_to/i4x://mitX/101/course/test"
         )
 
+
 class ExtraPanelTabTestCase(TestCase):
     """ Tests adding and removing extra course tabs. """
 
@@ -151,4 +150,3 @@ class ExtraPanelTabTestCase(TestCase):
                 changed, actual_tabs = utils.remove_extra_panel_tab(tab_type, course)
                 self.assertFalse(changed)
                 self.assertEqual(actual_tabs, expected_tabs)
-

common/lib/xmodule/xmodule/modulestore/tests/test_split_modulestore.py

@@ -332,9 +332,6 @@ class SplitModuleItemTests(SplitModuleTest):
 
         def verify_greek_hero(block):
             self.assertEqual(block.location.course_id, "GreekHero")
-
-
-            # look at this one in detail
             self.assertEqual(len(block.tabs), 6, "wrong number of tabs")
             self.assertEqual(block.display_name, "The Ancient Greek Hero")
             self.assertEqual(block.advertised_start, "Fall 2013")
@@ -463,7 +460,7 @@ class SplitModuleItemTests(SplitModuleTest):
         parents = modulestore().get_parent_locations(locator)
         self.assertEqual(len(parents), 1)
         self.assertEqual(parents[0].usage_id, 'head12345')
-        locator.usage_id='nosuchblock'
+        locator.usage_id = 'nosuchblock'
         parents = modulestore().get_parent_locations(locator)
         self.assertEqual(len(parents), 0)