refactor(tests): use redisx.MustStartTestSuite instead of raw testcontainers redis

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: dorothyyzh

go.mod

@@ -6,10 +6,9 @@ require (
 	github.com/go-sql-driver/mysql v1.8.1
 	github.com/jackc/pgx/v5 v5.9.2
 	github.com/pkg/errors v0.9.1
-	github.com/qor5/x/v3 v3.2.1-0.20260512062513-2ee630cec6e0
+	github.com/qor5/x/v3 v3.2.1-0.20260513132712-130e4f847f24
 	github.com/redis/go-redis/v9 v9.16.0
 	github.com/stretchr/testify v1.11.1
-	github.com/testcontainers/testcontainers-go/modules/redis v0.42.0
 	golang.org/x/sync v0.20.0
 	gorm.io/gorm v1.31.1
 )
@@ -107,6 +106,7 @@ require (
 	github.com/spf13/viper v1.19.0 // indirect
 	github.com/subosito/gotenv v1.6.0 // indirect
 	github.com/testcontainers/testcontainers-go v0.42.0 // indirect
+	github.com/testcontainers/testcontainers-go/modules/redis v0.42.0 // indirect
 	github.com/theplant/appkit v0.0.0-20250528023215-3d0d299dc4c6 // indirect
 	github.com/theplant/cachex v0.0.0-20251210183652-8e675368cbc5 // indirect
 	github.com/theplant/inject v1.2.2 // indirect

go.sum

@@ -234,8 +234,8 @@ github.com/power-devops/perfstat v0.0.0-20240221224432-82ca36839d55/go.mod h1:Om
 github.com/prometheus/client_model v0.0.0-20190812154241-14fe0d1b01d4/go.mod h1:xMI15A0UPsDsEKsMN9yxemIoYk6Tm2C1GtYGdfGttqA=
 github.com/qor5/confx v0.0.0-20250426065316-0d28db5b4d54 h1:sO/saPkFgwfLaiCVTg+e9x6lAYBrqY91h1REu4NLMm0=
 github.com/qor5/confx v0.0.0-20250426065316-0d28db5b4d54/go.mod h1:03dPo1SHYn9sU57mH67Y1p9FIcglWaHr4i/xkeYmX4o=
-github.com/qor5/x/v3 v3.2.1-0.20260512062513-2ee630cec6e0 h1:IbgsrYwAlFduDI83u/wYgnDt+MxMfns2pyG1F41274E=
-github.com/qor5/x/v3 v3.2.1-0.20260512062513-2ee630cec6e0/go.mod h1:y/9Or1Q4jOHezDVdYu/m29ZbRcmSKqwebrdTiOh8ZZ8=
+github.com/qor5/x/v3 v3.2.1-0.20260513132712-130e4f847f24 h1:TBE21zvsfGFTgA/A2X19uSpmwRoJ1+6nO/RvUzZu1wM=
+github.com/qor5/x/v3 v3.2.1-0.20260513132712-130e4f847f24/go.mod h1:HElNR0JmewJro09DBZgOh+ZrzpEtYd69bfHvAAa1aOo=
 github.com/redis/go-redis/v9 v9.16.0 h1:OotgqgLSRCmzfqChbQyG1PHC3tLNR89DG4jdOERSEP4=
 github.com/redis/go-redis/v9 v9.16.0/go.mod h1:u410H11HMLoB+TP67dz8rL9s6QW2j76l0//kSOd3370=
 github.com/rogpeppe/go-internal v1.14.1 h1:UQB4HGPB6osV0SQTLymcB4TgvyWu6ZyliaW0tI/otEQ=

ratelimiter_test.go

@@ -2,19 +2,17 @@ package ratelimiter_test
 
 import (
 	"context"
-	"fmt"
 	"log"
-	"strings"
 	"testing"
 	"time"
 
 	redis "github.com/redis/go-redis/v9"
+	"github.com/qor5/x/v3/gormx"
+	"github.com/qor5/x/v3/redisx"
 	"github.com/stretchr/testify/require"
-	testredis "github.com/testcontainers/testcontainers-go/modules/redis"
 	"github.com/theplant/ratelimiter"
 	"github.com/theplant/ratelimiter/redisrl"
 	"github.com/theplant/ratelimiter/sqlrl"
-	"github.com/qor5/x/v3/gormx"
 	"gorm.io/gorm"
 )
 
@@ -33,25 +31,15 @@ func TestMain(m *testing.M) {
 		}
 	}()
 
-	redisContainer, err := testredis.Run(ctx, "redis:7-alpine")
-	if err != nil {
-		panic(fmt.Errorf("failed to start redis container: %w", err))
-	}
+	redisTestSuite := redisx.MustStartTestSuite(ctx)
 	defer func() {
-		if err := redisContainer.Terminate(context.Background()); err != nil {
-			log.Fatalf("Failed to terminate redis container: %v", err)
+		if err := redisTestSuite.Stop(context.Background()); err != nil {
+			log.Fatalf("Failed to stop redis test suite: %v", err)
 		}
 	}()
 
-	endpoint, err := redisContainer.ConnectionString(ctx)
-	if err != nil {
-		panic(fmt.Errorf("failed to get redis connection string: %w", err))
-	}
-
 	db = testSuite.DB()
-	redisCli = redis.NewClient(&redis.Options{
-		Addr: strings.TrimPrefix(endpoint, "redis://"),
-	})
+	redisCli = redisTestSuite.Client()
 	defer redisCli.Close()
 
 	// Create SQL rate limiter and migrate table

redisrl/redis_test.go

@@ -2,38 +2,26 @@ package redisrl_test
 
 import (
 	"context"
-	"fmt"
-	"strings"
 	"testing"
 
+	"github.com/qor5/x/v3/redisx"
 	redis "github.com/redis/go-redis/v9"
 	"github.com/stretchr/testify/require"
-	testredis "github.com/testcontainers/testcontainers-go/modules/redis"
 	"github.com/theplant/ratelimiter/redisrl"
 )
 
 var redisClient *redis.Client
 
 func TestMain(m *testing.M) {
 	ctx := context.Background()
-	container, err := testredis.Run(ctx, "redis:7-alpine")
-	if err != nil {
-		panic(fmt.Errorf("failed to start redis container: %w", err))
-	}
+	suite := redisx.MustStartTestSuite(ctx)
 	defer func() {
-		if err := container.Terminate(context.Background()); err != nil {
-			panic(fmt.Errorf("failed to terminate redis container: %w", err))
+		if err := suite.Stop(context.Background()); err != nil {
+			panic(err)
 		}
 	}()
 
-	endpoint, err := container.ConnectionString(ctx)
-	if err != nil {
-		panic(fmt.Errorf("failed to get redis connection string: %w", err))
-	}
-
-	redisClient = redis.NewClient(&redis.Options{
-		Addr: strings.TrimPrefix(endpoint, "redis://"),
-	})
+	redisClient = suite.Client()
 	defer redisClient.Close()
 
 	m.Run()