pr comments

Author: jpdriver

__tests__/src/rules/has-valid-accessibility-state-test.js

@@ -42,7 +42,11 @@ ruleTester.run('has-valid-accessibility-state', rule, {
   valid: [
     { code: '<TouchableOpacity accessibilityState={{ disabled: true }} />;' },
     { code: '<TouchableOpacity accessibilityState={{ checked: true }} />;' },
-    { code: '<TouchableOpacity accessibilityState={{ checked: "mixed" }} />;' }
+    { code: '<TouchableOpacity accessibilityState={{ checked: "mixed" }} />;' },
+    {
+      code:
+        '<TouchableOpacity accessibilityState={{ disabled: true, checked: true }} />;'
+    }
   ].map(parserOptionsMapper),
   invalid: [
     {

docs/rules/has-valid-accessibility-state.md

@@ -26,6 +26,7 @@ This rule takes no arguments.
 <TouchableOpacity accessibilityState={{ disabled: true }} />
 <TouchableOpacity accessibilityState={{ checked: true }} />
 <TouchableOpacity accessibilityState={{ checked: "mixed" }} />
+<TouchableOpacity accessibilityState={{ disabled: true, checked: true }} />
 ```
 
 ### Fail

src/rules/has-valid-accessibility-state.js

@@ -40,17 +40,17 @@ module.exports = {
           Object.entries(attrValue).map(([key, value]) => {
             if (validKeys.indexOf(key) < 0) {
               error(`accessibilityState object: "${key}" is not a valid key`);
-            } else if (key !== 'checked' && typeof value !== 'boolean') {
-              error(
-                `accessibilityState object: "${key}" value is not a boolean`
-              );
             } else if (
               key === 'checked' &&
               !(typeof value === 'boolean' || value === 'mixed')
             ) {
               error(
                 `accessibilityState object: "checked" value is not either a boolean or 'mixed'`
               );
+            } else if (key !== 'checked' && typeof value !== 'boolean') {
+              error(
+                `accessibilityState object: "${key}" value is not a boolean`
+              );
             }
           });
         }