Merge pull request #13754 from edx/flaky/video-test

Fix flaky test: test_youtube_id_w_different_local_server_sub
This commit is contained in:
Nimisha Asthagiri
2016-10-18 16:42:43 -04:00
committed by GitHub
2 changed files with 7 additions and 1 deletions

View File

@@ -124,6 +124,12 @@ class VideoComponentPage(VideoPage):
'Video Buffering Completed')
self._wait_for(self.is_controls_visible, 'Player Controls are Visible')
def wait_for_message(self, message_type, expected_message):
"""
Wait until the message of the requested type is as expected.
"""
self._wait_for(lambda: self.message(message_type) == expected_message, "Waiting for message update.")
@wait_for_js
def is_controls_visible(self):
"""

View File

@@ -178,7 +178,7 @@ class VideoTranscriptTest(CMSVideoBaseTest):
self.assertEqual(self.video.message('status'), 'Timed Transcript Conflict')
self.assertTrue(self.video.is_transcript_button_visible('replace'))
self.video.click_button_subtitles()
self.assertEqual(self.video.message('status'), 'Timed Transcript Found')
self.video.wait_for_message('status', 'Timed Transcript Found')
self.open_advanced_tab()
self.assertTrue(self.video.verify_field_value('Default Timed Transcript', 't_neq_exist'))