Files
Scoreboard/server/internal/httpapi/handlers_results_test.go
T
AskaEth 8dd599bd97 fix: 实机复测六批发现的 16 条缺陷(导入编码静默错误 / 导出观感 / 错误分级 / 大屏裁切)
先把「实机复测」能在这台 Linux 机器上验的全部自己验掉(真服务端 + 真 Chromium + 真样本),
六批全部通过,另挖出 16 条缺陷,逐条修完并复验:

导入
- 坏字节被静默替换成 U+FFFD 却报「组别不存在」→ 检测 RuneError 回 400 + 可操作中文文案
- UTF-16 无 BOM 误报「缺少列 event_code」→ 保守判据识别并正确解码
- CSV 语法错只有英文 → 中文提示,英文原文保留在 detail
- 表头缺列补「第 1 行应是表头」的指向

导出
- 名次列宽 8 → 11(放得下「3(并列)」,Excel 里不再硬裁)
- 运动员/队列宽 18 → 24(含奖牌明细的同一列)
- 聚合流水行补组别(仅单位在该小项只参加一个组别时回推;多组别宁可留空也不猜)
- docProps 作者从 excelize 默认的 xuri 改成中性名 + 真实导出时间
- Content-Disposition 补 ASCII 退化 filename(9 个下载端点,含此前没有测试的 project.zip)

错误分级与限流
- 未知大屏指令 500 → 400 VALIDATION_FAILED(传输层真失败仍走 5xx)
- 5xx 的 ERROR 日志补 role / ip(4xx 的 WARN 一直有)
- 写限流 30 → 120 次/分钟/用户(登录 5 次/分/IP 不动,性质不同)
- 赛事名含非法字符时导出给 400 + 可操作提示(原来永远 500「服务端异常」)

大屏与基础设施
- 实时赛况 2 栏成绩末位被裁(1:09.00 显示成 1:09.0)→ 成绩列按内容定宽 + nowrap;
  字号不缩,让位给可省略号截断的运动员列。真 Chromium 实测 slack −45px → +6px、0 裁切、
  scrollWidth 恒 1920(无横向滚动条);顺带修掉 1 栏长成绩同样被裁 8px 的问题
- logging 启动即预建两个日志目录(原来 interceptions/ 懒创建,现场会以为日志坏了)
- C→S 裸帧由静默忽略改为 WARN,并把「要套 {event,data} 外壳」写进日志
- 建单位/大项漏传 active 不再静默建成「停用」(新增 decodeJSONPresence 按键是否存在判意图)

文档
- README 单位 logo 措辞收窄到「大屏端」(管理端只负责上传配置)
- 导入规范补三类编码报错说明 + 现场另存结论 + 限流提示
- 部署指南补「导出前先看赛事名称」与「切换后怎么确认真的切过去了」
- devdocs 05/08/13 同步(含第三十五轮的完整记录与教训)

自检:服务端 13 包连跑两遍全绿;前端 257 例;vue-tsc + build 通过;Console 18 文件。
版本号 dev_0.1.4 → dev_0.1.5。
2026-09-11 15:35:42 +08:00

228 lines
9.2 KiB
Go

