Commit cae6e82b by Jay Zoldak

Knock 10 more seconds off the acceptance tests.

parent 19ee540a
...@@ -26,6 +26,8 @@ def i_see_only_the_settings_and_values(step): ...@@ -26,6 +26,8 @@ def i_see_only_the_settings_and_values(step):
@step('creating a discussion takes a single click') @step('creating a discussion takes a single click')
def discussion_takes_a_single_click(step): def discussion_takes_a_single_click(step):
assert(not world.is_css_present('.xmodule_DiscussionModule')) component_css = '.xmodule_DiscussionModule'
assert world.is_css_not_present(component_css)
world.css_click("a[data-category='discussion']") world.css_click("a[data-category='discussion']")
assert(world.is_css_present('.xmodule_DiscussionModule')) assert world.is_css_present(component_css)
...@@ -6,8 +6,6 @@ from xmodule.modulestore import Location ...@@ -6,8 +6,6 @@ from xmodule.modulestore import Location
from contentstore.utils import get_modulestore from contentstore.utils import get_modulestore
############### ACTIONS ####################
@step('I have created a Video component$') @step('I have created a Video component$')
def i_created_a_video_component(step): def i_created_a_video_component(step):
world.create_component_instance( world.create_component_instance(
...@@ -19,7 +17,7 @@ def i_created_a_video_component(step): ...@@ -19,7 +17,7 @@ def i_created_a_video_component(step):
@step('I have created a Video component with subtitles$') @step('I have created a Video component with subtitles$')
def i_created_a_video_component(step): def i_created_a_video_component_subtitles(step):
step.given('I have created a Video component') step.given('I have created a Video component')
# Store the current URL so we can return here # Store the current URL so we can return here
...@@ -46,9 +44,11 @@ def does_not_autoplay(_step, video_type): ...@@ -46,9 +44,11 @@ def does_not_autoplay(_step, video_type):
@step('creating a video takes a single click$') @step('creating a video takes a single click$')
def video_takes_a_single_click(_step): def video_takes_a_single_click(_step):
assert(not world.is_css_present('.xmodule_VideoModule')) component_css = '.xmodule_VideoModule'
assert world.is_css_not_present(component_css)
world.css_click("a[data-category='video']") world.css_click("a[data-category='video']")
assert(world.is_css_present('.xmodule_VideoModule')) assert world.is_css_present(component_css)
@step('I edit the component$') @step('I edit the component$')
...@@ -93,7 +93,7 @@ def xml_only_video(step): ...@@ -93,7 +93,7 @@ def xml_only_video(step):
# Create a new Video component, but ensure that it doesn't have # Create a new Video component, but ensure that it doesn't have
# metadata. This allows us to test that we are correctly parsing # metadata. This allows us to test that we are correctly parsing
# out XML # out XML
video = world.ItemFactory.create( world.ItemFactory.create(
parent_location=parent_location, parent_location=parent_location,
category='video', category='video',
data='<video youtube="1.00:%s"></video>' % youtube_id data='<video youtube="1.00:%s"></video>' % youtube_id
......
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