fix(test): Fix test and linting

Author: Khaledgarbaya

lib/get/get-outdated-destination-content.js

@@ -10,44 +10,36 @@ const BATCH_CHAR_LIMIT = 1990
  * Only the supplied entry/asset IDs will be retrieved. All contentTypes
  * and Locales will be retrieved.
  */
-export default function getOutdatedDestinationContent ({
-  managementClient,
-  spaceId,
-  entryIds = [],
-  assetIds = [],
-  webhookIds = [],
-  skipContentModel,
-  skipContent
-}) {
+export default function getOutdatedDestinationContent ({managementClient, spaceId, entryIds = [], assetIds = [], webhookIds = [], skipContentModel, skipContent}) {
   log.info('Checking if destination space already has any content and retrieving it')
 
   return managementClient.getSpace(spaceId)
     .then((space) => {
-    return Promise.props({
-      contentTypes: skipContentModel ? Promise.resolve([]) : space.getContentTypes().then(extractItems),
-      entries: skipContent ? Promise.resolve([]) : batchedIdQuery(space, 'getEntries', entryIds),
-      assets: skipContent ? Promise.resolve([]) : batchedIdQuery(space, 'getAssets', assetIds),
-      locales: skipContentModel ? Promise.resolve([]) : space.getLocales().then(extractItems),
-      webhooks: []
-    })  }, (err) => {
-    log.error(`
+      return Promise.props({
+        contentTypes: skipContentModel ? Promise.resolve([]) : space.getContentTypes().then(extractItems),
+        entries: skipContent ? Promise.resolve([]) : batchedIdQuery(space, 'getEntries', entryIds),
+        assets: skipContent ? Promise.resolve([]) : batchedIdQuery(space, 'getAssets', assetIds),
+        locales: skipContentModel ? Promise.resolve([]) : space.getLocales().then(extractItems),
+        webhooks: []
+      }) }, (err) => {
+        log.error(`
 The destination space was not found. This can happen for multiple reasons:
 - If you haven't yet, you should create your space manually.
 - If your destination space is in another organization, and your user from the source space does not have access to it, you'll need to specify separate sourceManagementToken and destinationManagementToken
 
 Full error details below.
 `)
-    throw err
-  })
+        throw err
+      })
 }
 
 function batchedIdQuery (space, method, ids) {
   return Promise.reduce(getIdBatches(ids), (fullResponse, batch) => {
     return space[method]({'sys.id[in]': batch})
-    .then((response) => {
-      fullResponse = fullResponse.concat(response.items)
-      return fullResponse
-    })
+      .then((response) => {
+        fullResponse = fullResponse.concat(response.items)
+        return fullResponse
+      })
   }, [])
 }
 

test/get/get-source-space-via-sync-test.js

@@ -13,6 +13,7 @@ function setup () {
     sync: sinon.stub().returns(Promise.resolve({
       entries: [{sys: {type: 'Entry'}}],
       assets: [{sys: {type: 'Asset'}}],
+      webhooks: [],
       deletedEntries: [{sys: {type: 'DeletedEntry'}}],
       deletedAssets: [{sys: {type: 'DeletedAsset'}}],
       nextSyncToken: 'token'
@@ -35,6 +36,7 @@ function setup () {
   const preparedResponse = {
     entries: [{sys: {type: 'Entry'}}],
     assets: [{sys: {type: 'Asset'}}],
+    webhooks: [],
     deletedEntries: [{sys: {type: 'DeletedEntry'}}],
     deletedAssets: [{sys: {type: 'DeletedAsset'}}],
     contentTypes: [{sys: {type: 'ContentType'}}],