Handle JSXMemberExpression as node type names.

Author: beefancohen

.eslintignore

@@ -1,3 +1,4 @@
 node_modules/**
 reports/**
 index.js
+lib/

src/rules/img-uses-alt.js

@@ -9,13 +9,14 @@
 // ----------------------------------------------------------------------------
 
 import hasAttribute from '../util/hasAttribute';
+import getNodeType from '../util/getNodeType';
 
 const errorMessage = type => `${type} elements must have an alt tag.`;
 
 module.exports = context => ({
   JSXOpeningElement: node => {
     const typeCheck = [ 'img' ].concat(context.options[0]);
-    const nodeType = node.name.name;
+    const nodeType = getNodeType(node);
 
     // Only check 'img' elements and custom types.
     if (typeCheck.indexOf(nodeType) === -1) {

src/rules/label-uses-for.js

@@ -9,14 +9,15 @@
 // ----------------------------------------------------------------------------
 
 import hasAttribute from '../util/hasAttribute';
+import getNodeType from '../util/getNodeType';
 
 const errorMessage = 'Form controls using a label to identify them must be ' +
   'programmatically associated with the control using htmlFor';
 
 module.exports = context => ({
   JSXOpeningElement: node => {
     const typeCheck = [ 'label' ].concat(context.options[0]);
-    const nodeType = node.name.name;
+    const nodeType = getNodeType(node);
 
     // Only check 'label' elements and custom types.
     if (typeCheck.indexOf(nodeType) === -1) {

src/rules/no-hash-href.js

@@ -9,13 +9,14 @@
 // ----------------------------------------------------------------------------
 
 import hasAttribute from '../util/hasAttribute';
+import getNodeType from '../util/getNodeType';
 
 const errorMessage = 'Links must not point to "#". Use a more descriptive href or use a button instead.';
 
 module.exports = context => ({
   JSXOpeningElement: node => {
     const typeCheck = [ 'a' ].concat(context.options[0]);
-    const nodeType = node.name.name;
+    const nodeType = getNodeType(node);
 
     // Only check 'a' elements and custom types.
     if (typeCheck.indexOf(nodeType) === -1) {

src/rules/onclick-uses-role.js

@@ -8,6 +8,7 @@
 import isHiddenFromScreenReader from '../util/isHiddenFromScreenReader';
 import isInteractiveElement from '../util/isInteractiveElement';
 import hasAttribute from '../util/hasAttribute';
+import getNodeType from '../util/getNodeType';
 
 // ----------------------------------------------------------------------------
 // Rule Definition
@@ -24,7 +25,7 @@ module.exports = context => ({
     }
 
     const isVisible = isHiddenFromScreenReader(attributes) === false;
-    const isNonInteractive = isInteractiveElement(node.name.name, attributes) === false;
+    const isNonInteractive = isInteractiveElement(getNodeType(node), attributes) === false;
     const noRoleAttribute = hasAttribute(attributes, 'role') === false;
 
     // Visible, non-interactive elements require role attribute.

src/rules/redundant-alt.js

@@ -10,6 +10,7 @@
 
 import hasAttribute from '../util/hasAttribute';
 import isHiddenFromScreenReader from '../util/isHiddenFromScreenReader';
+import getNodeType from '../util/getNodeType';
 
 const REDUNDANT_WORDS = [
   'image',
@@ -22,7 +23,7 @@ const errorMessage = 'Redundant alt attribute. Screen-readers already announce `
 
 module.exports = context => ({
   JSXOpeningElement: node => {
-    const type = node.name.name;
+    const type = getNodeType(node);
     if (type !== 'img') {
       return;
     }

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

@@ -54,6 +54,7 @@ ruleTester.run('img-uses-alt', rule, {
     { code: '<div alt={function(e) {} } />', parserOptions },
     { code: '<img alt={() => void 0} />', parserOptions },
     { code: '<IMG />', parserOptions },
+    { code: '<UX.Layout>test</UX.Layout>', parserOptions },
 
     // CUSTOM ELEMENT TESTS FOR STRING OPTION
     { code: '<Avatar alt="foo" />;', options: string, parserOptions },

tests/src/rules/label-uses-for.js

@@ -45,6 +45,7 @@ ruleTester.run('label-uses-for', rule, {
     { code: '<label htmlFor="foo">Test!</label>', parserOptions },
     { code: '<Label />', parserOptions }, // lower-case convention refers to real HTML elements.
     { code: '<Label htmlFor="foo" />', parserOptions },
+    { code: '<UX.Layout>test</UX.Layout>', parserOptions },
 
     // CUSTOM ELEMENT STRING OPTION TESTS
     { code: '<Label htmlFor="foo" />', options: string, parserOptions },

tests/src/rules/no-hash-href.js

@@ -48,6 +48,7 @@ ruleTester.run('no-hash-href', rule, {
     { code: '<a href={`#foo`}/>', parserOptions },
     { code: '<a href={"foo"}/>', parserOptions },
     { code: '<a href="#foo" />', parserOptions },
+    { code: '<UX.Layout>test</UX.Layout>', parserOptions },
 
     // CUSTOM ELEMENT TEST FOR STRING OPTION
     { code: '<Link />;', options: string, parserOptions },

tests/src/rules/redundant-alt.js

@@ -54,7 +54,8 @@ ruleTester.run('redundant-alt', rule, {
     { code: '<img alt={"undefined"} />', parserOptions },
     { code: '<img alt={() => {}} />', parserOptions },
     { code: '<img alt={function(e){}} />', parserOptions },
-    { code: '<img aria-hidden={false} alt="Doing cool things." />', parserOptions }
+    { code: '<img aria-hidden={false} alt="Doing cool things." />', parserOptions },
+    { code: '<UX.Layout>test</UX.Layout>', parserOptions }
   ],
   invalid: [
     { code: '<img alt="Photo of friend." />;', errors: [ expectedError ], parserOptions },