Fixing JS erros while running against test data

Author: jessebeach

src/rules/no-noninteractive-element-handlers.js

@@ -21,6 +21,8 @@ import isAbstractRole from '../util/isAbstractRole';
 import isHiddenFromScreenReader from '../util/isHiddenFromScreenReader';
 import isInteractiveElement from '../util/isInteractiveElement';
 import isInteractiveRole from '../util/isInteractiveRole';
+import isNonInteractiveElement from '../util/isNonInteractiveElement';
+import isNonInteractiveRole from '../util/isNonInteractiveRole';
 import isPresentationRole from '../util/isPresentationRole';
 
 const errorMessage =
@@ -65,6 +67,10 @@ module.exports = {
       } else if (
         isInteractiveElement(type, attributes)
         || isInteractiveRole(type, attributes)
+        || (
+          !isNonInteractiveElement(type, attributes)
+          && !isNonInteractiveRole(type, attributes)
+        )
         || isAbstractRole(type, attributes)
       ) {
         // This rule has no opinion about abtract roles.

src/util/isInteractiveElement.js

@@ -37,11 +37,11 @@ const pureInteractiveRoleElements = [...elementRoles.entries()]
   ): ElementCallbackMap => {
     const interactiveElements = accumulator;
     const elementName = elementSchema.name;
-    const elementAttributes = elementSchema.attributes || new Map([]);
+    const elementAttributes = elementSchema.attributes || [];
     interactiveElements[elementName] = (attributes: Array<Node>): boolean => {
       const passedAttrCheck =
-        elementAttributes.size === 0 ||
-        [...elementAttributes.values()].every(
+        elementAttributes.length === 0 ||
+        elementAttributes.every(
           (controlAttr): boolean => attributes.some(
             (attr): boolean => (
               controlAttr.name === propName(attr).toLowerCase()