update test and add note

jsx-eslint · Sep 4, 2024 · 67210fc · 67210fc
1 parent f842fde
commit 67210fc
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/__tests__/src/rules/no-interactive-element-to-noninteractive-role-test.js b/__tests__/src/rules/no-interactive-element-to-noninteractive-role-test.js
@@ -372,6 +372,7 @@ const neverValid = [
   { code: '<select className="foo" role="listitem" />', errors: [expectedError] },
   { code: '<textarea className="foo" role="listitem" />', errors: [expectedError] },
   { code: '<tr role="listitem" />;', errors: [expectedError] },
+  { code: '<summary role="listitem" />;', errors: [expectedError] },
   /* Custom elements */
   { code: '<Link href="http://x.y.z" role="img" />', errors: [expectedError], settings: componentsSettings },
 ];

diff --git a/src/util/isInteractiveElement.js b/src/util/isInteractiveElement.js
@@ -102,7 +102,10 @@ function checkIsInteractiveElement(tagName, attributes): boolean {
   }
   // Check in elementAXObjects for AX Tree associations for this element.
   const isInteractiveAXElement = some(iterFrom(interactiveElementAXObjectSchemas), elementSchemaMatcher);
-  if (isInteractiveAXElement) {
+  if (
+    isInteractiveAXElement
+    || tagName === 'summary' // TODO: Remove this hard-coded addition once axobject-query is updated.
+  ) {
     return true;
   }