Skip tests that are too flaky to be reliable. FEDX-88.
This commit is contained in:
@@ -3,6 +3,7 @@ Acceptance tests for Studio related to the asset index page.
|
||||
"""
|
||||
|
||||
from flaky import flaky
|
||||
from unittest import skip
|
||||
|
||||
from ...pages.studio.asset_index import AssetIndexPage
|
||||
|
||||
@@ -11,6 +12,7 @@ from ...fixtures.base import StudioApiLoginError
|
||||
from ..helpers import skip_if_browser
|
||||
|
||||
|
||||
@skip('FEDX-88')
|
||||
class AssetIndexTest(StudioCourseTest):
|
||||
|
||||
"""
|
||||
@@ -32,7 +34,7 @@ class AssetIndexTest(StudioCourseTest):
|
||||
self.course_fixture.add_asset(['image.jpg', 'textbook.pdf'])
|
||||
|
||||
@skip_if_browser('chrome') # TODO Need to fix test_page_existance for this for chrome browser
|
||||
@flaky # TODO fix this TNL-4302
|
||||
@flaky # TODO fix this FEDX-88
|
||||
def test_type_filter_exists(self):
|
||||
"""
|
||||
Make sure type filter is on the page.
|
||||
|
||||
@@ -3,7 +3,6 @@ Acceptance tests for Studio.
|
||||
"""
|
||||
|
||||
from bok_choy.web_app_test import WebAppTest
|
||||
from flaky import flaky
|
||||
|
||||
from ...pages.studio.asset_index import AssetIndexPage
|
||||
from ...pages.studio.auto_auth import AutoAuthPage
|
||||
@@ -80,7 +79,8 @@ class CoursePagesTest(StudioCourseTest):
|
||||
self.pages = [
|
||||
clz(self.browser, self.course_info['org'], self.course_info['number'], self.course_info['run'])
|
||||
for clz in [
|
||||
AssetIndexPage, CourseUpdatesPage,
|
||||
# AssetIndexPage, # TODO: Skip testing this page due to FEDX-88
|
||||
CourseUpdatesPage,
|
||||
PagesPage, ExportCoursePage, ImportCoursePage, CourseTeamPage, CourseOutlinePage, SettingsPage,
|
||||
AdvancedSettingsPage, GradingPage, TextbooksPage
|
||||
]
|
||||
@@ -95,7 +95,6 @@ class CoursePagesTest(StudioCourseTest):
|
||||
self.dashboard_page.visit()
|
||||
self.assertEqual(self.browser.current_url.strip('/').rsplit('/')[-1], 'home')
|
||||
|
||||
@flaky # TODO: FEDX-88
|
||||
def test_page_existence(self):
|
||||
"""
|
||||
Make sure that all these pages are accessible once you have a course.
|
||||
|
||||
Reference in New Issue
Block a user