Fixing aria-invalid=true and aria-invalid=false validation (#126)

Author: fracmak

__tests__/src/rules/aria-proptypes-test.js

@@ -123,6 +123,12 @@ ruleTester.run('aria-proptypes', rule, {
     { code: '<div aria-sort={`other`} />', parserOptions },
     { code: '<div aria-sort={foo} />', parserOptions },
     { code: '<div aria-sort={foo.bar} />', parserOptions },
+    { code: '<div aria-invalid={true} />', parserOptions },
+    { code: '<div aria-invalid="true" />', parserOptions },
+    { code: '<div aria-invalid={false} />', parserOptions },
+    { code: '<div aria-invalid="false" />', parserOptions },
+    { code: '<div aria-invalid="grammar" />', parserOptions },
+    { code: '<div aria-invalid="spelling" />', parserOptions },
 
     // TOKENLIST
     { code: '<div aria-relevant="additions" />', parserOptions },

src/rules/aria-proptypes.js

@@ -42,7 +42,7 @@ const validityCheck = (value, expectedType, permittedValues) => {
       // Booleans resolve to 0/1 values so hard check that it's not first.
       return typeof value !== 'boolean' && isNaN(Number(value)) === false;
     case 'token':
-      return typeof value === 'string' && permittedValues.indexOf(value.toLowerCase()) > -1;
+      return permittedValues.indexOf(typeof value === 'string' ? value.toLowerCase() : value) > -1;
     case 'tokenlist':
       return typeof value === 'string' &&
         value.split(' ').every(token => permittedValues.indexOf(token.toLowerCase()) > -1);

src/util/attributes/ARIA.json

@@ -47,7 +47,7 @@
   },
   "ARIA-INVALID": {
     "type": "token",
-    "values": [ "grammar", "false", "spelling", "true" ]
+    "values": [ "grammar", false, "spelling", true ]
   },
   "ARIA-LABEL": {
     "type": "string"