From e3bd509a398b2c1140c701046b9e680c02bfc0a0 Mon Sep 17 00:00:00 2001 From: Usman Khalid <2200617@gmail.com> Date: Thu, 30 Jul 2015 15:51:56 +0500 Subject: [PATCH] Re-enable skipped video player tests. This reverts commit f58764875c5bca26c50112a955836a0e6cc2cdf0. --- common/test/acceptance/tests/video/test_video_events.py | 3 --- common/test/acceptance/tests/video/test_video_module.py | 2 -- 2 files changed, 5 deletions(-) diff --git a/common/test/acceptance/tests/video/test_video_events.py b/common/test/acceptance/tests/video/test_video_events.py index ec7bceaa30..f851a02bca 100644 --- a/common/test/acceptance/tests/video/test_video_events.py +++ b/common/test/acceptance/tests/video/test_video_events.py @@ -3,7 +3,6 @@ import datetime import json import ddt -from unittest import skip from ..helpers import EventsTestMixin from .test_video_module import VideoBaseTest @@ -196,7 +195,6 @@ class VideoBumperEventsTest(VideoEventsTestMixin): ('edx.video.bumper.stopped', wait_for_state) ) @ddt.unpack - @skip("Failing on master; To see remove is_youtube_available() form base class") def test_video_control_events(self, event_type, action): """ Scenario: Video component with pre-roll emits events correctly @@ -287,7 +285,6 @@ class VideoBumperEventsTest(VideoEventsTestMixin): } self.assert_events_match([expected_event], [video_event]) - @skip("Failing on master; To see remove is_youtube_available() form base class") def test_strict_event_format(self): """ This test makes a very strong assertion about the fields present in events. The goal of it is to ensure that new diff --git a/common/test/acceptance/tests/video/test_video_module.py b/common/test/acceptance/tests/video/test_video_module.py index 16ef14e19a..48ee3b5dd8 100644 --- a/common/test/acceptance/tests/video/test_video_module.py +++ b/common/test/acceptance/tests/video/test_video_module.py @@ -404,7 +404,6 @@ class YouTubeVideoTest(VideoBaseTest): self.assertTrue(self.video.is_video_rendered('html5')) - @skip('Failing on master; To see remove is_youtube_available() form base class') def test_download_transcript_button_works_correctly(self): """ Scenario: Download Transcript button works correctly @@ -713,7 +712,6 @@ class YouTubeVideoTest(VideoBaseTest): self.assertEqual(self.video.caption_languages, {'zh_HANS': 'Simplified Chinese', 'zh_HANT': 'Traditional Chinese'}) - @skip('Failing on master; To see remove is_youtube_available() form base class') def test_video_bumper_render(self): """ Scenario: Multiple videos with bumper in sequentials all load and work, switching between sequentials