[fix] - Fix resolving element type with multiple member expressions.

jsx-eslint · Nov 15, 2016 · c35c2f5 · c35c2f5
1 parent 9a8397f
commit c35c2f5
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 3 deletions.
diff --git a/__tests__/src/elementType-test.js b/__tests__/src/elementType-test.js
@@ -49,7 +49,7 @@ describe('elementType tests', () => {
     assert.equal(expected, actual);
   });
 
-  it('should return the correct type of the custom object element given its node object', () => {
+  it('should return the correct type of the namespaced element given its node object', () => {
     const code = '<UX:Slider />';
     const node = getOpeningElement(code);
 
@@ -58,4 +58,15 @@ describe('elementType tests', () => {
 
     assert.equal(expected, actual);
   });
+
+  it('should return the correct type of the multiple custom object element given its node object',
+    () => {
+      const code = '<UX.Slider.Blue.Light />';
+      const node = getOpeningElement(code);
+
+      const expected = 'UX.Slider.Blue.Light';
+      const actual = elementType(node);
+
+      assert.equal(expected, actual);
+    });
 });
diff --git a/src/elementType.js b/src/elementType.js
@@ -1,3 +1,11 @@
+function resolveMemberExpressions(object = {}, property = {}) {
+  if (object.type === 'JSXMemberExpression') {
+    return `${resolveMemberExpressions(object.object, object.property)}.${property.name}`;
+  }
+
+  return `${object.name}.${property.name}`;
+}
+
 /**
  * Returns the tagName associated with a JSXElement.
  */
@@ -9,8 +17,8 @@ export default function elementType(node = {}) {
   }
 
   if (name.type === 'JSXMemberExpression') {
-    const { object, property } = name;
-    return `${object.name}.${property.name}`;
+    const { object = {}, property = {} } = name;
+    return resolveMemberExpressions(object, property);
   } else if (name.type === 'JSXNamespacedName') {
     return `${name.namespace.name}:${name.name.name}`;
   }