From 5b367e8321279d65771da91b465a1253a00f7b01 Mon Sep 17 00:00:00 2001 From: Don Mitchell Date: Wed, 17 Jul 2013 09:22:34 -0400 Subject: [PATCH] Missing comma on template -> category refactoring And several pep8 changes --- lms/djangoapps/courseware/features/common.py | 21 ++++++++++---------- 1 file changed, 11 insertions(+), 10 deletions(-) diff --git a/lms/djangoapps/courseware/features/common.py b/lms/djangoapps/courseware/features/common.py index 55e82e0e90..7632f22d6a 100644 --- a/lms/djangoapps/courseware/features/common.py +++ b/lms/djangoapps/courseware/features/common.py @@ -5,11 +5,10 @@ from __future__ import absolute_import from lettuce import world, step from nose.tools import assert_equals, assert_in -from lettuce.django import django_url from django.contrib.auth.models import User from student.models import CourseEnrollment from xmodule.modulestore import Location -from xmodule.modulestore.django import _MODULESTORES, modulestore +from xmodule.modulestore.django import modulestore from xmodule.course_module import CourseDescriptor from courseware.courses import get_course_by_id from xmodule import seq_module, vertical_module @@ -19,7 +18,7 @@ logger = getLogger(__name__) @step(u'The course "([^"]*)" exists$') -def create_course(step, course): +def create_course(_step, course): # First clear the modulestore so we don't try to recreate # the same course twice @@ -37,9 +36,10 @@ def create_course(step, course): world.scenario_dict['SECTION'] = world.ItemFactory.create(parent_location=world.scenario_dict['COURSE'].location, display_name='Test Section') - problem_section = world.ItemFactory.create(parent_location=world.scenario_dict['SECTION'].location, - category='sequential' - display_name='Test Section') + world.ItemFactory.create( + parent_location=world.scenario_dict['SECTION'].location, + category='sequential', + display_name='Test Section') @step(u'I am registered for the course "([^"]*)"$') @@ -59,10 +59,11 @@ def i_am_registered_for_the_course(step, course): @step(u'The course "([^"]*)" has extra tab "([^"]*)"$') -def add_tab_to_course(step, course, extra_tab_name): - section_item = world.ItemFactory.create(parent_location=course_location(course), - category="static_tab", - display_name=str(extra_tab_name)) +def add_tab_to_course(_step, course, extra_tab_name): + world.ItemFactory.create( + parent_location=course_location(course), + category="static_tab", + display_name=str(extra_tab_name)) def course_id(course_num):