fix(stats): 回答耗时改用 answered_at,修测试环境实测出的三处问题
在测试环境(box-dev-api)上逐项验算后发现「回答耗时」这一口径是错的。 1. 回答耗时不能用 updated_at - created_at(P0) GORM 的 Update 会自动刷新 updated_at,而已读、置顶、公开/私密切换全走 Update。 实测:一条 10 天前的提问被 SetRead 标记已读后,updated_at 被推到当下(delta=240h)。 于是「回答耗时」实际测的是「提问到最后一次被碰」:线上那 6 条「超过 7 天」 和偏低的 24 小时回复占比都是这么来的。 修法:新增 questions.answered_at(首次回答时间,可空),只在 answer 由空变非空时 写一次,改写回答不重置;聚合改用 COALESCE(answered_at, updated_at), 首次加列时用 updated_at 冻结历史一次(Migrator().HasColumn 判断是否刚加列), 否则历史值会随着后续的已读/置顶继续变大。 2. 环比句式读不通 中文「上月提问量比前一个月 -69.6%」、英文「...: -69.6% questions.」都是病句。 改为「上月提问量环比 %s%%」/「Month-over-month change for last month: %s%%」。 3. 脚注说「当日 0 点的快照」不实 实际是当天第一次访问时算、服务重启也会重算(实测算出时刻 01:11:50, 基准日却写 2026-10-24,说明它含了 0 点之后的数据)。改成准确描述。 4. 数据库故障时请求会排成 N×30s ovRefresh 串行化 + 30 秒超时意味着数据库挂起时第 N 个请求要等 N×30 秒。 加 30 秒失败冷却:冷却期内直接降级(有旧快照用旧快照),不再碰数据库。 todayNumbers 失败也盖时间戳,不再每个请求都打两条 COUNT。 测试:新增 internal/db/answered_at_test.go(5 例:只记一次、私密路径也记、 分桶以 answered_at 为准、历史行回退、回填),并做过变异验证 —— 把口径改回 updated_at 后测试立即失败。internal/stats 补 3 例冷却测试。全量构建与测试通过 (internal/storage 的构建失败是 overlay 桩的既有问题)。
This commit is contained in:
@@ -6,6 +6,7 @@ package stats
|
||||
|
||||
import (
|
||||
"context"
|
||||
"errors"
|
||||
"strings"
|
||||
"testing"
|
||||
"time"
|
||||
@@ -180,3 +181,100 @@ func TestTodayCacheExpiresAcrossMidnight(t *testing.T) {
|
||||
require.EqualValues(t, 7, q, "跨日后必须重算,不能沿用昨天的数字")
|
||||
require.EqualValues(t, 5, a)
|
||||
}
|
||||
|
||||
// failingAnalytics 模拟数据库挂掉:每次调用都报错,并记录被调用次数。
|
||||
type failingAnalytics struct {
|
||||
siteCalls int
|
||||
}
|
||||
|
||||
func (f *failingAnalytics) SiteAnalytics(context.Context, time.Time) (*db.SiteAnalytics, error) {
|
||||
f.siteCalls++
|
||||
return nil, errors.New("db is down")
|
||||
}
|
||||
|
||||
func (f *failingAnalytics) TodayCounts(context.Context, time.Time) (int64, int64, error) {
|
||||
return 0, 0, errors.New("db is down")
|
||||
}
|
||||
|
||||
// TestOverviewFailureCooldown 聚合失败后要冷却,不能每个请求都重排一次全表聚合。
|
||||
//
|
||||
// 没有冷却时,数据库挂起(连接不断、查询不返回)会让排队请求各自耗满 30 秒超时,
|
||||
// 第 N 个请求要等 N×30 秒 —— 页面对所有人一起超时。
|
||||
func TestOverviewFailureCooldown(t *testing.T) {
|
||||
prev := db.Analytics
|
||||
t.Cleanup(func() {
|
||||
db.Analytics = prev
|
||||
InvalidateOverview()
|
||||
})
|
||||
|
||||
f := &failingAnalytics{}
|
||||
db.Analytics = f
|
||||
InvalidateOverview()
|
||||
|
||||
_, err := OverviewData(context.Background())
|
||||
require.Error(t, err, "没有任何快照时应该把错误交给调用方")
|
||||
|
||||
// 冷却期内的请求直接失败,不再打库。
|
||||
for i := 0; i < 5; i++ {
|
||||
_, err := OverviewData(context.Background())
|
||||
require.Error(t, err)
|
||||
}
|
||||
require.Equal(t, 1, f.siteCalls, "冷却期内不该重复聚合")
|
||||
}
|
||||
|
||||
// TestOverviewCooldownClearedOnSuccess 冷却只针对失败,成功后照常刷新。
|
||||
func TestOverviewCooldownClearedOnSuccess(t *testing.T) {
|
||||
prev := db.Analytics
|
||||
t.Cleanup(func() {
|
||||
db.Analytics = prev
|
||||
InvalidateOverview()
|
||||
})
|
||||
|
||||
bad := &failingAnalytics{}
|
||||
db.Analytics = bad
|
||||
InvalidateOverview()
|
||||
_, err := OverviewData(context.Background())
|
||||
require.Error(t, err)
|
||||
|
||||
// 数据库恢复
|
||||
good := &fakeAnalytics{todayQ: 1, todayA: 1}
|
||||
db.Analytics = good
|
||||
// 手动把冷却时间挪到过去,模拟「过了 30 秒」
|
||||
ovMu.Lock()
|
||||
ovFailAt = time.Now().Add(-2 * ovFailCooldown)
|
||||
ovMu.Unlock()
|
||||
|
||||
v, err := OverviewData(context.Background())
|
||||
require.NoError(t, err)
|
||||
require.EqualValues(t, 10, v.TotalQuestions)
|
||||
require.Equal(t, 1, good.siteCalls)
|
||||
|
||||
// 成功后冷却状态要清掉,否则下次失败判断会用到旧时间
|
||||
ovMu.Lock()
|
||||
failAt, failErr := ovFailAt, ovFailErr
|
||||
ovMu.Unlock()
|
||||
require.True(t, failAt.IsZero(), "成功后必须清掉失败时间")
|
||||
require.NoError(t, failErr)
|
||||
}
|
||||
|
||||
// TestTodayFailureStillThrottled 数据库挂掉时「今日」两项也不能每个请求都打两条 COUNT。
|
||||
func TestTodayFailureStillThrottled(t *testing.T) {
|
||||
prev := db.Analytics
|
||||
t.Cleanup(func() {
|
||||
db.Analytics = prev
|
||||
InvalidateOverview()
|
||||
})
|
||||
|
||||
db.Analytics = &failingAnalytics{}
|
||||
InvalidateOverview()
|
||||
|
||||
now := time.Now()
|
||||
q, a := todayNumbers(context.Background(), now, 7, 3)
|
||||
require.EqualValues(t, 7, q, "失败时回退到调用方给的值")
|
||||
require.EqualValues(t, 3, a)
|
||||
|
||||
// 紧接一次:命中失败缓存,不该再打库(打了也没法区分,直接看返回值仍是回退值)
|
||||
q, a = todayNumbers(context.Background(), now.Add(time.Second), 0, 0)
|
||||
require.EqualValues(t, 7, q, "冷却期内沿用上一次的结果,不再打库")
|
||||
require.EqualValues(t, 3, a)
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user