From c6d237315621c749989a2973b619e3b3f0a362d6 Mon Sep 17 00:00:00 2001 From: raeeschachar Date: Wed, 10 Feb 2016 17:51:06 +0500 Subject: [PATCH] Fixed test_video_component_stores_speed_correctly_for_multiple_videos failing on Chrome --- common/test/acceptance/pages/lms/course_nav.py | 2 ++ common/test/acceptance/pages/lms/video/video.py | 2 ++ common/test/acceptance/tests/video/test_video_module.py | 3 +++ 3 files changed, 7 insertions(+) diff --git a/common/test/acceptance/pages/lms/course_nav.py b/common/test/acceptance/pages/lms/course_nav.py index d2f2d6e543..e8a13d72f1 100644 --- a/common/test/acceptance/pages/lms/course_nav.py +++ b/common/test/acceptance/pages/lms/course_nav.py @@ -126,6 +126,8 @@ class CourseNavPage(PageObject): # Convert the list index (starts at 0) to a CSS index (starts at 1) seq_css = "ol#sequence-list>li:nth-of-type({0})>a".format(seq_index + 1) self.q(css=seq_css).first.click() + # Click triggers an ajax event + self.wait_for_ajax() def _section_titles(self): """ diff --git a/common/test/acceptance/pages/lms/video/video.py b/common/test/acceptance/pages/lms/video/video.py index 6e421b018c..1a33d8c6af 100644 --- a/common/test/acceptance/pages/lms/video/video.py +++ b/common/test/acceptance/pages/lms/video/video.py @@ -443,6 +443,8 @@ class VideoPage(PageObject): speed_selector = self.get_element_selector('li[data-speed="{speed}"] .control'.format(speed=speed)) self.q(css=speed_selector).first.click() + # Click triggers an ajax event + self.wait_for_ajax() def verify_speed_changed(self, expected_speed): """ diff --git a/common/test/acceptance/tests/video/test_video_module.py b/common/test/acceptance/tests/video/test_video_module.py index 0ed1493e2c..f90215bffd 100644 --- a/common/test/acceptance/tests/video/test_video_module.py +++ b/common/test/acceptance/tests/video/test_video_module.py @@ -679,14 +679,17 @@ class YouTubeVideoTest(VideoBaseTest): # select the "2.0" speed on video "A" self.course_nav.go_to_sequential('A') + self.video.wait_for_video_player_render() self.video.speed = '2.0' # select the "0.50" speed on video "B" self.course_nav.go_to_sequential('B') + self.video.wait_for_video_player_render() self.video.speed = '0.50' # open video "C" self.course_nav.go_to_sequential('C') + self.video.wait_for_video_player_render() # Since the playback speed was set to .5 in "B", this video will also be impacted # because a playback speed has never explicitly been set for it. However, this video