From 16b55f9519d46711228a1b81792e971542557ca6 Mon Sep 17 00:00:00 2001 From: kiryazovi-redis Date: Thu, 4 Dec 2025 17:19:34 +0200 Subject: [PATCH] chore(tests): Reduce flakyness of a single test related to Smart Client Handoff (#3641) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * test: reduce flakiness in e2e tests - Increase traffic analysis duration from 30s to 60s in endpoint types test to allow sufficient time for push notification analysis - Fix TLS config test function name by replacing Cyrillic 'Т' with ASCII 'T' to ensure proper test discovery * test: disable TLS configurations test Skip TLS configurations test due to missing TLS environment in test setup --- maintnotifications/e2e/scenario_endpoint_types_test.go | 2 +- maintnotifications/e2e/scenario_tls_configs_test.go | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/maintnotifications/e2e/scenario_endpoint_types_test.go b/maintnotifications/e2e/scenario_endpoint_types_test.go index 90115ecb..af568086 100644 --- a/maintnotifications/e2e/scenario_endpoint_types_test.go +++ b/maintnotifications/e2e/scenario_endpoint_types_test.go @@ -317,7 +317,7 @@ func TestEndpointTypesPushNotifications(t *testing.T) { p("Bind action completed for %s: %s %s", endpointTest.name, bindStatus.Status, actionOutputIfFailed(bindStatus)) // Continue traffic for analysis - time.Sleep(30 * time.Second) + time.Sleep(60 * time.Second) commandsRunner.Stop() // Analyze results for this endpoint type diff --git a/maintnotifications/e2e/scenario_tls_configs_test.go b/maintnotifications/e2e/scenario_tls_configs_test.go index 673fcacc..eb571a9a 100644 --- a/maintnotifications/e2e/scenario_tls_configs_test.go +++ b/maintnotifications/e2e/scenario_tls_configs_test.go @@ -13,9 +13,9 @@ import ( "github.com/redis/go-redis/v9/maintnotifications" ) -// TODO ADD TLS CONFIGS // TestTLSConfigurationsPushNotifications tests push notifications with different TLS configurations -func ТestTLSConfigurationsPushNotifications(t *testing.T) { +func TestTLSConfigurationsPushNotifications(t *testing.T) { + t.Skip("Test disabled due to tls environment missing in test environment") if os.Getenv("E2E_SCENARIO_TESTS") != "true" { t.Skip("Scenario tests require E2E_SCENARIO_TESTS=true") }