Merge remote-tracking branch 'refs/remotes/origin/224-caching' into 224-caching

Author: thehabes

cache/__tests__/cache.test.js

@@ -486,198 +486,3 @@ describe('Cache Statistics', () => {
         expect(cache.cache.size).toBe(1)
     })
 })
-
-describe('GOG Endpoint Cache Middleware', () => {
-    let mockReq
-    let mockRes
-    let mockNext
-
-    beforeEach(() => {
-        // Clear cache before each test
-        cache.clear()
-        
-        // Reset mock request
-        mockReq = {
-            method: 'POST',
-            body: {},
-            query: {},
-            params: {}
-        }
-        
-        // Reset mock response
-        mockRes = {
-            statusCode: 200,
-            headers: {},
-            set: jest.fn(function(key, value) {
-                if (typeof key === 'object') {
-                    Object.assign(this.headers, key)
-                } else {
-                    this.headers[key] = value
-                }
-                return this
-            }),
-            status: jest.fn(function(code) {
-                this.statusCode = code
-                return this
-            }),
-            json: jest.fn(function(data) {
-                this.jsonData = data
-                return this
-            })
-        }
-        
-        // Reset mock next
-        mockNext = jest.fn()
-    })
-
-    afterEach(() => {
-        cache.clear()
-    })
-
-    describe('cacheGogFragments middleware', () => {
-        it('should pass through when ManuscriptWitness is missing', () => {
-            mockReq.body = {}
-            
-            cacheGogFragments(mockReq, mockRes, mockNext)
-            
-            expect(mockNext).toHaveBeenCalled()
-            expect(mockRes.json).not.toHaveBeenCalled()
-        })
-
-        it('should pass through when ManuscriptWitness is invalid', () => {
-            mockReq.body = { ManuscriptWitness: 'not-a-url' }
-            
-            cacheGogFragments(mockReq, mockRes, mockNext)
-            
-            expect(mockNext).toHaveBeenCalled()
-            expect(mockRes.json).not.toHaveBeenCalled()
-        })
-
-        it('should return cache MISS on first request', () => {
-            mockReq.body = { ManuscriptWitness: 'https://example.org/manuscript/1' }
-            mockReq.query = { limit: '50', skip: '0' }
-            
-            cacheGogFragments(mockReq, mockRes, mockNext)
-            
-            expect(mockRes.headers['X-Cache']).toBe('MISS')
-            expect(mockNext).toHaveBeenCalled()
-        })
-
-        it('should return cache HIT on second identical request', () => {
-            mockReq.body = { ManuscriptWitness: 'https://example.org/manuscript/1' }
-            mockReq.query = { limit: '50', skip: '0' }
-            
-            // First request - populate cache
-            cacheGogFragments(mockReq, mockRes, mockNext)
-            mockRes.json([{ '@id': 'fragment1', '@type': 'WitnessFragment' }])
-            
-            // Reset mocks for second request
-            mockRes.headers = {}
-            mockRes.json = jest.fn()
-            mockNext = jest.fn()
-            
-            // Second request - should hit cache
-            cacheGogFragments(mockReq, mockRes, mockNext)
-            
-            expect(mockRes.headers['X-Cache']).toBe('HIT')
-            expect(mockRes.json).toHaveBeenCalledWith([{ '@id': 'fragment1', '@type': 'WitnessFragment' }])
-            expect(mockNext).not.toHaveBeenCalled()
-        })
-
-        it('should cache based on pagination parameters', () => {
-            const manuscriptURI = 'https://example.org/manuscript/1'
-            
-            // Request with limit=50, skip=0
-            mockReq.body = { ManuscriptWitness: manuscriptURI }
-            mockReq.query = { limit: '50', skip: '0' }
-            
-            cacheGogFragments(mockReq, mockRes, mockNext)
-            mockRes.json([{ '@id': 'fragment1' }])
-            
-            // Request with different pagination - should be MISS
-            mockRes.headers = {}
-            mockRes.json = jest.fn()
-            mockNext = jest.fn()
-            mockReq.query = { limit: '100', skip: '0' }
-            
-            cacheGogFragments(mockReq, mockRes, mockNext)
-            
-            expect(mockRes.headers['X-Cache']).toBe('MISS')
-            expect(mockNext).toHaveBeenCalled()
-        })
-    })
-
-    describe('cacheGogGlosses middleware', () => {
-        it('should pass through when ManuscriptWitness is missing', () => {
-            mockReq.body = {}
-            
-            cacheGogGlosses(mockReq, mockRes, mockNext)
-            
-            expect(mockNext).toHaveBeenCalled()
-            expect(mockRes.json).not.toHaveBeenCalled()
-        })
-
-        it('should pass through when ManuscriptWitness is invalid', () => {
-            mockReq.body = { ManuscriptWitness: 'not-a-url' }
-            
-            cacheGogGlosses(mockReq, mockRes, mockNext)
-            
-            expect(mockNext).toHaveBeenCalled()
-            expect(mockRes.json).not.toHaveBeenCalled()
-        })
-
-        it('should return cache MISS on first request', () => {
-            mockReq.body = { ManuscriptWitness: 'https://example.org/manuscript/1' }
-            mockReq.query = { limit: '50', skip: '0' }
-            
-            cacheGogGlosses(mockReq, mockRes, mockNext)
-            
-            expect(mockRes.headers['X-Cache']).toBe('MISS')
-            expect(mockNext).toHaveBeenCalled()
-        })
-
-        it('should return cache HIT on second identical request', () => {
-            mockReq.body = { ManuscriptWitness: 'https://example.org/manuscript/1' }
-            mockReq.query = { limit: '50', skip: '0' }
-            
-            // First request - populate cache
-            cacheGogGlosses(mockReq, mockRes, mockNext)
-            mockRes.json([{ '@id': 'gloss1', '@type': 'Gloss' }])
-            
-            // Reset mocks for second request
-            mockRes.headers = {}
-            mockRes.json = jest.fn()
-            mockNext = jest.fn()
-            
-            // Second request - should hit cache
-            cacheGogGlosses(mockReq, mockRes, mockNext)
-            
-            expect(mockRes.headers['X-Cache']).toBe('HIT')
-            expect(mockRes.json).toHaveBeenCalledWith([{ '@id': 'gloss1', '@type': 'Gloss' }])
-            expect(mockNext).not.toHaveBeenCalled()
-        })
-
-        it('should cache based on pagination parameters', () => {
-            const manuscriptURI = 'https://example.org/manuscript/1'
-            
-            // Request with limit=50, skip=0
-            mockReq.body = { ManuscriptWitness: manuscriptURI }
-            mockReq.query = { limit: '50', skip: '0' }
-            
-            cacheGogGlosses(mockReq, mockRes, mockNext)
-            mockRes.json([{ '@id': 'gloss1' }])
-            
-            // Request with different pagination - should be MISS
-            mockRes.headers = {}
-            mockRes.json = jest.fn()
-            mockNext = jest.fn()
-            mockReq.query = { limit: '100', skip: '0' }
-            
-            cacheGogGlosses(mockReq, mockRes, mockNext)
-            
-            expect(mockRes.headers['X-Cache']).toBe('MISS')
-            expect(mockNext).toHaveBeenCalled()
-        })
-    })
-})
-