[fix] - Handle spread attributes in multiple rules.

May fix #59

Author: beefancohen

src/rules/aria-unsupported-elements.js

@@ -30,6 +30,10 @@ module.exports = context => ({
     const invalidAttributes = Object.keys(ARIA).concat('ROLE');
 
     node.attributes.forEach(prop => {
+      if (prop.type === 'JSXSpreadAttribute') {
+        return;
+      }
+
       if (invalidAttributes.indexOf(prop.name.name.toUpperCase()) > -1) {
         context.report({
           node,

src/rules/role-supports-aria-props.js

@@ -43,6 +43,10 @@ module.exports = context => ({
       .filter(attribute => propertySet.indexOf(attribute) === -1);
 
     node.attributes.forEach(prop => {
+      if (prop.type === 'JSXSpreadAttribute') {
+        return;
+      }
+
       if (invalidAriaPropsForRole.indexOf(prop.name.name.toUpperCase()) > -1) {
         context.report({
           node,

tests/src/rules/aria-unsupported-elements.js

@@ -57,15 +57,15 @@ const ariaValidityTests = Object.keys(DOM).map(element => {
 const invalidRoleValidityTests = Object.keys(DOM)
   .filter(element => Boolean(DOM[element].reserved))
   .map(reservedElem => ({
-    code: `<${reservedElem} role />`,
+    code: `<${reservedElem} role {...props} />`,
     errors: [errorMessage('role')],
     parserOptions,
   }));
 
 const invalidAriaValidityTests = Object.keys(DOM)
   .filter(element => Boolean(DOM[element].reserved))
   .map(reservedElem => ({
-    code: `<${reservedElem} aria-hidden />`,
+    code: `<${reservedElem} aria-hidden {...props} />`,
     errors: [errorMessage('aria-hidden')],
     parserOptions,
   }));

tests/src/rules/role-supports-aria-props.js

@@ -70,6 +70,7 @@ ruleTester.run('role-supports-aria-props', rule, {
     { code: '<Foo bar />', parserOptions },
     { code: '<div />', parserOptions },
     { code: '<div id="main" />', parserOptions },
+    { code: '<div role="presentation" {...props} />', parserOptions },
     { code: '<Foo.Bar baz={true} />', parserOptions },
 
     // IMPLICIT ROLE TESTS