diff --git a/common/test/acceptance/pages/studio/video/video.py b/common/test/acceptance/pages/studio/video/video.py index 36effb74de..28d0016c3a 100644 --- a/common/test/acceptance/pages/studio/video/video.py +++ b/common/test/acceptance/pages/studio/video/video.py @@ -9,6 +9,7 @@ from bok_choy.javascript import wait_for_js, js_defined from ....tests.helpers import YouTubeStubConfig from ...lms.video.video import VideoPage from selenium.webdriver.common.keys import Keys +from ..utils import wait_for_notification CLASS_SELECTORS = { @@ -132,7 +133,7 @@ class VideoComponentPage(VideoPage): """ return self.q(css=CLASS_SELECTORS['video_controls']).visible - def click_button(self, button_name, index=0): + def click_button(self, button_name, index=0, require_notification=False): """ Click on a button as specified by `button_name` @@ -142,6 +143,8 @@ class VideoComponentPage(VideoPage): """ self.q(css=BUTTON_SELECTORS[button_name]).nth(index).click() + if require_notification: + wait_for_notification(self) self.wait_for_ajax() @staticmethod @@ -237,7 +240,7 @@ class VideoComponentPage(VideoPage): Create a Video Component by clicking on Video button and wait for rendering completion. """ # Create video - self.click_button('create_video') + self.click_button('create_video', require_notification=True) self.wait_for_video_component_render() def xblocks(self): diff --git a/common/test/acceptance/tests/video/test_studio_video_module.py b/common/test/acceptance/tests/video/test_studio_video_module.py index 1c25dab69b..7f8da4afad 100644 --- a/common/test/acceptance/tests/video/test_studio_video_module.py +++ b/common/test/acceptance/tests/video/test_studio_video_module.py @@ -4,7 +4,7 @@ Acceptance tests for CMS Video Module. """ from nose.plugins.attrib import attr -from unittest import skipIf, skip +from unittest import skipIf from ...pages.studio.auto_auth import AutoAuthPage from ...pages.studio.overview import CourseOutlinePage from ...pages.studio.video.video import VideoComponentPage @@ -227,7 +227,6 @@ class CMSVideoTest(CMSVideoBaseTest): self.assertFalse(self.video.is_captions_visible()) - @skip('TNL-392') def test_video_controls_shown_correctly(self): """ Scenario: Video controls for all videos show correctly