[fix] - Add ConditionalExpression expression handler.

Author: beefancohen

src/util/values/expressions/ConditionalExpression.js

@@ -0,0 +1,21 @@
+'use strict';
+
+import getValue from './index';
+
+
+
+/**
+ * Extractor function for a ConditionalExpression type value node.
+ *
+ * @param - value - AST Value object with type `ConditionalExpression`
+ * @returns - The extracted value converted to correct type.
+ */
+export default function extractValueFromConditionalExpression(value) {
+  const {
+    test,
+    alternate,
+    consequent
+  } = value;
+
+  return getValue(test) ? getValue(alternate) : getValue(consequent);
+}

src/util/values/expressions/index.js

@@ -11,6 +11,7 @@ import MemberExpression from './MemberExpression';
 import CallExpression from './CallExpression';
 import UnaryExpression from './UnaryExpression';
 import ThisExpression from './ThisExpression';
+import ConditionalExpression from './ConditionalExpression';
 
 
 
@@ -25,7 +26,8 @@ const TYPES = {
   MemberExpression,
   CallExpression,
   UnaryExpression,
-  ThisExpression
+  ThisExpression,
+  ConditionalExpression
 };
 
 const noop = () => null;
@@ -52,7 +54,8 @@ const LITERAL_TYPES = assign({}, TYPES, {
     const extractedVal = TYPES.UnaryExpression(value);
     return extractedVal === undefined ? null : extractedVal;
   },
-  ThisExpression: noop
+  ThisExpression: noop,
+  ConditionalExpression: noop
 });
 
 /**

tests/src/rules/aria-proptypes.js

@@ -69,6 +69,8 @@ ruleTester.run('aria-proptypes', rule, {
     { code: '<div aria-label={`Close`} />', parserOptions },
     { code: '<div aria-label={foo} />', parserOptions },
     { code: '<div aria-label={foo.bar} />', parserOptions },
+    { code: '<input aria-invalid={error ? "true" : "false"} />', parserOptions },
+    { code: '<input aria-invalid={undefined ? "true" : "false"} />', parserOptions },
 
     // TRISTATE
     { code: '<div aria-checked={true} />', parserOptions },

tests/src/rules/img-has-alt.js

@@ -72,6 +72,8 @@ ruleTester.run('img-has-alt', rule, {
     { code: '<img alt={undefined} role="presentation" />;', parserOptions },
     { code: '<img alt role="presentation" />;', parserOptions },
     { code: '<img alt="this is lit..." role="presentation" />', parserOptions },
+    { code: '<img alt={error ? "not working": "working"} />', parserOptions },
+    { code: '<img alt={undefined ? "working": "not working"} />', parserOptions },
 
     // CUSTOM ELEMENT TESTS FOR STRING OPTION
     { code: '<Avatar alt="foo" />;', options: string, parserOptions },