Commit 5434f275 by Peter Fogg

Cleaning up code to fix pylint/pep8 errors.

parent 2277c82e
...@@ -3,4 +3,4 @@ Feature: Video Component ...@@ -3,4 +3,4 @@ Feature: Video Component
Scenario: Autoplay is disabled in Studio Scenario: Autoplay is disabled in Studio
Given I have created a Video component Given I have created a Video component
Then when I view it it does not autoplay Then when I view the video it does not have autoplay enabled
\ No newline at end of file
#pylint: disable=C0111 #pylint: disable=C0111
from lettuce import world, step from lettuce import world, step
from common import *
############### ACTIONS #################### ############### ACTIONS ####################
@step('when I view it it does not autoplay') @step('when I view the video it does not have autoplay enabled')
def does_not_autoplay(step): def does_not_autoplay(step):
assert world.css_find('.video')[0]['data-autoplay'] == 'False' assert world.css_find('.video')[0]['data-autoplay'] == 'False'
assert world.css_find('.video_control')[0].has_class('play') assert world.css_find('.video_control')[0].has_class('play')
...@@ -20,7 +20,7 @@ logger = getLogger(__name__) ...@@ -20,7 +20,7 @@ logger = getLogger(__name__)
TEST_COURSE_ORG = 'edx' TEST_COURSE_ORG = 'edx'
TEST_COURSE_NAME = 'Test Course' TEST_COURSE_NAME = 'Test Course'
TEST_SECTION_NAME = "Problem" TEST_SECTION_NAME = 'Test Section'
@step(u'The course "([^"]*)" exists$') @step(u'The course "([^"]*)" exists$')
......
...@@ -3,4 +3,4 @@ Feature: Video component ...@@ -3,4 +3,4 @@ Feature: Video component
Scenario: Autoplay is enabled in LMS Scenario: Autoplay is enabled in LMS
Given the course has a Video component Given the course has a Video component
Then when I view it it does autoplay Then when I view the video it has autoplay enabled
\ No newline at end of file
#pylint: disable=C0111 #pylint: disable=C0111
from lettuce import world, step from lettuce import world, step
from common import * from lettuce.django import django_url
from common import TEST_COURSE_NAME, TEST_SECTION_NAME, i_am_registered_for_the_course, section_location
############### ACTIONS #################### ############### ACTIONS ####################
@step('when I view it it does autoplay') @step('when I view the video it has autoplay enabled')
def does_autoplay(step): def does_autoplay(step):
assert(world.css_find('.video')[0]['data-autoplay'] == 'True') assert(world.css_find('.video')[0]['data-autoplay'] == 'True')
...@@ -17,7 +18,7 @@ def view_video(step): ...@@ -17,7 +18,7 @@ def view_video(step):
i_am_registered_for_the_course(step, coursename) i_am_registered_for_the_course(step, coursename)
# Make sure we have a video # Make sure we have a video
video = add_video_to_course(coursename) add_video_to_course(coursename)
chapter_name = TEST_SECTION_NAME.replace(" ", "_") chapter_name = TEST_SECTION_NAME.replace(" ", "_")
section_name = chapter_name section_name = chapter_name
url = django_url('/courses/edx/Test_Course/Test_Course/courseware/%s/%s' % url = django_url('/courses/edx/Test_Course/Test_Course/courseware/%s/%s' %
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment