redis/go-redis
1
0
Fork 0
mirror of https://github.com/redis/go-redis.git synced 2025-12-02 06:22:31 +03:00
Code Activity

fix flaky test

Browse Source
This commit is contained in:
Nedyalko Dyakov
2025-11-05 21:35:20 +02:00
parent 95af71c50b
commit 18b46a1ca7
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
internal/pool/conn_used_at_test.go
View File

@@ -22,7 +22,7 @@ func TestConn_UsedAtUpdatedOnRead(t *testing.T) {
// Get initial usedAt time // Get initial usedAt time
initialUsedAt := cn.UsedAt() initialUsedAt := cn.UsedAt()
// Wait at least 50ms to ensure time difference (usedAt has ~50ms precision from cached time) // Wait 100ms to ensure time difference (usedAt has ~50ms precision from cached time)
time.Sleep(100 * time.Millisecond) time.Sleep(100 * time.Millisecond)
// Simulate a read operation by calling WithReader // Simulate a read operation by calling WithReader
@@ -45,10 +45,10 @@ func TestConn_UsedAtUpdatedOnRead(t *testing.T) {
initialUsedAt, updatedUsedAt) initialUsedAt, updatedUsedAt)
} }
// Verify the difference is reasonable (should be around 50ms, accounting for ~50ms cache precision) // Verify the difference is reasonable (should be around 100ms, accounting for ~50ms cache precision and ~5ms sleep precision)
diff := updatedUsedAt.Sub(initialUsedAt) diff := updatedUsedAt.Sub(initialUsedAt)
if diff < 50*time.Millisecond || diff > 200*time.Millisecond { if diff < 45*time.Millisecond || diff > 155*time.Millisecond {
t.Errorf("Expected usedAt difference to be around 50ms (±50ms for cache), got %v", diff) t.Errorf("Expected usedAt difference to be around 100ms (±50ms for cache, ±5ms for sleep), got %v", diff)
} }
} }