Use the TypeError class instead of Error for relevant errors.

Author: jaydenseric

changelog.md

@@ -5,6 +5,7 @@
 ### Patch
 
 - Replaced Node.js deprecated `notEqual` assertions with `notStrictEqual` in tests.
+- Use the `TypeError` class instead of `Error` for relevant errors.
 
 ## 11.1.0
 

src/server/ssr.js

@@ -80,15 +80,15 @@ module.exports = async function ssr(
   render = ReactDOMServer.renderToStaticMarkup
 ) {
   if (!(graphql instanceof GraphQL))
-    throw new Error('ssr() argument 1 must be a GraphQL instance.');
+    throw new TypeError('ssr() argument 1 must be a GraphQL instance.');
 
   // Check argument 2 exists, allowing an undefined value as that is a valid
   // React node.
   if (arguments.length < 2)
-    throw new Error('ssr() argument 2 must be a React node.');
+    throw new TypeError('ssr() argument 2 must be a React node.');
 
   if (typeof render !== 'function')
-    throw new Error('ssr() argument 3 must be a function.');
+    throw new TypeError('ssr() argument 3 must be a function.');
 
   // Signal that queries should load at render.
   graphql.ssr = true;

src/test/server/ssr.test.js

@@ -13,7 +13,7 @@ const listen = require('../listen');
 
 module.exports = (tests) => {
   tests.add('`ssr` argument 1 validation', async () => {
-    const error = new Error('ssr() argument 1 must be a GraphQL instance.');
+    const error = new TypeError('ssr() argument 1 must be a GraphQL instance.');
 
     await rejects(ssr(), error);
     await rejects(ssr(true), error);
@@ -24,7 +24,7 @@ module.exports = (tests) => {
 
     await rejects(
       ssr(graphql),
-      new Error('ssr() argument 2 must be a React node.')
+      new TypeError('ssr() argument 2 must be a React node.')
     );
 
     strictEqual(await ssr(graphql, undefined), '');
@@ -38,7 +38,7 @@ module.exports = (tests) => {
 
     await rejects(
       ssr(graphql, node, false),
-      new Error('ssr() argument 3 must be a function.')
+      new TypeError('ssr() argument 3 must be a function.')
     );
   });
 

src/test/universal/GraphQL.test.js

@@ -271,7 +271,7 @@ module.exports = (tests) => {
           reloadOnLoad: true,
           resetOnLoad: true,
         });
-      }, new Error('operate() options “reloadOnLoad” and “resetOnLoad” can’t both be true.'));
+      }, new TypeError('operate() options “reloadOnLoad” and “resetOnLoad” can’t both be true.'));
     }
   );
 

src/test/universal/useGraphQL.test.js

@@ -1613,7 +1613,7 @@ module.exports = (tests) => {
             />
           </GraphQLProvider>
         );
-      }, new Error('useGraphQL() options “reloadOnLoad” and “resetOnLoad” can’t both be true.'));
+      }, new TypeError('useGraphQL() options “reloadOnLoad” and “resetOnLoad” can’t both be true.'));
     }
   );
 
@@ -1622,7 +1622,7 @@ module.exports = (tests) => {
       ReactDOMServer.renderToString(
         <RenderUseGraphQL operation={{ query: '' }} />
       );
-    }, new Error('GraphQL context missing.'));
+    }, new TypeError('GraphQL context missing.'));
   });
 
   tests.add('`useGraphQL` with GraphQL context not a GraphQL instance', () => {
@@ -1632,7 +1632,7 @@ module.exports = (tests) => {
           <RenderUseGraphQL operation={{ query: '' }} />
         </GraphQLContext.Provider>
       );
-    }, new Error('GraphQL context must be a GraphQL instance.'));
+    }, new TypeError('GraphQL context must be a GraphQL instance.'));
   });
 
   tests.add(

src/universal/GraphQL.js

@@ -164,7 +164,7 @@ module.exports = class GraphQL {
         ? fetch
         : () =>
             Promise.reject(
-              new Error('Global fetch API or polyfill unavailable.')
+              new TypeError('Global fetch API or polyfill unavailable.')
             );
     const cacheValue = {};
     const cacheValuePromise = fetcher(url, options)
@@ -245,7 +245,7 @@ module.exports = class GraphQL {
     resetOnLoad,
   }) => {
     if (reloadOnLoad && resetOnLoad)
-      throw new Error(
+      throw new TypeError(
         'operate() options “reloadOnLoad” and “resetOnLoad” can’t both be true.'
       );
 

src/universal/useGraphQL.js

@@ -59,17 +59,17 @@ module.exports = function useGraphQL({
   resetOnLoad,
 }) {
   if (reloadOnLoad && resetOnLoad)
-    throw new Error(
+    throw new TypeError(
       'useGraphQL() options “reloadOnLoad” and “resetOnLoad” can’t both be true.'
     );
 
   const graphql = React.useContext(GraphQLContext);
 
   if (typeof graphql === 'undefined')
-    throw new Error('GraphQL context missing.');
+    throw new TypeError('GraphQL context missing.');
 
   if (!(graphql instanceof GraphQL))
-    throw new Error('GraphQL context must be a GraphQL instance.');
+    throw new TypeError('GraphQL context must be a GraphQL instance.');
 
   const fetchOptionsHash = React.useMemo(() => {
     const fetchOptions = graphqlFetchOptions(operation);