Merge pull request #23486 from edx/feanil/flaky_a11y
Get rid of 2 flaky a11y tests.
This commit is contained in:
@@ -83,19 +83,3 @@ class CourseHomeA11yTest(CourseHomeBaseTest):
|
||||
]
|
||||
})
|
||||
course_home_page.a11y_audit.check_for_accessibility_errors()
|
||||
|
||||
def test_course_search_a11y(self):
|
||||
"""
|
||||
Test the accessibility of the search results page.
|
||||
"""
|
||||
course_home_page = CourseHomePage(self.browser, self.course_id)
|
||||
course_home_page.visit()
|
||||
course_search_results_page = course_home_page.search_for_term("Test Search")
|
||||
course_search_results_page.a11y_audit.config.set_rules({
|
||||
"ignore": [
|
||||
'aria-valid-attr', # TODO: LEARNER-6611 & LEARNER-6865
|
||||
'region', # TODO: AC-932
|
||||
'landmark-no-duplicate-banner', # TODO: AC-934
|
||||
]
|
||||
})
|
||||
course_search_results_page.a11y_audit.check_for_accessibility_errors()
|
||||
|
||||
@@ -493,16 +493,6 @@ class DataDownloadsTest(BaseInstructorDashboardTest):
|
||||
self.data_download_section.wait_for_available_report()
|
||||
self.verify_report_download(report_name)
|
||||
|
||||
@attr('a11y')
|
||||
def test_data_download_a11y(self):
|
||||
"""
|
||||
Data download page accessibility tests
|
||||
"""
|
||||
self.data_download_section.a11y_audit.config.set_scope([
|
||||
'.data-download-container'
|
||||
])
|
||||
self.data_download_section.a11y_audit.check_for_accessibility_errors()
|
||||
|
||||
|
||||
@ddt.ddt
|
||||
class DataDownloadsWithMultipleRoleTests(BaseInstructorDashboardTest):
|
||||
|
||||
Reference in New Issue
Block a user