Skip to content

Commit 16b55f9

Browse files
chore(tests): Reduce flakyness of a single test related to Smart Client Handoff (#3641)
* 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
1 parent e2153f5 commit 16b55f9

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

maintnotifications/e2e/scenario_endpoint_types_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -317,7 +317,7 @@ func TestEndpointTypesPushNotifications(t *testing.T) {
317317
p("Bind action completed for %s: %s %s", endpointTest.name, bindStatus.Status, actionOutputIfFailed(bindStatus))
318318

319319
// Continue traffic for analysis
320-
time.Sleep(30 * time.Second)
320+
time.Sleep(60 * time.Second)
321321
commandsRunner.Stop()
322322

323323
// Analyze results for this endpoint type

maintnotifications/e2e/scenario_tls_configs_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,9 @@ import (
1313
"github.com/redis/go-redis/v9/maintnotifications"
1414
)
1515

16-
// TODO ADD TLS CONFIGS
1716
// TestTLSConfigurationsPushNotifications tests push notifications with different TLS configurations
18-
func ТestTLSConfigurationsPushNotifications(t *testing.T) {
17+
func TestTLSConfigurationsPushNotifications(t *testing.T) {
18+
t.Skip("Test disabled due to tls environment missing in test environment")
1919
if os.Getenv("E2E_SCENARIO_TESTS") != "true" {
2020
t.Skip("Scenario tests require E2E_SCENARIO_TESTS=true")
2121
}

0 commit comments

Comments
 (0)