All checks were successful
构建并部署到测试环境(无 SSH) / build-and-deploy (push) Successful in 5m30s
新增功能: - 门店套餐分配管理(shop_package_allocation):支持门店套餐库存管理 - 门店套餐系列分配管理(shop_series_allocation):支持套餐系列分配和佣金层级设置 - 我的套餐查询(my_package):支持门店查询自己的套餐分配情况 测试改进: - 统一集成测试基础设施,新增 testutils.NewIntegrationTestEnv - 重构所有集成测试使用新的测试环境设置 - 移除旧的测试辅助函数和冗余测试文件 - 新增 test_helpers_test.go 统一任务测试辅助 技术细节: - 新增数据库迁移 000025_create_shop_allocation_tables - 新增 3 个 Handler、Service、Store 和对应的单元测试 - 更新 OpenAPI 文档和文档生成器 - 测试覆盖率:Service 层 > 90% Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
338 lines
9.3 KiB
Go
338 lines
9.3 KiB
Go
package integration
|
|
|
|
import (
|
|
"fmt"
|
|
"io"
|
|
"net/http/httptest"
|
|
"testing"
|
|
"time"
|
|
|
|
"github.com/break/junhong_cmp_fiber/internal/middleware"
|
|
"github.com/break/junhong_cmp_fiber/pkg/errors"
|
|
"github.com/break/junhong_cmp_fiber/pkg/logger"
|
|
"github.com/break/junhong_cmp_fiber/pkg/response"
|
|
"github.com/gofiber/fiber/v2"
|
|
"github.com/stretchr/testify/assert"
|
|
"github.com/stretchr/testify/require"
|
|
"go.uber.org/zap"
|
|
)
|
|
|
|
// setupRateLimiterTestApp creates a Fiber app with rate limiter for testing
|
|
func setupRateLimiterTestApp(t *testing.T, max int, expiration time.Duration) *fiber.App {
|
|
t.Helper()
|
|
|
|
// Initialize logger
|
|
appLogConfig := logger.LogRotationConfig{
|
|
Filename: "logs/app_test.log",
|
|
MaxSize: 10,
|
|
MaxBackups: 3,
|
|
MaxAge: 7,
|
|
Compress: false,
|
|
}
|
|
accessLogConfig := logger.LogRotationConfig{
|
|
Filename: "logs/access_test.log",
|
|
MaxSize: 10,
|
|
MaxBackups: 3,
|
|
MaxAge: 7,
|
|
Compress: false,
|
|
}
|
|
if err := logger.InitLoggers("info", false, appLogConfig, accessLogConfig); err != nil {
|
|
t.Fatalf("failed to initialize logger: %v", err)
|
|
}
|
|
|
|
zapLogger, _ := zap.NewDevelopment()
|
|
app := fiber.New(fiber.Config{
|
|
ErrorHandler: errors.SafeErrorHandler(zapLogger),
|
|
})
|
|
|
|
// Add rate limiter middleware (nil storage = in-memory)
|
|
app.Use(middleware.RateLimiter(max, expiration, nil))
|
|
|
|
// Add test route
|
|
app.Get("/api/v1/test", func(c *fiber.Ctx) error {
|
|
return response.Success(c, fiber.Map{
|
|
"message": "success",
|
|
})
|
|
})
|
|
|
|
return app
|
|
}
|
|
|
|
// TestRateLimiter_LimitExceeded tests that rate limiter returns 429 when limit is exceeded
|
|
func TestRateLimiter_LimitExceeded(t *testing.T) {
|
|
// Create app with low limit for easy testing
|
|
max := 5
|
|
expiration := 1 * time.Minute
|
|
app := setupRateLimiterTestApp(t, max, expiration)
|
|
|
|
// Make requests up to the limit
|
|
for i := 1; i <= max; i++ {
|
|
req := httptest.NewRequest("GET", "/api/v1/test", nil)
|
|
req.Header.Set("X-Forwarded-For", "192.168.1.100") // Simulate same IP
|
|
|
|
resp, err := app.Test(req, -1)
|
|
require.NoError(t, err)
|
|
resp.Body.Close()
|
|
|
|
assert.Equal(t, 200, resp.StatusCode, "Request %d should succeed", i)
|
|
}
|
|
|
|
// The next request should be rate limited
|
|
req := httptest.NewRequest("GET", "/api/v1/test", nil)
|
|
req.Header.Set("X-Forwarded-For", "192.168.1.100")
|
|
|
|
resp, err := app.Test(req, -1)
|
|
require.NoError(t, err)
|
|
defer resp.Body.Close()
|
|
|
|
// Should get 429 Too Many Requests
|
|
assert.Equal(t, 429, resp.StatusCode, "Request should be rate limited")
|
|
|
|
// Check response body
|
|
body, err := io.ReadAll(resp.Body)
|
|
require.NoError(t, err)
|
|
t.Logf("Rate limit response: %s", string(body))
|
|
|
|
// Should contain error code 1008 (CodeTooManyRequests)
|
|
assert.Contains(t, string(body), `"code":1008`, "Response should have too many requests error code")
|
|
// Message is in Chinese: "请求过多,请稍后重试"
|
|
assert.Contains(t, string(body), "请求过多", "Response should have rate limit message")
|
|
}
|
|
|
|
// TestRateLimiter_ResetAfterExpiration tests that rate limit resets after window expiration
|
|
func TestRateLimiter_ResetAfterExpiration(t *testing.T) {
|
|
// Create app with short expiration for testing
|
|
max := 3
|
|
expiration := 2 * time.Second
|
|
app := setupRateLimiterTestApp(t, max, expiration)
|
|
|
|
// Make requests up to the limit
|
|
for i := 1; i <= max; i++ {
|
|
req := httptest.NewRequest("GET", "/api/v1/test", nil)
|
|
req.Header.Set("X-Forwarded-For", "192.168.1.101")
|
|
|
|
resp, err := app.Test(req, -1)
|
|
require.NoError(t, err)
|
|
resp.Body.Close()
|
|
|
|
assert.Equal(t, 200, resp.StatusCode, "Request %d should succeed", i)
|
|
}
|
|
|
|
// Next request should be rate limited
|
|
req := httptest.NewRequest("GET", "/api/v1/test", nil)
|
|
req.Header.Set("X-Forwarded-For", "192.168.1.101")
|
|
|
|
resp, err := app.Test(req, -1)
|
|
require.NoError(t, err)
|
|
resp.Body.Close()
|
|
|
|
assert.Equal(t, 429, resp.StatusCode, "Request should be rate limited")
|
|
|
|
// Wait for rate limit window to expire
|
|
t.Log("Waiting for rate limit window to reset...")
|
|
time.Sleep(expiration + 500*time.Millisecond)
|
|
|
|
// Request should succeed after reset
|
|
req = httptest.NewRequest("GET", "/api/v1/test", nil)
|
|
req.Header.Set("X-Forwarded-For", "192.168.1.101")
|
|
|
|
resp, err = app.Test(req, -1)
|
|
require.NoError(t, err)
|
|
defer resp.Body.Close()
|
|
|
|
assert.Equal(t, 200, resp.StatusCode, "Request should succeed after rate limit reset")
|
|
|
|
body, err := io.ReadAll(resp.Body)
|
|
require.NoError(t, err)
|
|
|
|
assert.Contains(t, string(body), `"code":0`, "Response should be successful after reset")
|
|
}
|
|
|
|
// TestRateLimiter_PerIPRateLimiting tests that different IPs have separate rate limits
|
|
func TestRateLimiter_PerIPRateLimiting(t *testing.T) {
|
|
max := 5
|
|
expiration := 1 * time.Minute
|
|
|
|
// Test with multiple different IPs
|
|
ips := []string{
|
|
"192.168.1.10",
|
|
"192.168.1.20",
|
|
"192.168.1.30",
|
|
}
|
|
|
|
for _, ip := range ips {
|
|
ip := ip // Capture for closure
|
|
t.Run(fmt.Sprintf("IP_%s", ip), func(t *testing.T) {
|
|
// Create fresh app for each IP test to avoid shared limiter state
|
|
freshApp := setupRateLimiterTestApp(t, max, expiration)
|
|
|
|
// Each IP should be able to make 'max' successful requests
|
|
for i := 1; i <= max; i++ {
|
|
req := httptest.NewRequest("GET", "/api/v1/test", nil)
|
|
req.Header.Set("X-Forwarded-For", ip)
|
|
|
|
resp, err := freshApp.Test(req, -1)
|
|
require.NoError(t, err)
|
|
resp.Body.Close()
|
|
|
|
assert.Equal(t, 200, resp.StatusCode, "IP %s request %d should succeed", ip, i)
|
|
}
|
|
|
|
// The next request for this IP should be rate limited
|
|
req := httptest.NewRequest("GET", "/api/v1/test", nil)
|
|
req.Header.Set("X-Forwarded-For", ip)
|
|
|
|
resp, err := freshApp.Test(req, -1)
|
|
require.NoError(t, err)
|
|
resp.Body.Close()
|
|
|
|
assert.Equal(t, 429, resp.StatusCode, "IP %s should be rate limited", ip)
|
|
})
|
|
}
|
|
}
|
|
|
|
// TestRateLimiter_ConcurrentRequests tests rate limiter with concurrent requests from same IP
|
|
func TestRateLimiter_ConcurrentRequests(t *testing.T) {
|
|
// Create app with limit
|
|
max := 10
|
|
expiration := 1 * time.Minute
|
|
app := setupRateLimiterTestApp(t, max, expiration)
|
|
|
|
// Make concurrent requests
|
|
concurrentRequests := 15
|
|
results := make(chan int, concurrentRequests)
|
|
|
|
for i := 0; i < concurrentRequests; i++ {
|
|
go func() {
|
|
req := httptest.NewRequest("GET", "/api/v1/test", nil)
|
|
req.Header.Set("X-Forwarded-For", "192.168.1.200")
|
|
|
|
resp, err := app.Test(req, -1)
|
|
if err != nil {
|
|
results <- 0
|
|
return
|
|
}
|
|
defer resp.Body.Close()
|
|
|
|
results <- resp.StatusCode
|
|
}()
|
|
}
|
|
|
|
// Collect results
|
|
var successCount, rateLimitedCount int
|
|
for i := 0; i < concurrentRequests; i++ {
|
|
status := <-results
|
|
if status == 200 {
|
|
successCount++
|
|
} else if status == 429 {
|
|
rateLimitedCount++
|
|
}
|
|
}
|
|
|
|
t.Logf("Concurrent requests: %d success, %d rate limited", successCount, rateLimitedCount)
|
|
|
|
// Should have exactly 'max' successful requests
|
|
assert.Equal(t, max, successCount, "Should have exactly max successful requests")
|
|
|
|
// Remaining requests should be rate limited
|
|
assert.Equal(t, concurrentRequests-max, rateLimitedCount, "Remaining requests should be rate limited")
|
|
}
|
|
|
|
// TestRateLimiter_DifferentLimits tests rate limiter configuration with different limits
|
|
func TestRateLimiter_DifferentLimits(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
max int
|
|
expiration time.Duration
|
|
}{
|
|
{
|
|
name: "low_limit",
|
|
max: 2,
|
|
expiration: 1 * time.Minute,
|
|
},
|
|
{
|
|
name: "medium_limit",
|
|
max: 10,
|
|
expiration: 1 * time.Minute,
|
|
},
|
|
{
|
|
name: "high_limit",
|
|
max: 100,
|
|
expiration: 1 * time.Minute,
|
|
},
|
|
}
|
|
|
|
for _, tt := range tests {
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
app := setupRateLimiterTestApp(t, tt.max, tt.expiration)
|
|
|
|
// Make requests up to limit
|
|
for i := 1; i <= tt.max; i++ {
|
|
req := httptest.NewRequest("GET", "/api/v1/test", nil)
|
|
req.Header.Set("X-Forwarded-For", fmt.Sprintf("192.168.1.%d", 50+i))
|
|
|
|
resp, err := app.Test(req, -1)
|
|
require.NoError(t, err)
|
|
resp.Body.Close()
|
|
|
|
assert.Equal(t, 200, resp.StatusCode)
|
|
}
|
|
|
|
// Next request should be rate limited
|
|
req := httptest.NewRequest("GET", "/api/v1/test", nil)
|
|
req.Header.Set("X-Forwarded-For", fmt.Sprintf("192.168.1.%d", 50))
|
|
|
|
resp, err := app.Test(req, -1)
|
|
require.NoError(t, err)
|
|
resp.Body.Close()
|
|
|
|
assert.Equal(t, 429, resp.StatusCode, "Should be rate limited after %d requests", tt.max)
|
|
})
|
|
}
|
|
}
|
|
|
|
// TestRateLimiter_ShortWindow tests rate limiter with very short time window
|
|
func TestRateLimiter_ShortWindow(t *testing.T) {
|
|
// Create app with short window
|
|
max := 3
|
|
expiration := 1 * time.Second
|
|
app := setupRateLimiterTestApp(t, max, expiration)
|
|
|
|
// Make first batch of requests
|
|
for i := 1; i <= max; i++ {
|
|
req := httptest.NewRequest("GET", "/api/v1/test", nil)
|
|
req.Header.Set("X-Forwarded-For", "192.168.1.250")
|
|
|
|
resp, err := app.Test(req, -1)
|
|
require.NoError(t, err)
|
|
resp.Body.Close()
|
|
|
|
assert.Equal(t, 200, resp.StatusCode)
|
|
}
|
|
|
|
// Should be rate limited now
|
|
req := httptest.NewRequest("GET", "/api/v1/test", nil)
|
|
req.Header.Set("X-Forwarded-For", "192.168.1.250")
|
|
|
|
resp, err := app.Test(req, -1)
|
|
require.NoError(t, err)
|
|
resp.Body.Close()
|
|
|
|
assert.Equal(t, 429, resp.StatusCode)
|
|
|
|
// Wait for window to expire
|
|
time.Sleep(expiration + 200*time.Millisecond)
|
|
|
|
// Should be able to make requests again
|
|
for i := 1; i <= max; i++ {
|
|
req := httptest.NewRequest("GET", "/api/v1/test", nil)
|
|
req.Header.Set("X-Forwarded-For", "192.168.1.250")
|
|
|
|
resp, err := app.Test(req, -1)
|
|
require.NoError(t, err)
|
|
resp.Body.Close()
|
|
|
|
assert.Equal(t, 200, resp.StatusCode, "Request %d should succeed after window reset", i)
|
|
}
|
|
}
|