@defra-fish/dynamics-lib 1.69.0-rc.6 → 1.70.0-dynamics-upgrade

package/README.md

@@ -16,7 +16,7 @@ shared by the different packages which comprise the Rod Licensing digital servic
 | OAUTH_AUTHORITY_HOST_URL | OAuth 2.0 authority host                                         |   yes    |          |                               |                                                                                                               |
 | OAUTH_TENANT             | OAuth 2.0 tenant                                                 |   yes    |          |                               |                                                                                                               |
 | OAUTH_SCOPE              | OAuth 2.0 scope to request (client credentials resource)         |   yes    |          |                               |                                                                                                               |
-| DYNAMICS_API_PATH        | Full URL to the dynamics API                                     |   yes    |          |                               | The full URL to the dynamics web api. e.g. https://dynamics-server/api/data/v9.1/                             |
+| DYNAMICS_API_PATH        | The root domain of the dynamics server                           |   yes    |          |                               | The root domain of the dynamics server. e.g. https://dynamics-server                                          |
 | DYNAMICS_API_VERSION     | The version of the Dynamics API                                  |   yes    |          |                               | The version of the dynamics web api. e.g. 9.1                                                                 |
 | DYNAMICS_API_TIMEOUT     | The Dynamics API request timeout                                 |    no    | 90000    |                               | The time in milliseconds after which requests will timeout if Dynamics does not return a response, e.g. 90000 |
 | DYNAMICS_CACHE_TTL       | Default TTL for cached operations                                |    no    | 12 hours |                               | The default TTL for cached operations. Specified in seconds.                                                  |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@defra-fish/dynamics-lib",
-  "version": "1.69.0-rc.6",
+  "version": "1.70.0-dynamics-upgrade",
   "description": "Framework to support integration with dynamics",
   "type": "module",
   "engines": {
@@ -36,12 +36,11 @@
     "cache-manager": "3.6.0",
     "cache-manager-ioredis": "2.1.0",
     "debug": "4.3.3",
-    "dynamics-web-api": "1.7.3",
+    "dynamics-web-api": "^2.4.0",
     "joi": "17.13.3",
     "moment": "2.29.1",
     "pluralize": "8.0.0",
     "simple-oauth2": "4.3.0",
     "uuid": "8.3.2"
-  },
-  "gitHead": "8e77ed08621e055f9c691f190f109fab84fd472e"
+  }
 }

package/src/__mocks__/dynamics-web-api-mock-helper.js

@@ -3,7 +3,7 @@ const { readFileSync } = jest.requireActual('fs')
 const Path = jest.requireActual('path')
 const optionSetDataPath = Path.join(Project.root, 'src', '__mocks__', 'option-set-data.json')
 
