adapt-authoring-api 3.4.0 → 3.6.0

package/lib/AbstractApiModule.js

@@ -1,7 +1,6 @@
 import _ from 'lodash'
-import { AbstractModule, Hook, stringifyValues } from 'adapt-authoring-core'
+import { AbstractModule, DataCache, Hook, stringifyValues } from 'adapt-authoring-core'
 import { argsFromReq, generateApiMetadata, httpMethodToDBFunction } from './utils.js'
-import DataCache from './DataCache.js'
 import { loadRouteConfig } from 'adapt-authoring-server'
 /**
  * Abstract module for creating APIs
@@ -328,11 +327,9 @@ class AbstractApiModule extends AbstractModule {
    * @return {Promise} Resolves with the sanitised data
    */
   async sanitise (schemaName, data, options) {
+    const schema = await this.getSchema(schemaName, data)
     const isArray = Array.isArray(data)
-    const sanitised = await Promise.all((isArray ? data : [data]).map(async d => {
-      const schema = await this.getSchema(schemaName, d)
-      return schema.sanitise(d, options)
-    }))
+    const sanitised = (isArray ? data : [data]).map(d => schema.sanitise(d, options))
     return isArray ? sanitised : sanitised[0]
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "adapt-authoring-api",
-  "version": "3.4.0",
+  "version": "3.6.0",
   "description": "Abstract module for creating APIs",
   "homepage": "https://github.com/adapt-security/adapt-authoring-api",
   "license": "GPL-3.0",
@@ -11,7 +11,7 @@
     "test": "node --test 'tests/**/*.spec.js'"
   },
   "dependencies": {
-    "adapt-authoring-core": "^2.0.0",
+    "adapt-authoring-core": "^2.5.0",
     "adapt-authoring-server": "^2.1.0",
     "lodash": "^4.17.21"
   },

package/lib/DataCache.js

@@ -1,45 +0,0 @@
-import { App } from 'adapt-authoring-core'
-/**
- * Time-limited data cache
- * @memberof api
- */
-class DataCache {
-  /** @override */
-  constructor ({ enable, lifespan }) {
-    this.isEnabled = enable === true
-    this.lifespan = lifespan ?? App.instance.config.get('adapt-authoring-api.defaultCacheLifespan')
-    this.cache = {}
-  }
-
-  /**
-   * Retrieve cached data, or run fresh query if no cache exists or cache is invalid
-   * @param {Object} query
-   * @param Object} options
-   * @param {Object} mongoOptions
-   * @returns {*} The cached data
-   */
-  async get (query, options, mongoOptions) {
-    const key = JSON.stringify(query) + JSON.stringify(options) + JSON.stringify(mongoOptions)
-    this.prune()
-    if (this.isEnabled && this.cache[key]) {
-      return this.cache[key].data
-    }
-    const mongodb = await App.instance.waitForModule('mongodb')
-    const data = await mongodb.find(options.collectionName, query, mongoOptions)
-    this.cache[key] = { data, timestamp: Date.now() }
-    return data
-  }
-
-  /**
-   * Removes invalid cache data
-   */
-  prune () {
-    Object.keys(this.cache).forEach(k => {
-      if (Date.now() > (this.cache[k].timestamp + this.lifespan)) {
-        delete this.cache[k]
-      }
-    })
-  }
-}
-
-export default DataCache

package/tests/DataCache.spec.js

@@ -1,47 +0,0 @@
-import { describe, it, before } from 'node:test'
-import assert from 'node:assert/strict'
-
-describe('DataCache', () => {
-  let DataCache
-
-  before(async () => {
-    // DataCache constructor references App.instance.config, so we need to
-    // dynamically import after ensuring no app instance is required for prune tests.
-    // We import the module directly and test what we can without the full app.
-    DataCache = (await import('../lib/DataCache.js')).default
-  })
-
-  describe('#prune()', () => {
-    it('should remove expired entries from the cache', () => {
-      const instance = Object.create(DataCache.prototype)
-      instance.lifespan = 100
-      instance.cache = {
-        expired: { data: [1], timestamp: Date.now() - 200 },
-        valid: { data: [2], timestamp: Date.now() }
-      }
-      instance.prune()
-      assert.equal(instance.cache.expired, undefined)
-      assert.ok(instance.cache.valid)
-    })
-
-    it('should keep entries that have not expired', () => {
-      const instance = Object.create(DataCache.prototype)
-      instance.lifespan = 10000
-      instance.cache = {
-        a: { data: [1], timestamp: Date.now() },
-        b: { data: [2], timestamp: Date.now() }
-      }
-      instance.prune()
-      assert.ok(instance.cache.a)
-      assert.ok(instance.cache.b)
-    })
-
-    it('should handle an empty cache', () => {
-      const instance = Object.create(DataCache.prototype)
-      instance.lifespan = 100
-      instance.cache = {}
-      instance.prune()
-      assert.deepEqual(instance.cache, {})
-    })
-  })
-})