Commit a1945c18 by Mushtaq Ali

Remove icon-aria-hidden rule for a11y discussions

TNL-4637 TNL-4641 TNL-4645
parent dd876d8b
...@@ -211,7 +211,6 @@ class DiscussionHomePageTest(UniqueCourseTest): ...@@ -211,7 +211,6 @@ class DiscussionHomePageTest(UniqueCourseTest):
self.page.a11y_audit.config.set_rules({ self.page.a11y_audit.config.set_rules({
"ignore": [ "ignore": [
'section', # TODO: AC-491 'section', # TODO: AC-491
'icon-aria-hidden', # TNL-4637
] ]
}) })
self.page.a11y_audit.check_for_accessibility_errors() self.page.a11y_audit.check_for_accessibility_errors()
...@@ -456,7 +455,6 @@ class DiscussionTabMultipleThreadTest(BaseDiscussionTestCase): ...@@ -456,7 +455,6 @@ class DiscussionTabMultipleThreadTest(BaseDiscussionTestCase):
self.thread_page_1.a11y_audit.config.set_rules({ self.thread_page_1.a11y_audit.config.set_rules({
"ignore": [ "ignore": [
'section', # TODO: AC-491 'section', # TODO: AC-491
'icon-aria-hidden', # TNL-4641
] ]
}) })
...@@ -465,7 +463,6 @@ class DiscussionTabMultipleThreadTest(BaseDiscussionTestCase): ...@@ -465,7 +463,6 @@ class DiscussionTabMultipleThreadTest(BaseDiscussionTestCase):
self.thread_page_2.a11y_audit.config.set_rules({ self.thread_page_2.a11y_audit.config.set_rules({
"ignore": [ "ignore": [
'section', # TODO: AC-491 'section', # TODO: AC-491
'icon-aria-hidden', # TNL-4641
] ]
}) })
...@@ -527,7 +524,6 @@ class DiscussionOpenClosedThreadTest(BaseDiscussionTestCase): ...@@ -527,7 +524,6 @@ class DiscussionOpenClosedThreadTest(BaseDiscussionTestCase):
'ignore': [ 'ignore': [
'section', # TODO: AC-491 'section', # TODO: AC-491
'color-contrast', # Commented out for now because they reproducibly fail on Jenkis but not locally 'color-contrast', # Commented out for now because they reproducibly fail on Jenkis but not locally
'icon-aria-hidden', # TNL-4645
] ]
}) })
page.a11y_audit.check_for_accessibility_errors() page.a11y_audit.check_for_accessibility_errors()
...@@ -537,7 +533,6 @@ class DiscussionOpenClosedThreadTest(BaseDiscussionTestCase): ...@@ -537,7 +533,6 @@ class DiscussionOpenClosedThreadTest(BaseDiscussionTestCase):
'ignore': [ 'ignore': [
'section', # TODO: AC-491 'section', # TODO: AC-491
'color-contrast', # Commented out for now because they reproducibly fail on Jenkis but not locally 'color-contrast', # Commented out for now because they reproducibly fail on Jenkis but not locally
'icon-aria-hidden', # TNL-4645
] ]
}) })
page.a11y_audit.check_for_accessibility_errors() page.a11y_audit.check_for_accessibility_errors()
...@@ -824,7 +819,6 @@ class DiscussionResponseEditTest(BaseDiscussionTestCase): ...@@ -824,7 +819,6 @@ class DiscussionResponseEditTest(BaseDiscussionTestCase):
page.a11y_audit.config.set_rules({ page.a11y_audit.config.set_rules({
'ignore': [ 'ignore': [
'section', # TODO: AC-491 'section', # TODO: AC-491
'icon-aria-hidden', # TNL-4645
] ]
}) })
page.visit() page.visit()
...@@ -922,7 +916,6 @@ class DiscussionCommentEditTest(BaseDiscussionTestCase): ...@@ -922,7 +916,6 @@ class DiscussionCommentEditTest(BaseDiscussionTestCase):
page.a11y_audit.config.set_rules({ page.a11y_audit.config.set_rules({
'ignore': [ 'ignore': [
'section', # TODO: AC-491 'section', # TODO: AC-491
'icon-aria-hidden', # TNL-4645
] ]
}) })
page.a11y_audit.check_for_accessibility_errors() page.a11y_audit.check_for_accessibility_errors()
...@@ -1328,7 +1321,6 @@ class DiscussionSearchAlertTest(UniqueCourseTest): ...@@ -1328,7 +1321,6 @@ class DiscussionSearchAlertTest(UniqueCourseTest):
self.page.a11y_audit.config.set_rules({ self.page.a11y_audit.config.set_rules({
'ignore': [ 'ignore': [
'section', # TODO: AC-491 'section', # TODO: AC-491
'icon-aria-hidden', # TNL-4641
] ]
}) })
self.page.a11y_audit.check_for_accessibility_errors() self.page.a11y_audit.check_for_accessibility_errors()
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment