]> Piment Noir Git Repositories - e-mobility-charging-stations-simulator.git/commitdiff
fix(test): correct LRU eviction test to avoid duplicate identifier access
authorJérôme Benoit <jerome.benoit@sap.com>
Tue, 18 Nov 2025 18:33:33 +0000 (19:33 +0100)
committerJérôme Benoit <jerome.benoit@sap.com>
Tue, 18 Nov 2025 18:33:33 +0000 (19:33 +0100)
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.

tests/charging-station/ocpp/auth/cache/InMemoryAuthCache.test.ts

index 6b6a48f0827ddb6527e2667e824c75f84f9aae73..1e768aef04408e984c642040032456ee3b6fb32b 100644 (file)
@@ -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()
     })
   })