-export const configureDynamicsWebApiMock = (DynamicsWebApi = jest.genMockFromModule('dynamics-web-api')) => {
+export const configureDynamicsWebApiMock = DynamicsWebApi => {
   let expectedResponse = {}
   let nextResponses = {}
   let callError = {}
@@ -53,12 +53,12 @@ export const configureDynamicsWebApiMock = (DynamicsWebApi = jest.genMockFromMod
     }
   }
 
-  DynamicsWebApi.prototype.retrieveRequest = jest.fn(async () => responseCapableMethod('retrieveRequest'))
-  DynamicsWebApi.prototype.createRequest = jest.fn(async () => responseCapableMethod('createRequest'))
-  DynamicsWebApi.prototype.updateRequest = jest.fn(async () => responseCapableMethod('updateRequest'))
-  DynamicsWebApi.prototype.retrieveMultipleRequest = jest.fn(async () => responseCapableMethod('retrieveMultipleRequest'))
+  DynamicsWebApi.prototype.retrieve = jest.fn(async () => responseCapableMethod('retrieve'))
+  DynamicsWebApi.prototype.create = jest.fn(async () => responseCapableMethod('create'))
+  DynamicsWebApi.prototype.update = jest.fn(async () => responseCapableMethod('update'))
+  DynamicsWebApi.prototype.retrieveMultiple = jest.fn(async () => responseCapableMethod('retrieveMultiple'))
   DynamicsWebApi.prototype.retrieveGlobalOptionSets = jest.fn(async () => responseCapableMethod('retrieveGlobalOptionSets'))
-  DynamicsWebApi.prototype.executeUnboundFunction = jest.fn(async functionName => {
+  DynamicsWebApi.prototype.callFunction = jest.fn(async functionName => {
     let returnValue = null
     if (functionName === 'RetrieveVersion') {
       returnValue = {

package/src/__mocks__/dynamics-web-api.js

@@ -1,3 +1,5 @@
 import { configureDynamicsWebApiMock } from './dynamics-web-api-mock-helper.js'
-const DynamicsWebApi = jest.genMockFromModule('dynamics-web-api')
-export default configureDynamicsWebApiMock(DynamicsWebApi)
+const DynamicsWebApi = jest.fn()
+DynamicsWebApi.prototype = {}
+configureDynamicsWebApiMock(DynamicsWebApi)
+export { DynamicsWebApi }

package/src/client/__tests__/dynamics-client.spec.js

@@ -3,7 +3,7 @@ import SimpleOAuth2 from 'simple-oauth2'
 
 describe('dynamics-client', () => {
   it('is configured via environment variables', async () => {
-    process.env.DYNAMICS_API_PATH = 'https://test-server/api/data/v9.1/'
+    process.env.DYNAMICS_API_PATH = 'https://test-server'
     process.env.DYNAMICS_API_VERSION = '9.1'
     process.env.DYNAMICS_API_TIMEOUT = 60000
     process.env.OAUTH_AUTHORITY_HOST_URL = 'https://test-authority/'
@@ -14,26 +14,24 @@ describe('dynamics-client', () => {
     const dynamicsApiConfig = config()
 
     expect(dynamicsApiConfig).toMatchObject({
-      webApiUrl: process.env.DYNAMICS_API_PATH,
-      webApiVersion: process.env.DYNAMICS_API_VERSION,
+      serverUrl: process.env.DYNAMICS_API_PATH,
+      dataApi: { version: process.env.DYNAMICS_API_VERSION },
       timeout: `${process.env.DYNAMICS_API_TIMEOUT}`,
       onTokenRefresh: expect.any(Function)
     })
-    const testCallback = jest.fn()
-    await dynamicsApiConfig.onTokenRefresh(testCallback)
-    expect(testCallback).toHaveBeenCalledWith('MOCK TOKEN')
+    const token = await dynamicsApiConfig.onTokenRefresh()
+    expect(token).toBe('MOCK TOKEN')
   })
 
   it('caches tokens until they expire', async () => {
     const dynamicsApiConfig = config()
-    const testCallback = jest.fn()
-    await dynamicsApiConfig.onTokenRefresh(testCallback)
-    expect(testCallback).toHaveBeenLastCalledWith('MOCK TOKEN')
+    let token = await dynamicsApiConfig.onTokenRefresh()
+    expect(token).toBe('MOCK TOKEN')
     SimpleOAuth2.__setMockTokenReturnValue('NEW MOCK TOKEN')
-    await dynamicsApiConfig.onTokenRefresh(testCallback)
-    expect(testCallback).toHaveBeenLastCalledWith('MOCK TOKEN')
+    token = await dynamicsApiConfig.onTokenRefresh()
+    expect(token).toBe('MOCK TOKEN')
     SimpleOAuth2.__setMockTokenExpired(true)
-    await dynamicsApiConfig.onTokenRefresh(testCallback)
-    expect(testCallback).toHaveBeenLastCalledWith('NEW MOCK TOKEN')
+    token = await dynamicsApiConfig.onTokenRefresh()
+    expect(token).toBe('NEW MOCK TOKEN')
   })
 })

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

@@ -13,7 +13,7 @@ import {
 } from '../../index.js'
 import TestEntity from '../../__mocks__/TestEntity.js'
 import { v4 as uuidv4 } from 'uuid'
-import MockDynamicsWebApi from 'dynamics-web-api'
+import { DynamicsWebApi as MockDynamicsWebApi } from 'dynamics-web-api'
 import { PredefinedQuery } from '../../queries/predefined-query.js'
 import { BaseEntity, EntityDefinition } from '../../entities/base.entity.js'
 
@@ -33,7 +33,7 @@ describe('entity manager', () => {
       t.boolVal = true
 
       const result = await persist([t])
-      expect(MockDynamicsWebApi.prototype.createRequest).toHaveBeenCalled()
+      expect(MockDynamicsWebApi.prototype.create).toHaveBeenCalled()
       expect(result).toHaveLength(1)
       expect(result[0]).toEqual(resultUuid)
     })
@@ -54,7 +54,7 @@ describe('entity manager', () => {
       )
 
       const result = await persist([t])
-      expect(MockDynamicsWebApi.prototype.updateRequest).toHaveBeenCalled()
+      expect(MockDynamicsWebApi.prototype.update).toHaveBeenCalled()
       expect(result).toHaveLength(1)
       expect(result[0]).toEqual(resultUuid)
     })
@@ -69,7 +69,7 @@ describe('entity manager', () => {
       t.boolVal = true
 
       const result = await persist([t], 'foo')
-      expect(MockDynamicsWebApi.prototype.createRequest).toHaveBeenCalled()
+      expect(MockDynamicsWebApi.prototype.create).toHaveBeenCalled()
       expect(MockDynamicsWebApi.prototype.executeBatch).toHaveBeenCalledWith(
         expect.objectContaining({
           impersonateAAD: 'foo'
@@ -94,10 +94,10 @@ describe('entity manager', () => {
         {}
       )
       await expect(persist([newEntity, existingEntity])).rejects.toThrow('Test error')
-      // Expect the console error to contain details of the batch data (one createRequest, one updateRequest plus the exception object)
+      // 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.arrayContaining([{ createRequest: expect.any(Object) }, { updateRequest: expect.any(Object) }]),
+        expect.arrayContaining([{ create: expect.any(Object) }, { update: expect.any(Object) }]),
         expect.any(Error)
       )
     })
@@ -332,14 +332,14 @@ describe('entity manager', () => {
 
   describe('findById', () => {
     it('finds by a primary key guid', async () => {
-      MockDynamicsWebApi.__setResponse('retrieveRequest', {
+      MockDynamicsWebApi.__setResponse('retrieve', {
         '@odata.etag': 'W/"202465000"',
         idval: '9f1b34a0-0c66-e611-80dc-c4346bad0190',
         strval: 'example'
       })
 
       const result = await findById(TestEntity, '9f1b34a0-0c66-e611-80dc-c4346bad0190')
-      expect(MockDynamicsWebApi.prototype.retrieveRequest).toBeCalledWith({
+      expect(MockDynamicsWebApi.prototype.retrieve).toBeCalledWith({
         key: '9f1b34a0-0c66-e611-80dc-c4346bad0190',
         collection: TestEntity.definition.dynamicsCollection,
         select: TestEntity.definition.select
@@ -350,13 +350,13 @@ describe('entity manager', () => {
     it('returns null if not found', async () => {
       const notFoundError = new Error('Not found')
       notFoundError.status = 404
-      MockDynamicsWebApi.__throwWithErrorOn('retrieveRequest', notFoundError)
+      MockDynamicsWebApi.__throwWithErrorOn('retrieve', notFoundError)
       const result = await findById(TestEntity, '9f1b34a0-0c66-e611-80dc-c4346bad0190')
       expect(result).toEqual(null)
     })
 
     it('throws an exception on general errors', async () => {
-      MockDynamicsWebApi.__throwWithErrorOn('retrieveRequest')
+      MockDynamicsWebApi.__throwWithErrorOn('retrieve')
       const consoleErrorSpy = jest.spyOn(console, 'error').mockImplementation(jest.fn())
       await expect(findById(TestEntity, '9f1b34a0-0c66-e611-80dc-c4346bad0190')).rejects.toThrow('Test error')
       expect(consoleErrorSpy).toHaveBeenCalled()
@@ -365,13 +365,13 @@ describe('entity manager', () => {
 
   describe('findByAlternateKey', () => {
     it('finds by an alternate key', async () => {
-      MockDynamicsWebApi.__setResponse('retrieveRequest', {
+      MockDynamicsWebApi.__setResponse('retrieve', {
         '@odata.etag': 'W/"202465000"',
         idval: '9f1b34a0-0c66-e611-80dc-c4346bad0190',
         strval: 'example'
       })
       const result = await findByAlternateKey(TestEntity, 'example')
-      expect(MockDynamicsWebApi.prototype.retrieveRequest).toBeCalledWith({
+      expect(MockDynamicsWebApi.prototype.retrieve).toBeCalledWith({
         key: "strval='example'",
         collection: TestEntity.definition.dynamicsCollection,
         select: TestEntity.definition.select
@@ -381,13 +381,13 @@ describe('entity manager', () => {
     })
 
     it('escapes special characters in the key', async () => {
-      MockDynamicsWebApi.__setResponse('retrieveRequest', {
+      MockDynamicsWebApi.__setResponse('retrieve', {
         '@odata.etag': 'W/"202465000"',
         idval: '9f1b34a0-0c66-e611-80dc-c4346bad0190',
         strval: 'example'
       })
       const result = await findByAlternateKey(TestEntity, "test & example'")
-      expect(MockDynamicsWebApi.prototype.retrieveRequest).toBeCalledWith({
+      expect(MockDynamicsWebApi.prototype.retrieve).toBeCalledWith({
         key: "strval='test %26 example'''",
         collection: TestEntity.definition.dynamicsCollection,
         select: TestEntity.definition.select
@@ -399,7 +399,7 @@ describe('entity manager', () => {
 
   describe('findByExample', () => {
     it('builds a select statement appropriate to the type definition for each field', async () => {
-      MockDynamicsWebApi.__setResponse('retrieveMultipleRequest', { value: [{}] })
+      MockDynamicsWebApi.__setResponse('retrieveMultiple', { value: [{}] })
 
       const lookup = new TestEntity()
       lookup.strVal = 'StringData'
@@ -412,7 +412,7 @@ describe('entity manager', () => {
       const expectedLookupSelect =
         "strval eq 'StringData' and intval eq 123 and decval eq 123.45 and boolval eq true and dateval eq 1946-01-01 and datetimeval eq 1946-01-01T01:02:03Z and optionsetval eq 910400000"
       const result = await findByExample(lookup)
-      expect(MockDynamicsWebApi.prototype.retrieveMultipleRequest).toBeCalledWith({
+      expect(MockDynamicsWebApi.prototype.retrieveMultiple).toBeCalledWith({
         collection: TestEntity.definition.dynamicsCollection,
         select: TestEntity.definition.select,
         filter: expect.stringMatching(`${TestEntity.definition.defaultFilter} and ${expectedLookupSelect}`)
@@ -422,7 +422,7 @@ describe('entity manager', () => {
     })
 
     it('does not require the entity to define a default filter', async () => {
-      MockDynamicsWebApi.__setResponse('retrieveMultipleRequest', { value: [{}] })
+      MockDynamicsWebApi.__setResponse('retrieveMultiple', { value: [{}] })
 
       class SameEntity extends BaseEntity {
         static _definition = new EntityDefinition(() => ({
@@ -458,7 +458,7 @@ describe('entity manager', () => {
       lookup.testVal = 'StringData'
       const expectedLookupSelect = "testval eq 'StringData'"
       const result = await findByExample(lookup)
-      expect(MockDynamicsWebApi.prototype.retrieveMultipleRequest).toBeCalledWith({
+      expect(MockDynamicsWebApi.prototype.retrieveMultiple).toBeCalledWith({
         collection: SameEntity.definition.dynamicsCollection,
         select: SameEntity.definition.select,
         filter: expect.stringMatching(`${expectedLookupSelect}`)
@@ -468,13 +468,13 @@ describe('entity manager', () => {
     })
 
     it('only serializes fields into the select statement if they are set', async () => {
-      MockDynamicsWebApi.__setResponse('retrieveMultipleRequest', { value: [{}] })
+      MockDynamicsWebApi.__setResponse('retrieveMultiple', { value: [{}] })
 
       const lookup = new TestEntity()
       lookup.strVal = 'StringData'
       const expectedLookupSelect = "strval eq 'StringData'"
       const result = await findByExample(lookup)
-      expect(MockDynamicsWebApi.prototype.retrieveMultipleRequest).toBeCalledWith({
+      expect(MockDynamicsWebApi.prototype.retrieveMultiple).toBeCalledWith({
         collection: TestEntity.definition.dynamicsCollection,
         select: TestEntity.definition.select,
         filter: expect.stringMatching(`${TestEntity.definition.defaultFilter} and ${expectedLookupSelect}`)
@@ -484,7 +484,7 @@ describe('entity manager', () => {
     })
 
     it('throws an error object on failure', async () => {
-      MockDynamicsWebApi.__throwWithErrorOn('retrieveMultipleRequest')
+      MockDynamicsWebApi.__throwWithErrorOn('retrieveMultiple')
       const consoleErrorSpy = jest.spyOn(console, 'error').mockImplementation(jest.fn())
       await expect(findByExample(new TestEntity())).rejects.toThrow('Test error')
       expect(consoleErrorSpy).toHaveBeenCalled()
@@ -492,8 +492,8 @@ describe('entity manager', () => {
   })
 
   describe('executeQuery', () => {
-    it('calls retrieveMultipleRequest on the Dynamics API with the request data', async () => {
-      MockDynamicsWebApi.__setResponse('retrieveMultipleRequest', {
+    it('calls retrieveMultiple on the Dynamics API with the request data', async () => {
+      MockDynamicsWebApi.__setResponse('retrieveMultiple', {
         value: [
           {
             '@odata.etag': 'W/"202465000"',
@@ -504,7 +504,7 @@ describe('entity manager', () => {
       })
 
       const result = await executeQuery(new PredefinedQuery({ root: TestEntity, filter: "strval eq 'example'" }))
-      expect(MockDynamicsWebApi.prototype.retrieveMultipleRequest).toBeCalledWith({
+      expect(MockDynamicsWebApi.prototype.retrieveMultiple).toBeCalledWith({
         collection: TestEntity.definition.dynamicsCollection,
         filter: "strval eq 'example'",
         select: TestEntity.definition.select
@@ -516,7 +516,7 @@ describe('entity manager', () => {
     })
 
     it('throws an error object on failure', async () => {
-      MockDynamicsWebApi.__throwWithErrorOn('retrieveMultipleRequest')
+      MockDynamicsWebApi.__throwWithErrorOn('retrieveMultiple')
       const consoleErrorSpy = jest.spyOn(console, 'error').mockImplementation(jest.fn())
       await expect(executeQuery(new PredefinedQuery({ root: TestEntity, filter: "strval eq 'example'" }))).rejects.toThrow('Test error')
       expect(consoleErrorSpy).toHaveBeenCalled()
@@ -527,7 +527,7 @@ describe('entity manager', () => {
     beforeEach(async () => {
       MockDynamicsWebApi.__reset()
       MockDynamicsWebApi.__setNextResponses(
-        'retrieveMultipleRequest',
+        'retrieveMultiple',
         {
           value: [
             {
@@ -592,7 +592,7 @@ describe('entity manager', () => {
     })
 
     it('throws an error object on failure', async () => {
-      MockDynamicsWebApi.__throwWithErrorOn('retrieveMultipleRequest')
+      MockDynamicsWebApi.__throwWithErrorOn('retrieveMultiple')
       const consoleErrorSpy = jest.spyOn(console, 'error').mockImplementation(jest.fn())
       await expect(
         executePagedQuery(new PredefinedQuery({ root: TestEntity, filter: "strval eq 'example'" }), () => {}, 1)

package/src/client/dynamics-client.js

@@ -1,4 +1,4 @@
-import DynamicsWebApi from 'dynamics-web-api'
+import { DynamicsWebApi } from 'dynamics-web-api'
 import SimpleOAuth2 from 'simple-oauth2'
 const PREEMPTIVE_TOKEN_EXPIRY_SECONDS = 60
 
@@ -21,14 +21,16 @@ export function config () {
 
   let accessToken = null
   return {
-    webApiUrl: process.env.DYNAMICS_API_PATH,
-    webApiVersion: process.env.DYNAMICS_API_VERSION,
+    serverUrl: process.env.DYNAMICS_API_PATH,
+    dataApi: {
+      version: process.env.DYNAMICS_API_VERSION
+    },
     timeout: process.env.DYNAMICS_API_TIMEOUT || 90000,
-    onTokenRefresh: async dynamicsWebApiCallback => {
+    onTokenRefresh: async () => {
       if (!accessToken || accessToken.expired(PREEMPTIVE_TOKEN_EXPIRY_SECONDS)) {
         accessToken = await oauthClient.getToken({ scope: process.env.OAUTH_SCOPE })
       }
-      dynamicsWebApiCallback(accessToken.token.access_token)
+      return accessToken.token.access_token
     }
   }
 }

package/src/client/entity-manager.js

@@ -13,9 +13,9 @@ export async function persist (entities, createdBy) {
     dynamicsClient.startBatch()
     entities.forEach(entity => {
       if (entity.isNew()) {
-        dynamicsClient.createRequest(entity.toPersistRequest())
+        dynamicsClient.create(entity.toPersistRequest())
       } else {
-        dynamicsClient.updateRequest(entity.toPersistRequest())
+        dynamicsClient.update(entity.toPersistRequest())
       }
     })
     return await dynamicsClient.executeBatch({
@@ -23,7 +23,7 @@ export async function persist (entities, createdBy) {
     })
   } catch (e) {
     const error = e.length ? e[0] : e
-    const requestDetails = entities.map(entity => ({ [entity.isNew() ? 'createRequest' : 'updateRequest']: entity.toPersistRequest() }))
+    const requestDetails = entities.map(entity => ({ [entity.isNew() ? 'create' : 'update']: entity.toPersistRequest() }))
     console.error('Error persisting batch. Data: %j, Exception: %o', requestDetails, error)
     throw error
   }
@@ -32,7 +32,7 @@ export async function persist (entities, createdBy) {
 const retrieveMultipleFetchOperation = async entityClasses => {
   try {
     dynamicsClient.startBatch()
-    entityClasses.forEach(cls => dynamicsClient.retrieveMultipleRequest(cls.definition.toRetrieveRequest()))
+    entityClasses.forEach(cls => dynamicsClient.retrieveMultiple(cls.definition.toRetrieveRequest()))
     return await dynamicsClient.executeBatch()
   } catch (e) {
     const error = e.length ? e[0] : e
@@ -97,7 +97,10 @@ export function retrieveGlobalOptionSets () {
     'dynamics_optionsetmap',
     async () => {
       try {
-        const data = await dynamicsClient.retrieveGlobalOptionSets('Microsoft.Dynamics.CRM.OptionSetMetadata', ['Name', 'Options'])
+        const data = await dynamicsClient.retrieveGlobalOptionSets({
+          castType: 'Microsoft.Dynamics.CRM.OptionSetMetadata',
+          select: ['Name', 'Options']
+        })
         return data.value.reduce((acc, { Name: name, Options: options }) => {
           acc[name] = {
             name,
@@ -131,7 +134,7 @@ export function retrieveGlobalOptionSets () {
  */
 export async function findById (entityType, key) {
   try {
-    const record = await dynamicsClient.retrieveRequest({ key: key, ...entityType.definition.toRetrieveRequest(null) })
+    const record = await dynamicsClient.retrieve({ key: key, ...entityType.definition.toRetrieveRequest(null) })
     const optionSetData = await retrieveGlobalOptionSets().cached()
     return entityType.fromResponse(record, optionSetData)
   } catch (e) {
@@ -179,7 +182,7 @@ export async function findByExample (entity) {
         return acc
       }, [])
     ].join(' and ')
-    const results = await dynamicsClient.retrieveMultipleRequest(entity.constructor.definition.toRetrieveRequest(filter))
+    const results = await dynamicsClient.retrieveMultiple(entity.constructor.definition.toRetrieveRequest(filter))
     const optionSetData = await retrieveGlobalOptionSets().cached()
     return results.value.map(result => entity.constructor.fromResponse(result, optionSetData))
   } catch (e) {
@@ -198,7 +201,7 @@ export async function findByExample (entity) {
  */
 export async function executeQuery (query) {
   try {
-    const response = await dynamicsClient.retrieveMultipleRequest(query.toRetrieveRequest())
+    const response = await dynamicsClient.retrieveMultiple(query.toRetrieveRequest())
     const optionSetData = await retrieveGlobalOptionSets().cached()
     return query.fromResponse(response.value, optionSetData)
   } catch (e) {
@@ -225,7 +228,7 @@ export async function executePagedQuery (query, onPageReceived, maxPages) {
     const optionSetData = await retrieveGlobalOptionSets().cached()
     let nextLink = null
     do {
-      const response = await dynamicsClient.retrieveMultipleRequest(query.toRetrieveRequest(), nextLink)
+      const response = await dynamicsClient.retrieveMultiple(query.toRetrieveRequest(), nextLink)
       nextLink = response.oDataNextLink
       await onPageReceived(query.fromResponse(response.value, optionSetData))
       processed += response.value.length

package/src/entities/base.entity.js

@@ -308,7 +308,7 @@ export class BaseEntity {
       ...(!this.isNew() && { key: this.id }),
       collection: this.constructor.definition.dynamicsCollection,
       contentId: this.uniqueContentId,
-      entity: this.toRequestBody()
+      data: this.toRequestBody()
     }
   }
 

package/src/queries/__tests__/contact.queries.spec.js

@@ -4,11 +4,13 @@ import { Permission } from '../../entities/permission.entity.js'
 import { PredefinedQuery } from '../predefined-query.js'
 
 jest.mock('dynamics-web-api', () => {
-  return jest.fn().mockImplementation(() => {
-    return {
-      executeUnboundAction: jest.fn()
-    }
-  })
+  return {
+    DynamicsWebApi: jest.fn().mockImplementation(() => {
+      return {
+        callAction: jest.fn()
+      }
+    })
+  }
 })
 
 describe('Contact Queries', () => {