Prioritize aria names over hidden in byRole queries

testing-library · Oct 23, 2021 · aeb7f4a · aeb7f4a
1 parent 5bc9364
commit aeb7f4a
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/src/queries/role.js b/src/queries/role.js
@@ -133,13 +133,6 @@ function queryAllByRole(
       // don't care if aria attributes are unspecified
       return true
     })
-    .filter(element => {
-      return hidden === false
-        ? isInaccessible(element, {
-            isSubtreeInaccessible: cachedIsSubtreeInaccessible,
-          }) === false
-        : true
-    })
     .filter(element => {
       if (name === undefined) {
         // Don't care
@@ -156,6 +149,13 @@ function queryAllByRole(
         text => text,
       )
     })
+    .filter(element => {
+      return hidden === false
+        ? isInaccessible(element, {
+            isSubtreeInaccessible: cachedIsSubtreeInaccessible,
+          }) === false
+        : true
+    })
 }
 
 const getMultipleError = (c, role, {name} = {}) => {