From 13ef870b1bf09f709509fa8469d9bcb90d5655bb Mon Sep 17 00:00:00 2001 From: cahrens Date: Wed, 14 Aug 2013 09:54:30 -0400 Subject: [PATCH] pylint/pep8 cleanup. cleanup. --- cms/djangoapps/contentstore/tests/test_utils.py | 4 +--- .../xmodule/modulestore/tests/test_split_modulestore.py | 5 +---- 2 files changed, 2 insertions(+), 7 deletions(-) diff --git a/cms/djangoapps/contentstore/tests/test_utils.py b/cms/djangoapps/contentstore/tests/test_utils.py index 30e9e29451..c3335aaaa0 100644 --- a/cms/djangoapps/contentstore/tests/test_utils.py +++ b/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) - diff --git a/common/lib/xmodule/xmodule/modulestore/tests/test_split_modulestore.py b/common/lib/xmodule/xmodule/modulestore/tests/test_split_modulestore.py index 086d85b72c..9976a33a00 100644 --- a/common/lib/xmodule/xmodule/modulestore/tests/test_split_modulestore.py +++ b/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)