fix(byRole): Don't assume window context (#378)

eps1lon · Kent C. Dodds · commit bb3c145efb5d · 2019-10-10T09:44:01.000-06:00
* fix(shouldExcludeFromA11yTree): Don't assume window contextg

* Add test
diff --git a/src/__node_tests__/index.js b/src/__node_tests__/index.js
@@ -76,3 +76,23 @@ test('works without a browser context on a dom node (JSDOM Fragment)', () => {
     />
   `)
 })
+
+test('byRole works without a global DOM', () => {
+  const {
+    window: {
+      document: {body: container},
+    },
+  } = new JSDOM(`
+    <html>
+      <body>
+        <button>Say "Hello, Dave!"</button>
+      </body>
+    </html>
+  `)
+
+  expect(dtl.getByRole(container, 'button')).toMatchInlineSnapshot(`
+    <button>
+      Say "Hello, Dave!"
+    </button>
+  `)
+})
diff --git a/src/role-helpers.js b/src/role-helpers.js
@@ -15,6 +15,7 @@ const elementRoleList = buildElementRoleList(elementRoles)
  * @returns {boolean} true if excluded, otherwise false
  */
 function shouldExcludeFromA11yTree(element) {
+  const window = element.ownerDocument.defaultView
   const computedStyle = window.getComputedStyle(element)
   // since visibility is inherited we can exit early
   if (computedStyle.visibility === 'hidden') {