package httpapi_test
import (
"net/http"
"testing"
"git.yeij.top/AskaEth/Scoreboard/server/internal/httpapi"
"git.yeij.top/AskaEth/Scoreboard/server/internal/realtime"
)
// strictDisplaySender 与 main.go 的 displaySender 同口径:经实时层校验 action,未知 action 直接报错。
// matched 为 nil 时按「广播算命中一块屏、定向一条算命中一块」返回,便于断言正常路径;
// 需要模拟「目标已离线」时把它设成 0。
// failWith 非空时直接返回它,用来造一个**真正的传输层 5xx**(写 socket 失败之类):
// 未知 action 现在是 4xx,5xx 的日志用例不能再拿它当触发源。
type strictDisplaySender struct {
sent []string
targets [][]string
matched *int
failWith error
}
func (d *strictDisplaySender) SendDisplayCommand(action string, payload any, targets []string) (int, error) {
if _, err := realtime.NewDisplayCommand(action, payload); err != nil {
return 0, err
}
if d.failWith != nil {
return 0, d.failWith
}
d.sent = append(d.sent, action)
d.targets = append(d.targets, targets)
if d.matched != nil {
return *d.matched, nil
}
if len(targets) == 0 {
return 1, nil
}
return len(targets), nil
}
// settingValue 读取 settings 表里的点号键。
func settingValue(t *testing.T, env *testEnv, key string) string {
t.Helper()
var v string
if err := env.ev.QueryRow(`SELECT value FROM settings WHERE key = ?`, key).Scan(&v); err != nil {
t.Fatalf("读取配置 %s: %v", key, err)
}
return v
}
// 个人榜的单位展示字段必须取全量单位表:停用但有积分的单位也要显示名称与简称。
func TestAthleteStandingsUsesFullTeamTable(t *testing.T) {
env := newTestEnv(t)
ids := env.seed(t)
env.createUser("recorder", "recorder-pass", "recorder")
client, _ := env.client()
env.login(client, "recorder", "recorder-pass")
env.do(client, http.MethodPut, "/api/results/draft", map[string]any{
"eventId": ids.event,
"results": []map[string]any{
{"entryId": ids.entryA, "resultValue": 11.58, "unit": "s", "status": "ok"},
{"entryId": ids.entryB, "resultValue": 11.92, "unit": "s", "status": "ok"},
},
}, true)
if status, env2 := env.do(client, http.MethodPost, "/api/results/publish", map[string]any{
"eventId": ids.event,
}, true); status != http.StatusOK {
t.Fatalf("发布应 200,得到 %d %+v", status, env2.Error)
}
env.exec(t, `UPDATE teams SET active = 0 WHERE id = ?`, ids.teamB)
_, env2 := env.do(client, http.MethodGet, "/api/standings/athletes", nil, false)
rows := decodeData[[]map[string]any](t, env2)
seen := map[int64]map[string]any{}
for _, row := range rows {
seen[int64(row["athleteId"].(float64))] = row
}
active := seen[ids.athleteA]
if active == nil || active["teamName"] != "第一分公司" || active["shortName"] != "一司" {
t.Fatalf("在职单位的个人行应带单位名与简称: %+v", active)
}
stopped := seen[ids.athB]
if stopped == nil {
t.Fatalf("停用单位的运动员仍应在个人榜上: %+v", rows)
}
if stopped["teamName"] != "第二分公司" || stopped["shortName"] != "二司" {
t.Fatalf("停用单位的个人行应补齐单位名与简称,得到 %+v", stopped)
}
}
// 个人榜的 logoPath 取「运动员所属单位」的 logo:姓名前显示的是单位标识,不是运动员头像。
func TestAthleteStandingsIncludesTeamLogo(t *testing.T) {
env := newTestEnv(t)
ids := env.seed(t)
env.createUser("recorder", "recorder-pass", "recorder")
client, _ := env.client()
env.login(client, "recorder", "recorder-pass")
env.do(client, http.MethodPut, "/api/results/draft", map[string]any{
"eventId": ids.event,
"results": []map[string]any{
{"entryId": ids.entryA, "resultValue": 11.58, "unit": "s", "status": "ok"},
{"entryId": ids.entryB, "resultValue": 11.92, "unit": "s", "status": "ok"},
},
}, true)
if status, env2 := env.do(client, http.MethodPost, "/api/results/publish", map[string]any{
"eventId": ids.event,
}, true); status != http.StatusOK {
t.Fatalf("发布应 200,得到 %d %+v", status, env2.Error)
}
env.exec(t, `UPDATE teams SET logo_path = ? WHERE id = ?`, "/uploads/teams/a.png", ids.teamA)
// 历史数据里 logo 可能是空串,同样按「无 logo」下发 null。
env.exec(t, `UPDATE teams SET logo_path = '' WHERE id = ?`, ids.teamB)
_, env2 := env.do(client, http.MethodGet, "/api/standings/athletes", nil, false)
rows := decodeData[[]map[string]any](t, env2)
seen := map[int64]map[string]any{}
for _, row := range rows {
seen[int64(row["athleteId"].(float64))] = row
}
withLogo := seen[ids.athleteA]
if withLogo == nil || withLogo["logoPath"] != "/uploads/teams/a.png" {
t.Fatalf("有 logo 的单位,个人行应带该单位的 logo: %+v", withLogo)
}
noLogo, ok := seen[ids.athB]["logoPath"]
if !ok || noLogo != nil {
t.Fatalf("无 logo 的单位,个人行 logoPath 应为 null,得到 %+v", seen[ids.athB])
}
}
// 公告必须落库:只转发的话全屏推送结束(seconds 秒)后底栏会回落到「暂无公告」,
// 新连上的大屏也拿不到当前公告。
func TestDisplayAnnouncePersistsAndClears(t *testing.T) {
sender := &strictDisplaySender{}
env := newTestEnvWith(t, func(o *httpapi.Options) { o.Display = sender })
env.createUser("admin", "admin-pass", "admin")
env.createUser("recorder", "recorder-pass", "recorder")
recorder, _ := env.client()
env.login(recorder, "recorder", "recorder-pass")
admin, _ := env.client()
env.login(admin, "admin", "admin-pass")
// 公告的推送权限在 recorder,配置接口是 admin-only,所以落库必须由指令这一侧完成。
announceLogs := changeLogCount(t, env, "settings", "upsert")
status, env2 := env.do(recorder, http.MethodPost, "/api/display/command", map[string]any{
"action": "announce",
"payload": map[string]any{"text": "决赛推迟到 15:00", "seconds": 30},
}, true)
if status != http.StatusOK || !env2.OK {
t.Fatalf("推送公告应 200,得到 %d %+v", status, env2.Error)
}
if len(sender.sent) != 1 || sender.sent[0] != "announce" {
t.Fatalf("公告应照旧转发给大屏,得到 %v", sender.sent)
}
if got := settingValue(t, env, "display.announce_text"); got != "决赛推迟到 15:00" {
t.Fatalf("公告正文应落库,得到 %q", got)
}
if got := settingValue(t, env, "display.announce_seconds"); got != "30" {
t.Fatalf("全屏时长应落库,得到 %q", got)
}
// 已连接的大屏靠 settings 变更重拉快照,没有这条变更它们看不到公告变化。
if got := changeLogCount(t, env, "settings", "upsert"); got != announceLogs+1 {
t.Fatalf("公告落库应新增 1 条 settings 变更,得到 %d → %d", announceLogs, got)
}
_, env2 = env.do(admin, http.MethodGet, "/api/settings", nil, false)
display := decodeData[map[string]any](t, env2)["display"].(map[string]any)
if display["announce"] != "决赛推迟到 15:00" {
t.Fatalf("settings.display.announce 应带当前公告: %+v", display)
}
// payload 没带 seconds 时落默认 20。
env.do(recorder, http.MethodPost, "/api/display/command", map[string]any{
"action": "announce",
"payload": map[string]any{"text": "第二条公告"},
}, true)
if got := settingValue(t, env, "display.announce_seconds"); got != "20" {
t.Fatalf("payload 未带 seconds 时应落默认 20,得到 %q", got)
}
// 清空公告:正文清空,重连的大屏拿到 null。
status, env2 = env.do(recorder, http.MethodPost, "/api/display/command",
map[string]any{"action": "announce_clear"}, true)
if status != http.StatusOK || !env2.OK {
t.Fatalf("清空公告应 200,得到 %d %+v", status, env2.Error)
}
if got := settingValue(t, env, "display.announce_text"); got != "" {
t.Fatalf("清空后公告正文应为空串,得到 %q", got)
}
_, env2 = env.do(admin, http.MethodGet, "/api/settings", nil, false)
display = decodeData[map[string]any](t, env2)["display"].(map[string]any)
if v, ok := display["announce"]; !ok || v != nil {
t.Fatalf("清空后 settings.display.announce 应为 null: %+v", display)
}
}
// 公告归档只对 announce 系列生效,其它指令不能碰 settings;公告时长在清空后保留。
func TestDisplayAnnounceLeavesOtherCommandsAlone(t *testing.T) {
env := newTestEnv(t)
env.createUser("recorder", "recorder-pass", "recorder")
client, _ := env.client()
env.login(client, "recorder", "recorder-pass")
env.do(client, http.MethodPost, "/api/display/command", map[string]any{
"action": "announce",
"payload": map[string]any{"text": "公告一", "seconds": 45},
}, true)
if status, env2 := env.do(client, http.MethodPost, "/api/display/command", map[string]any{
"action": "lock", "payload": map[string]any{},
}, true); status != http.StatusOK {
t.Fatalf("锁屏指令应 200,得到 %d %+v", status, env2.Error)
}
if got := settingValue(t, env, "display.announce_text"); got != "公告一" {
t.Fatalf("非公告指令不应改公告正文,得到 %q", got)
}
env.do(client, http.MethodPost, "/api/display/command", map[string]any{"action": "announce_clear"}, true)
if got := settingValue(t, env, "display.announce_seconds"); got != "45" {
t.Fatalf("清空公告应保留全屏时长,得到 %q", got)
}
if got := changeLogCount(t, env, "settings", "upsert"); got != 2 {
t.Fatalf("推送、清空各应写 1 条 settings 变更、锁屏不写,共 2 条,得到 %d", got)
}
}