[fix] - Fix crash when <ELEMENT role /> for role-supports-aria-props

Author: Ethan Cohen

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

@@ -33,7 +33,7 @@ module.exports = context => ({
     // If there is no explicit or implicit role, then assume that the element
     // can handle the global set of aria-* properties.
     // This actually isn't true - should fix in future release.
-    if (!roleValue || ROLES[roleValue.toUpperCase()] === undefined) {
+    if (typeof roleValue !== 'string' || ROLES[roleValue.toUpperCase()] === undefined) {
       return;
     }
 

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 />', parserOptions },
     { code: '<div role="presentation" {...props} />', parserOptions },
     { code: '<Foo.Bar baz={true} />', parserOptions },