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

chore(tests): Reduce flakyness of a single test related to Smart Client Handoff (#3641)

Browse Source 
* 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
This commit is contained in:
kiryazovi-redis
2025-12-04 17:19:34 +02:00
committed by GitHub
parent e2153f5c16
commit 16b55f9519
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
maintnotifications/e2e/scenario_endpoint_types_test.go
View File

@@ -317,7 +317,7 @@ func TestEndpointTypesPushNotifications(t *testing.T) {
p("Bind action completed for %s: %s %s", endpointTest.name, bindStatus.Status, actionOutputIfFailed(bindStatus)) p("Bind action completed for %s: %s %s", endpointTest.name, bindStatus.Status, actionOutputIfFailed(bindStatus))
// Continue traffic for analysis // Continue traffic for analysis
time.Sleep(30 * time.Second) time.Sleep(60 * time.Second)
commandsRunner.Stop() commandsRunner.Stop()
// Analyze results for this endpoint type // Analyze results for this endpoint type

4
maintnotifications/e2e/scenario_tls_configs_test.go
View File

@@ -13,9 +13,9 @@ import (
"github.com/redis/go-redis/v9/maintnotifications" "github.com/redis/go-redis/v9/maintnotifications"
) )
// TODO ADD TLS CONFIGS
// TestTLSConfigurationsPushNotifications tests push notifications with different TLS configurations // 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" { if os.Getenv("E2E_SCENARIO_TESTS") != "true" {
t.Skip("Scenario tests require E2E_SCENARIO_TESTS=true") t.Skip("Scenario tests require E2E_SCENARIO_TESTS=true")
} }