From 4eb2c7fb7da1f44824b31da4d9c46e8baddcf419 Mon Sep 17 00:00:00 2001 From: Denis Kirillov Date: Wed, 14 Feb 2024 14:32:17 +0300 Subject: [PATCH] [#290] Fix TestErrorTimeoutChecking test Signed-off-by: Denis Kirillov --- CHANGELOG.md | 1 + internal/frostfs/frostfs_test.go | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 78a0b231..647f6cce 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -9,6 +9,7 @@ This document outlines major changes between releases. - Fix status code in GET/HEAD delete marker (#226) - Fix `NextVersionIDMarker` in `list-object-versions` (#248) - Fix possibility of panic during SIGHUP (#288) +- Fix flaky `TestErrorTimeoutChecking` (`make test` sometimes failed) (#290) ### Added - Add new `frostfs.buffer_max_size_for_put` config param and sync TZ hash for PUT operations (#197) diff --git a/internal/frostfs/frostfs_test.go b/internal/frostfs/frostfs_test.go index 134ff3c2..a3b9bf22 100644 --- a/internal/frostfs/frostfs_test.go +++ b/internal/frostfs/frostfs_test.go @@ -38,7 +38,7 @@ func TestErrorTimeoutChecking(t *testing.T) { t.Run("deadline exceeded", func(t *testing.T) { ctx, cancel := context.WithTimeout(context.Background(), 10*time.Millisecond) defer cancel() - time.Sleep(50 * time.Millisecond) + <-ctx.Done() require.True(t, errorsFrost.IsTimeoutError(ctx.Err())) })