Implemented partial error

Author: maneesht

packages/data-connect/src/network/fetch.ts

@@ -15,7 +15,12 @@
  * limitations under the License.
  */
 
-import { Code, DataConnectError } from '../core/error';
+import {
+  Code,
+  DataConnectError,
+  DataConnectOperationError,
+  DataConnectOperationResponse
+} from '../core/error';
 import { SDK_VERSION } from '../core/version';
 import { logDebug, logError } from '../logger';
 
@@ -106,10 +111,17 @@ export function dcFetch<T, U>(
       return jsonResponse;
     })
     .then(res => {
+      console.log(res.errors);
       if (res.errors && res.errors.length) {
         const stringified = JSON.stringify(res.errors);
-        logError('DataConnect error while performing request: ' + stringified);
-        throw new DataConnectError(Code.OTHER, stringified);
+        const response: DataConnectOperationResponse = {
+          errors: res.errors,
+          data: res.data
+        };
+        throw new DataConnectOperationError(
+          'DataConnect error while performing request: ' + stringified,
+          response
+        );
       }
       return res;
     });

packages/data-connect/test/unit/fetch.test.ts

@@ -34,7 +34,7 @@ function mockFetch(json: object, reject: boolean): sinon.SinonStub {
   initializeFetch(fakeFetchImpl);
   return fakeFetchImpl;
 }
-describe('fetch', () => {
+describe.only('fetch', () => {
   it('should throw an error with just the message when the server responds with an error with a message property in the body', async () => {
     const message = 'Failed to connect to Postgres instance';
     mockFetch(
@@ -85,6 +85,40 @@ describe('fetch', () => {
       )
     ).to.eventually.be.rejectedWith(JSON.stringify(json));
   });
+  it.only('should throw a stringified message when the server responds with an error without a message property in the body', async () => {
+    const json = {
+      'data': { 'abc': 'def' },
+      'errors': [
+        {
+          'message':
+            'SQL query error: pq: duplicate key value violates unique constraint movie_pkey',
+          'locations': [],
+          'path': ['the_matrix'],
+          'extensions': null
+        }
+      ]
+    };
+    mockFetch(json, false);
+    await expect(
+      dcFetch(
+        'http://localhost',
+        {
+          name: 'n',
+          operationName: 'n',
+          variables: {}
+        },
+        {} as AbortController,
+        null,
+        null,
+        null,
+        false,
+        CallerSdkTypeEnum.Base
+      )
+    ).to.eventually.be.rejected.then(error => {
+      expect(error.response.data).to.eq(json.data);
+      expect(error.response.errors).to.eq(json.errors);
+    });
+  });
   it('should assign different values to custom headers based on the _callerSdkType argument (_isUsingGen is false)', async () => {
     const json = {
       code: 200,