changes from testing

Author: thehabes

cache/__tests__/cache.test.js

@@ -365,12 +365,11 @@ describe('Cache Middleware Tests', () => {
             cacheStats(mockReq, mockRes)
 
             expect(mockRes.json).toHaveBeenCalled()
-            const stats = mockRes.json.mock.calls[0][0]
-            expect(stats).toHaveProperty('stats')
-            expect(stats.stats).toHaveProperty('hits')
-            expect(stats.stats).toHaveProperty('misses')
-            expect(stats.stats).toHaveProperty('hitRate')
-            expect(stats.stats).toHaveProperty('length')
+            const response = mockRes.json.mock.calls[0][0]
+            expect(response).toHaveProperty('hits')
+            expect(response).toHaveProperty('misses')
+            expect(response).toHaveProperty('hitRate')
+            expect(response).toHaveProperty('length')
         })
 
         it('should include details when requested', () => {
@@ -380,6 +379,8 @@ describe('Cache Middleware Tests', () => {
 
             const response = mockRes.json.mock.calls[0][0]
             expect(response).toHaveProperty('details')
+            expect(response).toHaveProperty('hits')
+            expect(response).toHaveProperty('misses')
         })
     })
 

cache/middleware.js

@@ -470,7 +470,7 @@ const invalidateCache = (req, res, next) => {
  */
 const cacheStats = (req, res) => {
     const stats = cache.getStats()
-    const response = { stats }
+    const response = { ...stats }
     if (req.query.details === 'true') response.details = cache.getDetailsByEntry()
     res.status(200).json(response)
 }