@defra-fish/dynamics-lib 1.77.0-rc.9 → 1.77.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@defra-fish/dynamics-lib",
-  "version": "1.77.0-rc.9",
+  "version": "1.77.1",
   "description": "Framework to support integration with dynamics",
   "type": "module",
   "engines": {
@@ -43,5 +43,5 @@
     "simple-oauth2": "4.3.0",
     "uuid": "8.3.2"
   },
-  "gitHead": "3cfd7fe39ca9db0267159d64702dab54ad183be1"
+  "gitHead": "3c12f0cae7daf9d931a85f1ab89e6ab7a218cb0a"
 }

package/src/client/__tests__/entity-manager.spec.js

@@ -81,7 +81,10 @@ describe('entity manager', () => {
 
     it('throws an error object on failure', async () => {
       const consoleErrorSpy = jest.spyOn(console, 'error').mockImplementation(jest.fn())
-      MockDynamicsWebApi.__throwWithErrorsOnBatchExecute()
+      const error = new Error('Test error')
+      const message = Symbol('We have a problem here')
+      error.error = { message }
+      MockDynamicsWebApi.__throwWithErrorsOnBatchExecute(error)
       const newEntity = Object.assign(new TestEntity(), { strVal: 'test', intVal: 0 })
       const existingEntity = TestEntity.fromResponse(
         {
@@ -93,12 +96,13 @@ describe('entity manager', () => {
         },
         {}
       )
-      await expect(persist([newEntity, existingEntity])).rejects.toThrow('Test error')
+      await expect(persist([newEntity, existingEntity])).rejects.toThrow(error)
       // Expect the console error to contain details of the batch data (one create, one update plus the exception object)
       expect(consoleErrorSpy).toHaveBeenCalledWith(
-        expect.stringMatching('Error persisting batch. Data: %j, Exception: %o'),
+        expect.stringMatching('Error persisting batch. Data: %j, Exception: %o, Message: %s'),
         expect.arrayContaining([{ create: expect.any(Object) }, { update: expect.any(Object) }]),
-        expect.any(Error)
+        error,
+        message
       )
     })
 

package/src/client/entity-manager.js

@@ -25,7 +25,7 @@ export async function persist (entities, createdBy) {
   } catch (e) {
     const error = e.length ? e[0] : e
     const requestDetails = entities.map(entity => ({ [entity.isNew() ? 'create' : 'update']: entity.toPersistRequest() }))
-    console.error('Error persisting batch. Data: %j, Exception: %o', requestDetails, error)
+    console.error('Error persisting batch. Data: %j, Exception: %o, Message: %s', requestDetails, error, error?.error?.message)
     throw error
   }
 }