[Fix] Make displayNameOfNode return function.name even if it is falsy

- previously it would return the function itself
enzymejs · Apr 10, 2017 · 85447ed · 85447ed
1 parent c8f2185
commit 85447ed
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/src/Utils.js b/src/Utils.js
@@ -430,5 +430,5 @@ export function displayNameOfNode(node) {
 
   if (!type) return null;
 
-  return type.displayName || (typeof type === 'function' ? functionName(type) : type.name) || type;
+  return type.displayName || (typeof type === 'function' ? functionName(type) : type.name || type);
 }
diff --git a/test/Utils-spec.jsx b/test/Utils-spec.jsx
@@ -598,6 +598,14 @@ describe('Utils', () => {
         expect(displayNameOfNode(<Foo />)).to.equal('Foo');
       });
 
+      it('should return the name even if it is falsy', () => {
+        const makeFoo = () => () => <div />;
+
+        const Foo = makeFoo();
+
+        expect(displayNameOfNode(<Foo />)).to.equal('');
+      });
+
       describeIf(!REACT013, 'stateless function components', () => {
         it('should return the name', () => {
           const Foo = () => <div />;