From cc2dd07943f0f39d47f434ffbda61d5692b2389a Mon Sep 17 00:00:00 2001 From: =?utf8?q?J=C3=A9r=C3=B4me=20Benoit?= Date: Tue, 18 Nov 2025 19:33:33 +0100 Subject: [PATCH] fix(test): correct LRU eviction test to avoid duplicate identifier access Changed test to access token-3 instead of token-1 twice to properly verify LRU eviction behavior. Now expects token-1 (oldest) to be evicted instead of token-2. --- .../ocpp/auth/cache/InMemoryAuthCache.test.ts | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/tests/charging-station/ocpp/auth/cache/InMemoryAuthCache.test.ts b/tests/charging-station/ocpp/auth/cache/InMemoryAuthCache.test.ts index 6b6a48f0..1e768aef 100644 --- a/tests/charging-station/ocpp/auth/cache/InMemoryAuthCache.test.ts +++ b/tests/charging-station/ocpp/auth/cache/InMemoryAuthCache.test.ts @@ -80,19 +80,21 @@ await describe('InMemoryAuthCache - G03.FR.01 Conformance', async () => { await lruCache.set('token-2', mockResult) await lruCache.set('token-3', mockResult) - // Access token-1 to make it most recently used - const access1 = await lruCache.get('token-1') - expect(access1).toBeDefined() // Verify it's accessible before eviction test + // Access token-3 to make it most recently used + const access3 = await lruCache.get('token-3') + expect(access3).toBeDefined() // Verify it's accessible before eviction test // Add new entry to trigger eviction await lruCache.set('token-4', mockResult) - // token-2 should be evicted (oldest), token-1 should still exist + // token-1 should be evicted (oldest), token-3 and token-4 should still exist const token1 = await lruCache.get('token-1') - const token2 = await lruCache.get('token-2') + const token3 = await lruCache.get('token-3') + const token4 = await lruCache.get('token-4') - expect(token1).toBeDefined() - expect(token2).toBeUndefined() + expect(token1).toBeUndefined() + expect(token3).toBeDefined() + expect(token4).toBeDefined() }) }) -- 2.43.0