ra-data-graphql: error thrown by GraphQL client not caught so data provider does not return rejected promise

[bayareacoder]

What you were expecting:

• when GQL client throws , the function const raDataProvider = (raFetchType, resource, params) => { ...} in index.js should return a rejected promise, per data provider requirements in react-admin.

What happened instead:
While a

catch (e) {
      return Promise.reject(e);
    }

was added, this code is never reached since the promise chain already returned earlier, so the error is not caught, eg:

          return client
            .query(apolloQuery)
            .then((response) => parseResponse(response))

Solution is to add to each promise chain individually:

.catch((e) => Promise.reject(new HttpError(e.message, 200))

to ensure a rejected promise is returned with an error object with the required syntax.

This was seen together with #5907