inspect: correctly handle custom objects without class name

graphql · Aug 2, 2019 · ae9e34b · ae9e34b
1 parent ce8b4cd
commit ae9e34b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/jsutils/__tests__/inspect-test.js b/src/jsutils/__tests__/inspect-test.js
@@ -177,5 +177,10 @@ describe('inspect', () => {
 
     (Foo.prototype: any)[Symbol.toStringTag] = 'Bar';
     expect(inspect([[new Foo()]])).to.equal('[[[Bar]]]');
+
+    const objectWithoutClassName = new (function() {
+      this.foo = 1;
+    })();
+    expect(inspect([[objectWithoutClassName]])).to.equal('[[[Object]]]');
   });
 });
diff --git a/src/jsutils/inspect.js b/src/jsutils/inspect.js
@@ -117,7 +117,7 @@ function getObjectTag(object) {
 
   if (tag === 'Object' && typeof object.constructor === 'function') {
     const name = object.constructor.name;
-    if (typeof name === 'string') {
+    if (typeof name === 'string' && name !== '') {
       return name;
     }
   }