feat(gatsby-source-graphql): Default Apollo Link fetch wrapper to sho…

…w better API errors (#28786)

* apollo fetch http error status

* update tests

Co-authored-by: Vladimir Razuvaev <vladimir.razuvaev@gmail.com>
Co-authored-by: gatsbybot <mathews.kyle+gatsbybot@gmail.com>

packages/gatsby-source-graphql/src/__tests__/gatsby-node.js

@@ -20,7 +20,7 @@ jest.mock(`gatsby/graphql`, () => {
   }
 })
 const { sourceNodes } = require(`../gatsby-node`)
-const nodeFetch = require(`node-fetch`)
+const { fetchWrapper } = require(`../fetch`)
 
 const getInternalGatsbyAPI = () => {
   const actions = {
@@ -98,7 +98,7 @@ describe(`createHttpLink`, () => {
     })
 
     expect(createHttpLink).toHaveBeenCalledWith(
-      expect.objectContaining({ fetch: nodeFetch })
+      expect.objectContaining({ fetch: fetchWrapper })
     )
   })
 })

packages/gatsby-source-graphql/src/fetch.js

@@ -0,0 +1,16 @@
+const nodeFetch = require(`node-fetch`)
+
+// this is passed to the Apollo Link
+// https://www.apollographql.com/docs/link/links/http/#fetch-polyfill
+
+exports.fetchWrapper = async (uri, options) => {
+  const response = await nodeFetch(uri, options)
+
+  if (response.status >= 400) {
+    throw new Error(
+      `Source GraphQL API: HTTP error ${response.status} ${response.statusText}`
+    )
+  }
+
+  return response
+}

packages/gatsby-source-graphql/src/gatsby-node.js

@@ -7,8 +7,8 @@ const {
 } = require(`@graphql-tools/wrap`)
 const { linkToExecutor } = require(`@graphql-tools/links`)
 const { createHttpLink } = require(`apollo-link-http`)
-const nodeFetch = require(`node-fetch`)
 const invariant = require(`invariant`)
+const { fetchWrapper } = require(`./fetch`)
 const { createDataloaderLink } = require(`./batching/dataloader-link`)
 
 const {
@@ -26,7 +26,7 @@ exports.sourceNodes = async (
     typeName,
     fieldName,
     headers = {},
-    fetch = nodeFetch,
+    fetch = fetchWrapper,
     fetchOptions = {},
     createLink,
     createSchema,