syncPermsDedup_audit_test.go 2.9 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172
  1. package pub
  2. import (
  3. "context"
  4. "testing"
  5. permModel "perms-system-server/internal/model/perm"
  6. productModel "perms-system-server/internal/model/product"
  7. "perms-system-server/internal/testutil/mocks"
  8. "github.com/stretchr/testify/assert"
  9. "github.com/stretchr/testify/require"
  10. "github.com/zeromicro/go-zero/core/stores/sqlx"
  11. "go.uber.org/mock/gomock"
  12. "golang.org/x/crypto/bcrypt"
  13. )
  14. // TC-0826: 请求体内 perm.code 重复时,service 必须先在入参上去重,避免同一个 tx 内
  15. // BatchInsert 自己和自己撞 UNIQUE(productCode, code) 引发 1062。
  16. // 旧文件 syncPermsConflict_audit_test.go 一并覆盖了 1062→409 的映射契约,但 H-3 引入
  17. // LockByCodeTx 串行化同步后,1062 在实践中已不可达,该 409 映射契约也被一起取消;只有
  18. // 这一条"请求内去重"仍然是当前产品契约,必须继续回归。
  19. func TestExecuteSyncPerms_DeduplicatesRequest(t *testing.T) {
  20. ctrl := gomock.NewController(t)
  21. t.Cleanup(ctrl.Finish)
  22. hashedSecret, err := bcrypt.GenerateFromPassword([]byte("s"), bcrypt.MinCost)
  23. require.NoError(t, err)
  24. mockProduct := mocks.NewMockSysProductModel(ctrl)
  25. mockProduct.EXPECT().FindOneByAppKey(gomock.Any(), "ak").
  26. Return(&productModel.SysProduct{
  27. Id: 1, Code: "pc_dedup", AppKey: "ak", AppSecret: string(hashedSecret), Status: 1,
  28. }, nil)
  29. mockProduct.EXPECT().LockByCodeTx(gomock.Any(), gomock.Any(), "pc_dedup").
  30. Return(&productModel.SysProduct{Id: 1, Code: "pc_dedup"}, nil)
  31. mockPerm := mocks.NewMockSysPermModel(ctrl)
  32. mockPerm.EXPECT().FindMapByProductCodeWithTx(gomock.Any(), gomock.Any(), "pc_dedup").
  33. Return(map[string]*permModel.SysPerm{}, nil)
  34. var captured []*permModel.SysPerm
  35. mockPerm.EXPECT().TransactCtx(gomock.Any(), gomock.Any()).
  36. DoAndReturn(func(ctx context.Context, fn func(context.Context, sqlx.Session) error) error {
  37. return fn(ctx, nil)
  38. })
  39. mockPerm.EXPECT().BatchInsertWithTx(gomock.Any(), nil, gomock.Any()).
  40. DoAndReturn(func(ctx context.Context, s sqlx.Session, items []*permModel.SysPerm) error {
  41. captured = items
  42. return nil
  43. })
  44. // 去重后 codes 只剩一个,DisableNotInCodesWithTx 用去重后的集合做 NOT IN。
  45. mockPerm.EXPECT().DisableNotInCodesWithTx(gomock.Any(), nil, "pc_dedup", []string{"dup_code"}, gomock.Any()).
  46. Return(int64(0), nil)
  47. svcCtx := mocks.NewMockServiceContext(mocks.MockModels{
  48. Product: mockProduct, Perm: mockPerm,
  49. })
  50. result, err := ExecuteSyncPerms(context.Background(), svcCtx, "ak", "s", []SyncPermItem{
  51. {Code: "dup_code", Name: "A"},
  52. {Code: "dup_code", Name: "A-again"},
  53. {Code: "dup_code", Name: "A-yet-again"},
  54. })
  55. require.NoError(t, err)
  56. require.NotNil(t, result)
  57. require.Len(t, captured, 1, "入参内 code 重复必须去重为 1 条,避免自撞 1062")
  58. assert.Equal(t, "dup_code", captured[0].Code)
  59. assert.Equal(t, "A", captured[0].Name,
  60. "去重策略应稳定到首次出现,使行为可预测")
  61. }