Merge pull request #419 from edx/dhm/template_acceptance

Missing comma on template -> category refactoring

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):