be4db0bf96
C1（关键）：2026-09-22-03 的街拍实体键索引改用最终表达式 (city, year, COALESCE(title, ''))，与体检及 StreetSnapEntityState 口径一致。 此前索引按 (city, year) 比体检更窄，同城同年多专题的合法数据会通过体检、 再由索引抛原生 23505；04 退化为幂等兜底并在注释说明真正常态由 03 承载。 新增 db/migrations/README.md，写清迁移链执行顺序与各步前提/不可逆点。 I1：processRunway 的 image_count 改为按实际落库主图行数（countMainImages）， 不再用 len(p.Looks)（空主图 / sha1 重复会被跳过，导致计数偏大且不再自愈）。 补 service 单测与「入库后 image_count == 存活主图行数」的 DB 断言。 I2：三个迁移文件不再把已删除的一次性搬迁脚本写成硬前置，改为写明取回方式 （git show c4bafc5:scripts/migrate_single_table/main.go，并须在旧代码树上运行）。 I3：新增 SetRecordStatus / SetStreetRecordStatus 仓储集成测试： pending 经仓储通过后在公开视图可见、驳回后不可见、不存在的 id 返回 ErrNotFound。 Minor：修正锁不住口径的走秀 image_count 测试（改为删主图、留细节图）； 修正去重注释与事实不符（含 FindNearDuplicateImage 注释）； ListRecords / ListStreetRecords 改用 Scope 杜绝 Count 后复用 *gorm.DB； dbtool 视图改 CREATE OR REPLACE 并重跑 dump（仍 4 视图、无草稿表）； 删除挂在 Popular 上的「IDs 返回…」注释；README 改为单表 + status 现状； SetRecordStatus 注释写明有意不校验前置状态；规格补两条已知不一致。
104 lines
3.9 KiB
Go
104 lines
3.9 KiB
Go
//go:build integration
|
||
|
||
// 集成测试：「通过 → 公开可见」这条单表发布主链路的**仓储级**证据。
|
||
//
|
||
// 背景：公开视图测试（TestPublicViewsHideUnpublished*）都是直接 `db.Model(...).Update("status", ...)`
|
||
// 改库、绕过了 `Approve` 走的 SetRecordStatus / SetStreetRecordStatus，这两个仓储方法在全仓零直接引用。
|
||
// 一旦它们改错列 / 漏写 status，那些测试仍会全绿、线上却无法通过审核。本文件补上这一段。
|
||
//
|
||
// 运行：go test -tags integration ./internal/repository/ -run TestSetRecordStatus -v
|
||
package repository
|
||
|
||
import (
|
||
"context"
|
||
"errors"
|
||
"testing"
|
||
|
||
"fashionapi/internal/model"
|
||
)
|
||
|
||
// TestSetRecordStatusDrivesPublicVisibility 走秀：pending 经仓储置 published 后在公开视图可见；
|
||
// 再置 rejected 后不可见；对不存在的 id 返回 ErrNotFound。
|
||
func TestSetRecordStatusDrivesPublicVisibility(t *testing.T) {
|
||
db := testDB(t)
|
||
applyMigration(t, db, "2026-09-22-01-single-table-publish.sql")
|
||
repo := NewReviewRepository(db)
|
||
ctx := context.Background()
|
||
|
||
id := seedRunway(t, db, model.StatusPending, 1991)
|
||
publicCount := func() int64 {
|
||
var n int64
|
||
db.Table("public_brand_runways").Where("id = ?", id).Count(&n)
|
||
return n
|
||
}
|
||
|
||
if got := publicCount(); got != 0 {
|
||
t.Fatalf("pending 不应出现在公开视图，实际 %d", got)
|
||
}
|
||
|
||
if err := repo.SetRecordStatus(ctx, id, model.StatusPublished, "admin", ""); err != nil {
|
||
t.Fatalf("SetRecordStatus(published) 出错: %v", err)
|
||
}
|
||
if got := publicCount(); got != 1 {
|
||
t.Fatalf("经仓储通过后应出现在公开视图，实际 %d（仓储可能写错列）", got)
|
||
}
|
||
|
||
if err := repo.SetRecordStatus(ctx, id, model.StatusRejected, "admin", "图片不对"); err != nil {
|
||
t.Fatalf("SetRecordStatus(rejected) 出错: %v", err)
|
||
}
|
||
if got := publicCount(); got != 0 {
|
||
t.Fatalf("经仓储驳回后不应出现在公开视图，实际 %d", got)
|
||
}
|
||
|
||
// 审核痕迹应落库，便于列表页展示。
|
||
var rw model.BrandRunway
|
||
if err := db.Where("id = ?", id).First(&rw).Error; err != nil {
|
||
t.Fatalf("读回记录失败: %v", err)
|
||
}
|
||
if rw.Status != model.StatusRejected || rw.Reviewer != "admin" || rw.RejectReason != "图片不对" {
|
||
t.Fatalf("审核态与痕迹应落库，实际 status=%s reviewer=%s reason=%s", rw.Status, rw.Reviewer, rw.RejectReason)
|
||
}
|
||
|
||
// 不存在的 id：必须返回 ErrNotFound，而不是假成功。
|
||
if err := repo.SetRecordStatus(ctx, 4_294_967_290, model.StatusPublished, "admin", ""); !errors.Is(err, ErrNotFound) {
|
||
t.Fatalf("对不存在 id 应返回 ErrNotFound，实际 %v", err)
|
||
}
|
||
}
|
||
|
||
// TestSetStreetRecordStatusDrivesPublicVisibility 街拍：与走秀同形，覆盖另一组方法与视图。
|
||
func TestSetStreetRecordStatusDrivesPublicVisibility(t *testing.T) {
|
||
db := testDB(t)
|
||
applyMigration(t, db, "2026-09-22-01-single-table-publish.sql")
|
||
repo := NewReviewRepository(db)
|
||
ctx := context.Background()
|
||
|
||
id := seedStreetSnap(t, db, model.StatusPending, 1990)
|
||
publicCount := func() int64 {
|
||
var n int64
|
||
db.Table("public_street_snaps").Where("id = ?", id).Count(&n)
|
||
return n
|
||
}
|
||
|
||
if got := publicCount(); got != 0 {
|
||
t.Fatalf("pending 不应出现在公开视图，实际 %d", got)
|
||
}
|
||
|
||
if err := repo.SetStreetRecordStatus(ctx, id, model.StatusPublished, "admin", ""); err != nil {
|
||
t.Fatalf("SetStreetRecordStatus(published) 出错: %v", err)
|
||
}
|
||
if got := publicCount(); got != 1 {
|
||
t.Fatalf("经仓储通过后应出现在公开视图，实际 %d（仓储可能写错列）", got)
|
||
}
|
||
|
||
if err := repo.SetStreetRecordStatus(ctx, id, model.StatusRejected, "admin", "重复"); err != nil {
|
||
t.Fatalf("SetStreetRecordStatus(rejected) 出错: %v", err)
|
||
}
|
||
if got := publicCount(); got != 0 {
|
||
t.Fatalf("经仓储驳回后不应出现在公开视图，实际 %d", got)
|
||
}
|
||
|
||
if err := repo.SetStreetRecordStatus(ctx, 4_294_967_290, model.StatusPublished, "admin", ""); !errors.Is(err, ErrNotFound) {
|
||
t.Fatalf("对不存在 id 应返回 ErrNotFound，实际 %v", err)
|
||
}
|
||
}
|