Avoid optional chaining in instanceOf to achieve 100% coverage.

graphql · Jan 26, 2021 · 823d311 · 823d311
1 parent db4b2b8
commit 823d311
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/jsutils/instanceOf.js b/src/jsutils/instanceOf.js
@@ -23,7 +23,8 @@ export default process.env.NODE_ENV === 'production'
         return true;
       }
       if (value) {
-        const classTag = constructor?.prototype?.[SYMBOL_TO_STRING_TAG];
+        const proto = constructor && constructor.prototype;
+        const classTag = proto && proto[SYMBOL_TO_STRING_TAG];
         const className = classTag || constructor.name;
         // When the constructor class defines a Symbol.toStringTag
         // property, as most classes exported by graphql-js do, use it
@@ -38,7 +39,7 @@ export default process.env.NODE_ENV === 'production'
         // value is legitimately _not_ instanceof constructor.
         const valueName = classTag
           ? value[SYMBOL_TO_STRING_TAG]
-          : value.constructor?.name;
+          : value.constructor && value.constructor.name;
         if (typeof className === 'string' && valueName === className) {
           throw new Error(
             `Cannot use ${className} "${value}" from another module or realm.