feat: changes to lru-cache since dependency update

no more maxAge
del -> delete
reset -> clear

lib/plugin/components/services/caches/index.js

@@ -46,15 +46,10 @@ class CacheService {
    * @param cacheName the cache identifier
    * @param key key with the specified value is to be associated
    * @param value the value to be associated with the specified key
-   * @param maxAge optional parameter, which overrides the cache configuration if provided
    * @returns {undefined}
    */
-  set (cacheName, key, value, maxAge) {
-    if (maxAge) {
-      this[cacheName].set(key, value, maxAge)
-    } else {
-      this[cacheName].set(key, value)
-    }
+  set (cacheName, key, value) {
+    this[cacheName].set(key, value)
   }
 
   /**
@@ -93,16 +88,16 @@ class CacheService {
    * @param key the key whose mapping is to be removed from the specified cache
    * @returns {undefined}
    */
-  del (cacheName, key) {
-    this[cacheName].del(key)
+  delete (cacheName, key) {
+    this[cacheName].delete(key)
   }
 
   /**
    * Clears the specified cache entirely, throwing away all values
    * @param cacheName the cache identifier
    */
-  reset (cacheName) {
-    this[cacheName].reset()
+  clear (cacheName) {
+    this[cacheName].clear()
   }
 }
 

lib/plugin/components/services/caches/schema.json

@@ -8,10 +8,6 @@
         "max": {
           "type": "number",
           "description": "The maximum size of the cache, checked by applying the length function to all values in the cache. Not setting this will fall-back to a service-level default."
-        },
-        "maxAge": {
-          "type": "number",
-          "description": "Maximum age in ms. Items are not pro-actively pruned out as they age, but if you try to get an item that is too old, it'll drop it and return undefined"
         }
       }
     }

lib/plugin/components/services/registry/index.js

@@ -142,7 +142,7 @@ class RegistryService {
       return callbackify(this.clear(key), callback)
     }
     await this.registryKeyModel.destroyById(key)
-    this.caches.del(cacheName, key)
+    this.caches.delete(cacheName, key)
     await this.refresh()
   }
 }

lib/plugin/components/state-resources/delete-cache-item/index.js

@@ -7,7 +7,7 @@ module.exports = class DeleteCacheItem {
     const { cacheName, key } = event
 
     try {
-      this.caches.del(cacheName, key)
+      this.caches.delete(cacheName, key)
       return context.sendTaskSuccess()
     } catch (err) {
       return context.sendTaskFailure({ error: 'DeleteCacheItemFail', cause: err })

lib/plugin/components/state-resources/reset-cache/index.js

@@ -5,7 +5,7 @@ module.exports = class ResetCache {
 
   run (event, context) {
     try {
-      this.caches.reset(event)
+      this.caches.clear(event)
       return context.sendTaskSuccess()
     } catch (err) {
       return context.sendTaskFailure({ error: 'ResetCacheFail', cause: err })

test/cache-spec.js

@@ -35,7 +35,7 @@ describe('Cache tests', function () {
   })
 
   it('delete the cache', () => {
-    cacheService.del('users', 'user1')
+    cacheService.delete('users', 'user1')
   })
 
   it('check the key has been removed from the cahce', () => {
@@ -53,7 +53,7 @@ describe('Cache tests', function () {
   })
 
   it('reset the cache', () => {
-    cacheService.reset('users')
+    cacheService.clear('users')
   })
 
   it('check the cache is empty', () => {