From a1945c18d1171e999977d2bd15157b43533e54b6 Mon Sep 17 00:00:00 2001
From: Mushtaq Ali <mushtaque@edx.org>
Date: Fri, 9 Sep 2016 16:59:27 +0500
Subject: [PATCH] Remove icon-aria-hidden rule for a11y discussions TNL-4637
 TNL-4641 TNL-4645

---
 .../test/acceptance/tests/discussion/test_discussion.py   | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/common/test/acceptance/tests/discussion/test_discussion.py b/common/test/acceptance/tests/discussion/test_discussion.py
index 6450eab277..ae99594527 100644
--- a/common/test/acceptance/tests/discussion/test_discussion.py
+++ b/common/test/acceptance/tests/discussion/test_discussion.py
@@ -211,7 +211,6 @@ class DiscussionHomePageTest(UniqueCourseTest):
         self.page.a11y_audit.config.set_rules({
             "ignore": [
                 'section',  # TODO: AC-491
-                'icon-aria-hidden',  # TNL-4637
             ]
         })
         self.page.a11y_audit.check_for_accessibility_errors()
@@ -456,7 +455,6 @@ class DiscussionTabMultipleThreadTest(BaseDiscussionTestCase):
         self.thread_page_1.a11y_audit.config.set_rules({
             "ignore": [
                 'section',  # TODO: AC-491
-                'icon-aria-hidden',  # TNL-4641
             ]
         })
 
@@ -465,7 +463,6 @@ class DiscussionTabMultipleThreadTest(BaseDiscussionTestCase):
         self.thread_page_2.a11y_audit.config.set_rules({
             "ignore": [
                 'section',  # TODO: AC-491
-                'icon-aria-hidden',  # TNL-4641
             ]
         })
 
@@ -527,7 +524,6 @@ class DiscussionOpenClosedThreadTest(BaseDiscussionTestCase):
             'ignore': [
                 'section',  # TODO: AC-491
                 '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()
@@ -537,7 +533,6 @@ class DiscussionOpenClosedThreadTest(BaseDiscussionTestCase):
             'ignore': [
                 'section',  # TODO: AC-491
                 '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()
@@ -824,7 +819,6 @@ class DiscussionResponseEditTest(BaseDiscussionTestCase):
         page.a11y_audit.config.set_rules({
             'ignore': [
                 'section',  # TODO: AC-491
-                'icon-aria-hidden',  # TNL-4645
             ]
         })
         page.visit()
@@ -922,7 +916,6 @@ class DiscussionCommentEditTest(BaseDiscussionTestCase):
         page.a11y_audit.config.set_rules({
             'ignore': [
                 'section',  # TODO: AC-491
-                'icon-aria-hidden',  # TNL-4645
             ]
         })
         page.a11y_audit.check_for_accessibility_errors()
@@ -1328,7 +1321,6 @@ class DiscussionSearchAlertTest(UniqueCourseTest):
         self.page.a11y_audit.config.set_rules({
             'ignore': [
                 'section',  # TODO: AC-491
-                'icon-aria-hidden',  # TNL-4641
             ]
         })
         self.page.a11y_audit.check_for_accessibility_errors()