From ab3f7c7eece79867d59421ca64c0f9a79a09f796 Mon Sep 17 00:00:00 2001
From: Brian Di Palma <offler@gmail.com>
Date: Tue, 1 Oct 2019 12:28:33 +0100
Subject: [PATCH] fix: shouldExcludeFromA11yTree runs in IE11

IE11 returns `undefined` for missing `parentElement`.
---
 src/__tests__/element-queries.js | 4 ++++
 src/role-helpers.js              | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/__tests__/element-queries.js b/src/__tests__/element-queries.js
index 6d2becac..b9ad2ee0 100644
--- a/src/__tests__/element-queries.js
+++ b/src/__tests__/element-queries.js
@@ -385,6 +385,9 @@ test('queryAllByRole returns semantic html elements', () => {
       <h4>Heading 4</h4>
       <h5>Heading 5</h5>
       <h6>Heading 6</h6>
+      <svg role="img">
+        <rect width="100" height="100" />
+      </svg>
       <ol>
         <li></li>
         <li></li>
@@ -429,6 +432,7 @@ test('queryAllByRole returns semantic html elements', () => {
   expect(queryAllByRole('row')).toHaveLength(3)
   expect(queryAllByRole(/rowgroup/i)).toHaveLength(2)
   expect(queryAllByRole(/(table)|(textbox)/i)).toHaveLength(3)
+  expect(queryAllByRole(/img/i)).toHaveLength(1)
 })
 
 test('getAll* matchers return an array', () => {
diff --git a/src/role-helpers.js b/src/role-helpers.js
index 281e3e0c..14bbe735 100644
--- a/src/role-helpers.js
+++ b/src/role-helpers.js
@@ -26,7 +26,7 @@ function shouldExcludeFromA11yTree(element) {
   let visibility = computedStyle.visibility
 
   let currentElement = element
-  while (currentElement !== null) {
+  while (currentElement) {
     if (currentElement.hidden === true) {
       return true
     }