bindRolesLogic_test.go 29 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493494495496497498499500501502503504505506507508509510511512513514515516517518519520521522523524525526527528529530531532533534535536537538539540541542543544545546547548549550551552553554555556557558559560561562563564565566567568569570571572573574575576577578579580581582583584585586587588589590591592593594595596597598599600601602603604605606607608609610611612613614615616617618619620621622623624625626627628629630631632633634635636637638639640641642643644645646647648649650651652653654655656657658659660661662663664665666667668669670671672673674675676677678679680681682683684685686687688689690691692693694695696697698699700701702703704705706707708709710711712713714715716717718719720721722723724725726727728729730731732733734735736737738739740741742743744745746747748749750751752753754755756757758759760761762763764765766767768769770771772773774775776777778779780781782783784785786787788789790791792793794795796797798799800801802803804805806807808809810811812813814815816817818819820821
  1. package user
  2. import (
  3. "context"
  4. "errors"
  5. "fmt"
  6. "github.com/stretchr/testify/assert"
  7. "github.com/stretchr/testify/require"
  8. "math"
  9. "perms-system-server/internal/consts"
  10. "perms-system-server/internal/loaders"
  11. roleLogic "perms-system-server/internal/logic/role"
  12. deptModel "perms-system-server/internal/model/dept"
  13. memberModel "perms-system-server/internal/model/productmember"
  14. roleModel "perms-system-server/internal/model/role"
  15. userModel "perms-system-server/internal/model/user"
  16. userroleModel "perms-system-server/internal/model/userrole"
  17. "perms-system-server/internal/response"
  18. "perms-system-server/internal/svc"
  19. "perms-system-server/internal/testutil"
  20. "perms-system-server/internal/testutil/ctxhelper"
  21. "perms-system-server/internal/types"
  22. "sync"
  23. "sync/atomic"
  24. "testing"
  25. "time"
  26. )
  27. func insertTestMember(t *testing.T, svcCtx *svc.ServiceContext, productCode string, userId int64) int64 {
  28. t.Helper()
  29. now := time.Now().Unix()
  30. res, err := svcCtx.SysProductMemberModel.Insert(ctxhelper.SuperAdminCtx(), &memberModel.SysProductMember{
  31. ProductCode: productCode,
  32. UserId: userId,
  33. MemberType: "MEMBER",
  34. Status: 1,
  35. CreateTime: now,
  36. UpdateTime: now,
  37. })
  38. require.NoError(t, err)
  39. id, _ := res.LastInsertId()
  40. return id
  41. }
  42. func insertTestRole(t *testing.T, svcCtx *svc.ServiceContext, productCode string, status int64) int64 {
  43. t.Helper()
  44. now := time.Now().Unix()
  45. res, err := svcCtx.SysRoleModel.Insert(ctxhelper.SuperAdminCtx(), &roleModel.SysRole{
  46. ProductCode: productCode,
  47. Name: "role_" + testutil.UniqueId(),
  48. Status: status,
  49. PermsLevel: 1,
  50. CreateTime: now,
  51. UpdateTime: now,
  52. })
  53. require.NoError(t, err)
  54. id, _ := res.LastInsertId()
  55. return id
  56. }
  57. // TC-0184: 正常绑定
  58. func TestBindRoles_Success(t *testing.T) {
  59. ctx := ctxhelper.SuperAdminCtx()
  60. svcCtx := svc.NewServiceContext(testutil.GetTestConfig())
  61. conn := testutil.GetTestSqlConn()
  62. username := testutil.UniqueId()
  63. userId := insertTestUser(t, ctx, username, testutil.HashPassword("pass"))
  64. mId := insertTestMember(t, svcCtx, "test_product", userId)
  65. r1 := insertTestRole(t, svcCtx, "test_product", 1)
  66. r2 := insertTestRole(t, svcCtx, "test_product", 1)
  67. t.Cleanup(func() {
  68. testutil.CleanTableByField(ctx, conn, "`sys_user_role`", "userId", userId)
  69. testutil.CleanTable(ctx, conn, "`sys_product_member`", mId)
  70. testutil.CleanTable(ctx, conn, "`sys_user`", userId)
  71. testutil.CleanTable(ctx, conn, "`sys_role`", r1, r2)
  72. })
  73. logic := NewBindRolesLogic(ctx, svcCtx)
  74. err := logic.BindRoles(&types.BindRolesReq{
  75. UserId: userId,
  76. RoleIds: []int64{r1, r2},
  77. })
  78. require.NoError(t, err)
  79. roleIds, err := svcCtx.SysUserRoleModel.FindRoleIdsByUserId(ctx, userId)
  80. require.NoError(t, err)
  81. assert.ElementsMatch(t, []int64{r1, r2}, roleIds)
  82. }
  83. // TC-0185: 用户不存在
  84. func TestBindRoles_UserNotFound(t *testing.T) {
  85. ctx := ctxhelper.SuperAdminCtx()
  86. svcCtx := svc.NewServiceContext(testutil.GetTestConfig())
  87. logic := NewBindRolesLogic(ctx, svcCtx)
  88. err := logic.BindRoles(&types.BindRolesReq{
  89. UserId: 999999999,
  90. RoleIds: []int64{1},
  91. })
  92. require.Error(t, err)
  93. var codeErr *response.CodeError
  94. require.True(t, errors.As(err, &codeErr))
  95. assert.Equal(t, 404, codeErr.Code())
  96. assert.Equal(t, "用户不存在", codeErr.Error())
  97. }
  98. // TC-0186: 清空角色
  99. func TestBindRoles_EmptyRoleIds_ClearsAll(t *testing.T) {
  100. ctx := ctxhelper.SuperAdminCtx()
  101. svcCtx := svc.NewServiceContext(testutil.GetTestConfig())
  102. conn := testutil.GetTestSqlConn()
  103. username := testutil.UniqueId()
  104. userId := insertTestUser(t, ctx, username, testutil.HashPassword("pass"))
  105. mId := insertTestMember(t, svcCtx, "test_product", userId)
  106. r1 := insertTestRole(t, svcCtx, "test_product", 1)
  107. t.Cleanup(func() {
  108. testutil.CleanTableByField(ctx, conn, "`sys_user_role`", "userId", userId)
  109. testutil.CleanTable(ctx, conn, "`sys_product_member`", mId)
  110. testutil.CleanTable(ctx, conn, "`sys_user`", userId)
  111. testutil.CleanTable(ctx, conn, "`sys_role`", r1)
  112. })
  113. logic := NewBindRolesLogic(ctx, svcCtx)
  114. err := logic.BindRoles(&types.BindRolesReq{
  115. UserId: userId,
  116. RoleIds: []int64{r1},
  117. })
  118. require.NoError(t, err)
  119. err = logic.BindRoles(&types.BindRolesReq{
  120. UserId: userId,
  121. RoleIds: []int64{},
  122. })
  123. require.NoError(t, err)
  124. roleIds, err := svcCtx.SysUserRoleModel.FindRoleIdsByUserId(ctx, userId)
  125. require.NoError(t, err)
  126. assert.Empty(t, roleIds)
  127. }
  128. // TC-0184: 正常重新绑定
  129. func TestBindRoles_Rebind(t *testing.T) {
  130. ctx := ctxhelper.SuperAdminCtx()
  131. svcCtx := svc.NewServiceContext(testutil.GetTestConfig())
  132. conn := testutil.GetTestSqlConn()
  133. username := testutil.UniqueId()
  134. userId := insertTestUser(t, ctx, username, testutil.HashPassword("pass"))
  135. mId := insertTestMember(t, svcCtx, "test_product", userId)
  136. r1 := insertTestRole(t, svcCtx, "test_product", 1)
  137. r2 := insertTestRole(t, svcCtx, "test_product", 1)
  138. r3 := insertTestRole(t, svcCtx, "test_product", 1)
  139. t.Cleanup(func() {
  140. testutil.CleanTableByField(ctx, conn, "`sys_user_role`", "userId", userId)
  141. testutil.CleanTable(ctx, conn, "`sys_product_member`", mId)
  142. testutil.CleanTable(ctx, conn, "`sys_user`", userId)
  143. testutil.CleanTable(ctx, conn, "`sys_role`", r1, r2, r3)
  144. })
  145. logic := NewBindRolesLogic(ctx, svcCtx)
  146. err := logic.BindRoles(&types.BindRolesReq{
  147. UserId: userId,
  148. RoleIds: []int64{r1, r2},
  149. })
  150. require.NoError(t, err)
  151. err = logic.BindRoles(&types.BindRolesReq{
  152. UserId: userId,
  153. RoleIds: []int64{r2, r3},
  154. })
  155. require.NoError(t, err)
  156. roleIds, err := svcCtx.SysUserRoleModel.FindRoleIdsByUserId(ctx, userId)
  157. require.NoError(t, err)
  158. assert.ElementsMatch(t, []int64{r2, r3}, roleIds)
  159. }
  160. // TC-0188: 角色不属于当前产品
  161. func TestBindRoles_RoleBelongsToOtherProduct(t *testing.T) {
  162. ctx := ctxhelper.SuperAdminCtx()
  163. svcCtx := svc.NewServiceContext(testutil.GetTestConfig())
  164. conn := testutil.GetTestSqlConn()
  165. username := testutil.UniqueId()
  166. userId := insertTestUser(t, ctx, username, testutil.HashPassword("pass"))
  167. mId := insertTestMember(t, svcCtx, "test_product", userId)
  168. otherRole := insertTestRole(t, svcCtx, "other_product", 1)
  169. t.Cleanup(func() {
  170. testutil.CleanTable(ctx, conn, "`sys_product_member`", mId)
  171. testutil.CleanTable(ctx, conn, "`sys_user`", userId)
  172. testutil.CleanTable(ctx, conn, "`sys_role`", otherRole)
  173. })
  174. logic := NewBindRolesLogic(ctx, svcCtx)
  175. err := logic.BindRoles(&types.BindRolesReq{
  176. UserId: userId,
  177. RoleIds: []int64{otherRole},
  178. })
  179. require.Error(t, err)
  180. var codeErr *response.CodeError
  181. require.True(t, errors.As(err, &codeErr))
  182. assert.Equal(t, 400, codeErr.Code())
  183. assert.Contains(t, codeErr.Error(), "其他产品的角色")
  184. }
  185. // TC-0189: 角色已禁用
  186. func TestBindRoles_RoleDisabled(t *testing.T) {
  187. ctx := ctxhelper.SuperAdminCtx()
  188. svcCtx := svc.NewServiceContext(testutil.GetTestConfig())
  189. conn := testutil.GetTestSqlConn()
  190. username := testutil.UniqueId()
  191. userId := insertTestUser(t, ctx, username, testutil.HashPassword("pass"))
  192. mId := insertTestMember(t, svcCtx, "test_product", userId)
  193. disabledRole := insertTestRole(t, svcCtx, "test_product", 2)
  194. t.Cleanup(func() {
  195. testutil.CleanTable(ctx, conn, "`sys_product_member`", mId)
  196. testutil.CleanTable(ctx, conn, "`sys_user`", userId)
  197. testutil.CleanTable(ctx, conn, "`sys_role`", disabledRole)
  198. })
  199. logic := NewBindRolesLogic(ctx, svcCtx)
  200. err := logic.BindRoles(&types.BindRolesReq{
  201. UserId: userId,
  202. RoleIds: []int64{disabledRole},
  203. })
  204. require.Error(t, err)
  205. var codeErr *response.CodeError
  206. require.True(t, errors.As(err, &codeErr))
  207. assert.Equal(t, 400, codeErr.Code())
  208. assert.Contains(t, codeErr.Error(), "已禁用的角色")
  209. }
  210. // TC-0190: 角色不存在
  211. func TestBindRoles_RoleNotExists(t *testing.T) {
  212. ctx := ctxhelper.SuperAdminCtx()
  213. svcCtx := svc.NewServiceContext(testutil.GetTestConfig())
  214. conn := testutil.GetTestSqlConn()
  215. username := testutil.UniqueId()
  216. userId := insertTestUser(t, ctx, username, testutil.HashPassword("pass"))
  217. mId := insertTestMember(t, svcCtx, "test_product", userId)
  218. t.Cleanup(func() {
  219. testutil.CleanTable(ctx, conn, "`sys_product_member`", mId)
  220. testutil.CleanTable(ctx, conn, "`sys_user`", userId)
  221. })
  222. logic := NewBindRolesLogic(ctx, svcCtx)
  223. err := logic.BindRoles(&types.BindRolesReq{
  224. UserId: userId,
  225. RoleIds: []int64{999999999},
  226. })
  227. require.Error(t, err)
  228. var codeErr *response.CodeError
  229. require.True(t, errors.As(err, &codeErr))
  230. assert.Equal(t, 400, codeErr.Code())
  231. assert.Contains(t, codeErr.Error(), "无效的角色ID")
  232. }
  233. func insertTestRoleWithLevel(t *testing.T, svcCtx *svc.ServiceContext, productCode string, status int64, permsLevel int64) int64 {
  234. t.Helper()
  235. now := time.Now().Unix()
  236. res, err := svcCtx.SysRoleModel.Insert(ctxhelper.SuperAdminCtx(), &roleModel.SysRole{
  237. ProductCode: productCode,
  238. Name: "role_" + testutil.UniqueId(),
  239. Status: status,
  240. PermsLevel: permsLevel,
  241. CreateTime: now,
  242. UpdateTime: now,
  243. })
  244. require.NoError(t, err)
  245. id, _ := res.LastInsertId()
  246. return id
  247. }
  248. // setupDeptForCaller 插入一个 dept,同时构造 caller(使用该 dept)与 target(同 dept 下)的环境
  249. // 返回 deptId、deptPath(caller 使用)、cleanup function
  250. func setupDeptForCaller(t *testing.T, svcCtx *svc.ServiceContext) (int64, string, func()) {
  251. t.Helper()
  252. now := time.Now().Unix()
  253. superCtx := ctxhelper.SuperAdminCtx()
  254. res, err := svcCtx.SysDeptModel.Insert(superCtx, &deptModel.SysDept{
  255. Name: "dept_" + testutil.UniqueId(),
  256. ParentId: 0,
  257. Path: "/",
  258. DeptType: consts.DeptTypeNormal,
  259. Status: consts.StatusEnabled,
  260. CreateTime: now,
  261. UpdateTime: now,
  262. })
  263. require.NoError(t, err)
  264. deptId, _ := res.LastInsertId()
  265. // 先占位再用真实 deptId 构造 path:"/{deptId}/"
  266. path := fmt.Sprintf("/%d/", deptId)
  267. _, err = svcCtx.SysDeptModel.Insert(superCtx, &deptModel.SysDept{}) // noop — keep linter happy
  268. _ = err
  269. // 更新 path
  270. dept, _ := svcCtx.SysDeptModel.FindOne(superCtx, deptId)
  271. dept.Path = path
  272. dept.UpdateTime = time.Now().Unix()
  273. require.NoError(t, svcCtx.SysDeptModel.Update(superCtx, dept))
  274. conn := testutil.GetTestSqlConn()
  275. cleanup := func() {
  276. testutil.CleanTable(superCtx, conn, "`sys_dept`", deptId)
  277. }
  278. return deptId, path, cleanup
  279. }
  280. // TC-0208: MEMBER 调用者不能分配权限级别高于自身的角色 (audit 修复后 permsLevel 仅对 MEMBER 生效)
  281. func TestBindRoles_PermsLevelEscalation_Rejected(t *testing.T) {
  282. svcCtx := svc.NewServiceContext(testutil.GetTestConfig())
  283. conn := testutil.GetTestSqlConn()
  284. superCtx := ctxhelper.SuperAdminCtx()
  285. deptId, deptPath, cleanupDept := setupDeptForCaller(t, svcCtx)
  286. t.Cleanup(cleanupDept)
  287. productCode := "test_product"
  288. // 目标用户:放进 dept 下,MEMBER 产品成员
  289. username := testutil.UniqueId()
  290. targetUserId := insertTestUserFull(t, superCtx, &userModel.SysUser{
  291. Username: username, Password: testutil.HashPassword("pass"),
  292. Nickname: "tgt", DeptId: deptId,
  293. IsSuperAdmin: 2, MustChangePassword: 2, Status: 1,
  294. })
  295. mId := insertTestMember(t, svcCtx, productCode, targetUserId)
  296. highLevelRole := insertTestRoleWithLevel(t, svcCtx, productCode, 1, 1)
  297. // 修复后 GuardRoleLevelAssignable 走 DB 强一致读取 caller 的 MinPermsLevel,
  298. // 因此需要在 DB 里为调用者落地真实的 user + role + user_role 关系链(permsLevel=50)。
  299. callerUserId, callerCleanup := seedCallerWithRoleLevel(t, svcCtx, productCode, 50)
  300. t.Cleanup(callerCleanup)
  301. t.Cleanup(func() {
  302. testutil.CleanTableByField(superCtx, conn, "`sys_user_role`", "userId", targetUserId)
  303. testutil.CleanTable(superCtx, conn, "`sys_product_member`", mId)
  304. testutil.CleanTable(superCtx, conn, "`sys_user`", targetUserId)
  305. testutil.CleanTable(superCtx, conn, "`sys_role`", highLevelRole)
  306. })
  307. // MEMBER 调用者与 target 同 dept,DB 中 MinPermsLevel=50,目标角色 permsLevel=1 → 越级
  308. ctx := ctxhelper.CustomCtx(&loaders.UserDetails{
  309. UserId: callerUserId,
  310. Username: "member_caller",
  311. IsSuperAdmin: false,
  312. MemberType: consts.MemberTypeMember,
  313. Status: consts.StatusEnabled,
  314. ProductCode: productCode,
  315. DeptId: deptId,
  316. DeptPath: deptPath,
  317. MinPermsLevel: 50,
  318. })
  319. logic := NewBindRolesLogic(ctx, svcCtx)
  320. err := logic.BindRoles(&types.BindRolesReq{
  321. UserId: targetUserId,
  322. RoleIds: []int64{highLevelRole},
  323. })
  324. require.Error(t, err)
  325. var ce *response.CodeError
  326. require.True(t, errors.As(err, &ce))
  327. assert.Equal(t, 403, ce.Code())
  328. assert.Contains(t, ce.Error(), "不能分配权限级别高于自身的角色")
  329. }
  330. // TC-0711: ADMIN 调用者(MinPermsLevel=math.MaxInt64)不受 permsLevel 校验约束 (audit 回归)
  331. // 修复前:ADMIN 通过 member_type 获得权限,MinPermsLevel 保持 math.MaxInt64,
  332. //
  333. // r.PermsLevel < math.MaxInt64 必然成立 → ADMIN 无法绑定任何角色。
  334. //
  335. // 修复后:代码显式豁免 ADMIN/DEVELOPER 的 permsLevel 校验。
  336. func TestBindRoles_AdminBypassesPermsLevelCheck(t *testing.T) {
  337. svcCtx := svc.NewServiceContext(testutil.GetTestConfig())
  338. conn := testutil.GetTestSqlConn()
  339. superCtx := ctxhelper.SuperAdminCtx()
  340. productCode := "test_product"
  341. username := testutil.UniqueId()
  342. userId := insertTestUser(t, superCtx, username, testutil.HashPassword("pass"))
  343. mId := insertTestMember(t, svcCtx, productCode, userId)
  344. lowLevelRole := insertTestRoleWithLevel(t, svcCtx, productCode, 1, 1)
  345. t.Cleanup(func() {
  346. testutil.CleanTableByField(superCtx, conn, "`sys_user_role`", "userId", userId)
  347. testutil.CleanTable(superCtx, conn, "`sys_product_member`", mId)
  348. testutil.CleanTable(superCtx, conn, "`sys_user`", userId)
  349. testutil.CleanTable(superCtx, conn, "`sys_role`", lowLevelRole)
  350. })
  351. // 关键:模拟 loader 真实产出——ADMIN 没有自定义角色,MinPermsLevel=math.MaxInt64
  352. ctx := ctxhelper.CustomCtx(&loaders.UserDetails{
  353. UserId: 999997,
  354. Username: "admin_caller",
  355. IsSuperAdmin: false,
  356. MemberType: consts.MemberTypeAdmin,
  357. Status: consts.StatusEnabled,
  358. ProductCode: productCode,
  359. DeptId: 1,
  360. DeptPath: "/1/",
  361. MinPermsLevel: math.MaxInt64, // 默认 sentinel
  362. })
  363. logic := NewBindRolesLogic(ctx, svcCtx)
  364. err := logic.BindRoles(&types.BindRolesReq{
  365. UserId: userId,
  366. RoleIds: []int64{lowLevelRole},
  367. })
  368. require.NoError(t, err, "ADMIN 调用者应当能绑定任意级别的角色")
  369. roleIds, err := svcCtx.SysUserRoleModel.FindRoleIdsByUserId(ctx, userId)
  370. require.NoError(t, err)
  371. assert.Contains(t, roleIds, lowLevelRole)
  372. }
  373. // TC-0712: DEVELOPER 调用者同样不受 permsLevel 校验约束 (audit 回归)
  374. func TestBindRoles_DeveloperBypassesPermsLevelCheck(t *testing.T) {
  375. svcCtx := svc.NewServiceContext(testutil.GetTestConfig())
  376. conn := testutil.GetTestSqlConn()
  377. superCtx := ctxhelper.SuperAdminCtx()
  378. deptId, deptPath, cleanupDept := setupDeptForCaller(t, svcCtx)
  379. t.Cleanup(cleanupDept)
  380. productCode := "test_product"
  381. username := testutil.UniqueId()
  382. userId := insertTestUserFull(t, superCtx, &userModel.SysUser{
  383. Username: username, Password: testutil.HashPassword("pass"),
  384. Nickname: "tgt_dev", DeptId: deptId,
  385. IsSuperAdmin: 2, MustChangePassword: 2, Status: 1,
  386. })
  387. mId := insertTestMember(t, svcCtx, productCode, userId)
  388. lowLevelRole := insertTestRoleWithLevel(t, svcCtx, productCode, 1, 1)
  389. t.Cleanup(func() {
  390. testutil.CleanTableByField(superCtx, conn, "`sys_user_role`", "userId", userId)
  391. testutil.CleanTable(superCtx, conn, "`sys_product_member`", mId)
  392. testutil.CleanTable(superCtx, conn, "`sys_user`", userId)
  393. testutil.CleanTable(superCtx, conn, "`sys_role`", lowLevelRole)
  394. })
  395. ctx := ctxhelper.CustomCtx(&loaders.UserDetails{
  396. UserId: 999996,
  397. Username: "developer_caller",
  398. IsSuperAdmin: false,
  399. MemberType: consts.MemberTypeDeveloper,
  400. Status: consts.StatusEnabled,
  401. ProductCode: productCode,
  402. DeptId: deptId,
  403. DeptPath: deptPath,
  404. MinPermsLevel: math.MaxInt64,
  405. })
  406. logic := NewBindRolesLogic(ctx, svcCtx)
  407. err := logic.BindRoles(&types.BindRolesReq{
  408. UserId: userId,
  409. RoleIds: []int64{lowLevelRole},
  410. })
  411. require.NoError(t, err, "DEVELOPER 调用者应当能绑定任意级别的角色")
  412. }
  413. // TC-0713: MinPermsLevel == math.MaxInt64 的 MEMBER 调用者也必须被豁免
  414. // (sentinel 判定路径:既不是 ADMIN/DEVELOPER,也没有角色,此时 r.PermsLevel<MaxInt64 的逐字面比较
  415. //
  416. // 曾经误伤此类 MEMBER;修复后代码用 MinPermsLevel==MaxInt64 做短路)
  417. func TestBindRoles_MemberWithSentinelMinLevel_NotBlocked(t *testing.T) {
  418. svcCtx := svc.NewServiceContext(testutil.GetTestConfig())
  419. conn := testutil.GetTestSqlConn()
  420. superCtx := ctxhelper.SuperAdminCtx()
  421. productCode := "test_product"
  422. username := testutil.UniqueId()
  423. userId := insertTestUser(t, superCtx, username, testutil.HashPassword("pass"))
  424. mId := insertTestMember(t, svcCtx, productCode, userId)
  425. role := insertTestRoleWithLevel(t, svcCtx, productCode, 1, 100)
  426. t.Cleanup(func() {
  427. testutil.CleanTableByField(superCtx, conn, "`sys_user_role`", "userId", userId)
  428. testutil.CleanTable(superCtx, conn, "`sys_product_member`", mId)
  429. testutil.CleanTable(superCtx, conn, "`sys_user`", userId)
  430. testutil.CleanTable(superCtx, conn, "`sys_role`", role)
  431. })
  432. // MEMBER 调用者没有绑定任何启用角色,MinPermsLevel=MaxInt64(sentinel)
  433. // 正式语义:"我自己无级别" → 不应触发越级校验(否则所有无角色 MEMBER 都永远无法分配角色)
  434. ctx := ctxhelper.CustomCtx(&loaders.UserDetails{
  435. UserId: 999995,
  436. Username: "member_no_role",
  437. IsSuperAdmin: false,
  438. MemberType: consts.MemberTypeMember,
  439. Status: consts.StatusEnabled,
  440. ProductCode: productCode,
  441. DeptId: 1,
  442. DeptPath: "/1/",
  443. MinPermsLevel: math.MaxInt64,
  444. })
  445. logic := NewBindRolesLogic(ctx, svcCtx)
  446. // 注意:业务层早期就会用 `RequireProductAdminFor` 拦住非 ADMIN/SUPER 的调用;此处是为了单独验证
  447. // bindRoles 内部的 permsLevel 分支。实际发生于 ADMIN 通过上层校验但 MemberType 上下文异常时的防御。
  448. // 这里只断言:"sentinel 路径不应报 403 '不能分配权限级别高于自身的角色'"。
  449. err := logic.BindRoles(&types.BindRolesReq{
  450. UserId: userId,
  451. RoleIds: []int64{role},
  452. })
  453. // 调用者非 ADMIN,且是 MEMBER,上游会拦 403 "仅ADMIN/超管可绑定角色";
  454. // 此处我们只校验"即使走到 permsLevel 分支,sentinel MinPermsLevel 不应命中"
  455. if err != nil {
  456. var ce *response.CodeError
  457. require.True(t, errors.As(err, &ce))
  458. assert.NotContains(t, ce.Error(), "不能分配权限级别高于自身的角色",
  459. "sentinel MinPermsLevel=math.MaxInt64 不应触发越级错误")
  460. }
  461. }
  462. // TC-0209: 超管可以分配任意权限级别的角色
  463. func TestBindRoles_SuperAdminCanAssignAnyLevel(t *testing.T) {
  464. ctx := ctxhelper.SuperAdminCtx()
  465. svcCtx := svc.NewServiceContext(testutil.GetTestConfig())
  466. conn := testutil.GetTestSqlConn()
  467. productCode := "test_product"
  468. username := testutil.UniqueId()
  469. userId := insertTestUser(t, ctx, username, testutil.HashPassword("pass"))
  470. mId := insertTestMember(t, svcCtx, productCode, userId)
  471. highLevelRole := insertTestRoleWithLevel(t, svcCtx, productCode, 1, 1)
  472. t.Cleanup(func() {
  473. testutil.CleanTableByField(ctx, conn, "`sys_user_role`", "userId", userId)
  474. testutil.CleanTable(ctx, conn, "`sys_product_member`", mId)
  475. testutil.CleanTable(ctx, conn, "`sys_user`", userId)
  476. testutil.CleanTable(ctx, conn, "`sys_role`", highLevelRole)
  477. })
  478. logic := NewBindRolesLogic(ctx, svcCtx)
  479. err := logic.BindRoles(&types.BindRolesReq{
  480. UserId: userId,
  481. RoleIds: []int64{highLevelRole},
  482. })
  483. require.NoError(t, err)
  484. roleIds, err := svcCtx.SysUserRoleModel.FindRoleIdsByUserId(ctx, userId)
  485. require.NoError(t, err)
  486. assert.Contains(t, roleIds, highLevelRole)
  487. }
  488. // TC-0191: 目标用户不是当前产品成员时拒绝绑定角色(修复验证)
  489. func TestBindRoles_NonMemberRejected(t *testing.T) {
  490. ctx := ctxhelper.SuperAdminCtx()
  491. svcCtx := svc.NewServiceContext(testutil.GetTestConfig())
  492. conn := testutil.GetTestSqlConn()
  493. username := testutil.UniqueId()
  494. userId := insertTestUser(t, ctx, username, testutil.HashPassword("pass"))
  495. t.Cleanup(func() { testutil.CleanTable(ctx, conn, "`sys_user`", userId) })
  496. logic := NewBindRolesLogic(ctx, svcCtx)
  497. err := logic.BindRoles(&types.BindRolesReq{
  498. UserId: userId,
  499. RoleIds: []int64{},
  500. })
  501. require.Error(t, err)
  502. var codeErr2 *response.CodeError
  503. require.True(t, errors.As(err, &codeErr2))
  504. assert.Equal(t, 400, codeErr2.Code())
  505. assert.Contains(t, codeErr2.Error(), "不是当前产品的成员")
  506. }
  507. func setupBindRolesOrphanFixture(t *testing.T, svcCtx *svc.ServiceContext, productCode string) (
  508. userId, memberId, roleId int64, cleanup func(),
  509. ) {
  510. t.Helper()
  511. superCtx := ctxhelper.SuperAdminCtx()
  512. conn := testutil.GetTestSqlConn()
  513. username := testutil.UniqueId()
  514. userId = insertTestUser(t, superCtx, username, testutil.HashPassword("pass"))
  515. memberId = insertTestMember(t, svcCtx, productCode, userId)
  516. now := time.Now().Unix()
  517. res, err := svcCtx.SysRoleModel.Insert(superCtx, &roleModel.SysRole{
  518. ProductCode: productCode,
  519. Name: "r12_1_" + testutil.UniqueId(),
  520. Status: consts.StatusEnabled,
  521. PermsLevel: 10,
  522. CreateTime: now,
  523. UpdateTime: now,
  524. })
  525. require.NoError(t, err)
  526. roleId, _ = res.LastInsertId()
  527. cleanup = func() {
  528. testutil.CleanTableByField(superCtx, conn, "`sys_user_role`", "userId", userId)
  529. testutil.CleanTable(superCtx, conn, "`sys_product_member`", memberId)
  530. testutil.CleanTable(superCtx, conn, "`sys_user`", userId)
  531. testutil.CleanTable(superCtx, conn, "`sys_role`", roleId)
  532. }
  533. return
  534. }
  535. // TC-1078: BindRoles 和 DeleteRole 并发:终态无孤儿
  536. // 用真实 MySQL + go-zero 事务跑多轮并发,断言每一轮都能落在以下两个合法终态之一:
  537. //
  538. // A) BindRoles 胜出 + DeleteRole 胜出(先后串行):
  539. // - 最常见:BindRoles 先提交,sys_user_role 出现新行;DeleteRole 随后提交,
  540. // 级联 DELETE 把新行一并带走;sys_role[R] 消失、sys_user_role 无 R 行 → 无孤儿。
  541. // B) DeleteRole 先提交 + BindRoles 收 400:
  542. // - sys_role[R] 消失;BindRoles 事务内 LockRolesForShareTx 读不到 → ErrNotFound → 400;
  543. // - sys_user_role 无 R 行 → 无孤儿。
  544. //
  545. // 不允许任何一轮出现:sys_role[R] 不在 + sys_user_role 仍有 (userId, R) —— 这就是 orphan。
  546. func TestBindRoles_Vs_DeleteRole_NoOrphanRows(t *testing.T) {
  547. svcCtx := svc.NewServiceContext(testutil.GetTestConfig())
  548. conn := testutil.GetTestSqlConn()
  549. superCtx := ctxhelper.SuperAdminCtx()
  550. productCode := "test_product"
  551. const rounds = 6
  552. for round := 0; round < rounds; round++ {
  553. userId, memberId, roleId, cleanup := setupBindRolesOrphanFixture(t, svcCtx, productCode)
  554. _ = memberId
  555. var (
  556. wg sync.WaitGroup
  557. bindErr atomic.Value
  558. delErr atomic.Value
  559. bindOK atomic.Bool
  560. deleteOK atomic.Bool
  561. )
  562. start := make(chan struct{})
  563. wg.Add(2)
  564. go func() {
  565. defer wg.Done()
  566. <-start
  567. err := NewBindRolesLogic(superCtx, svcCtx).BindRoles(&types.BindRolesReq{
  568. UserId: userId,
  569. RoleIds: []int64{roleId},
  570. })
  571. if err == nil {
  572. bindOK.Store(true)
  573. } else {
  574. bindErr.Store(err)
  575. }
  576. }()
  577. go func() {
  578. defer wg.Done()
  579. <-start
  580. err := roleLogic.NewDeleteRoleLogic(superCtx, svcCtx).DeleteRole(&types.DeleteRoleReq{
  581. Id: roleId,
  582. })
  583. if err == nil {
  584. deleteOK.Store(true)
  585. } else {
  586. delErr.Store(err)
  587. }
  588. }()
  589. close(start)
  590. wg.Wait()
  591. // 终态:绕过 go-zero cache 直接查 DB,避免 cache 把 DeleteRole 的真实删除遮住造成假阳。
  592. var roleCount, urCount int64
  593. require.NoError(t, conn.QueryRowCtx(context.Background(), &roleCount,
  594. "SELECT COUNT(*) FROM `sys_role` WHERE `id` = ?", roleId))
  595. require.NoError(t, conn.QueryRowCtx(context.Background(), &urCount,
  596. "SELECT COUNT(*) FROM `sys_user_role` WHERE `userId` = ? AND `roleId` = ?", userId, roleId))
  597. // 最严 orphan 判定:sys_role 不在 且 sys_user_role 仍在 → 孤儿
  598. if roleCount == 0 && urCount > 0 {
  599. t.Fatalf(
  600. "(轮次 %d):产生 orphan —— sys_role[%d] 已被 DeleteRole 删除,"+
  601. "但 sys_user_role 仍有 (userId=%d, roleId=%d) 行。bindOK=%v delOK=%v "+
  602. "bindErr=%v delErr=%v", round, roleId, userId, roleId, bindOK.Load(),
  603. deleteOK.Load(), bindErr.Load(), delErr.Load(),
  604. )
  605. }
  606. // 其它合法终态一并回归:至少一端做了有效操作(不能都失败)
  607. if !bindOK.Load() && !deleteOK.Load() {
  608. t.Logf("轮次 %d: bindErr=%v delErr=%v", round, bindErr.Load(), delErr.Load())
  609. t.Fatalf("轮次 %d:两端都失败,不是预期的并发交错(至少 DeleteRole 应成功,"+
  610. "因为它持有 FindOne 之后所有行的独占链路)", round)
  611. }
  612. // 如果 BindRoles 报错必须是 400 "已被删除或已禁用的角色ID"(由 LockRolesForShareTx 触发)
  613. if raw := bindErr.Load(); raw != nil {
  614. var ce *response.CodeError
  615. if errors.As(raw.(error), &ce) {
  616. assert.Equal(t, 400, ce.Code(),
  617. "BindRoles 在 DeleteRole 先成功时必须 400,不得泄漏为 500")
  618. assert.Contains(t, ce.Error(), "已被删除或已禁用的角色ID",
  619. "信号必须准确,方便上游重试 / 清理入参")
  620. }
  621. }
  622. cleanup()
  623. }
  624. }
  625. func seedCallerWithRoleLevel(t *testing.T, svcCtx *svc.ServiceContext, productCode string, callerLevel int64) (int64, func()) {
  626. t.Helper()
  627. superCtx := ctxhelper.SuperAdminCtx()
  628. conn := testutil.GetTestSqlConn()
  629. callerUserId := insertTestUserFull(t, superCtx, &userModel.SysUser{
  630. Username: "caller_" + testutil.UniqueId(), Password: testutil.HashPassword("pass"),
  631. Nickname: "caller_seed", DeptId: 0,
  632. IsSuperAdmin: consts.IsSuperAdminNo, MustChangePassword: 2, Status: consts.StatusEnabled,
  633. })
  634. mId := insertTestMember(t, svcCtx, productCode, callerUserId)
  635. roleId := insertTestRoleWithLevel(t, svcCtx, productCode, consts.StatusEnabled, callerLevel)
  636. now := time.Now().Unix()
  637. _, err := svcCtx.SysUserRoleModel.Insert(superCtx, &userroleModel.SysUserRole{
  638. UserId: callerUserId,
  639. RoleId: roleId,
  640. CreateTime: now,
  641. UpdateTime: now,
  642. })
  643. require.NoError(t, err)
  644. cleanup := func() {
  645. testutil.CleanTableByField(superCtx, conn, "`sys_user_role`", "userId", callerUserId)
  646. testutil.CleanTable(superCtx, conn, "`sys_product_member`", mId)
  647. testutil.CleanTable(superCtx, conn, "`sys_user`", callerUserId)
  648. testutil.CleanTable(superCtx, conn, "`sys_role`", roleId)
  649. }
  650. return callerUserId, cleanup
  651. }
  652. // ---------------------------------------------------------------------------
  653. // 覆盖目标:"不能分配与自己同级(或更高)的角色"。
  654. // 修复前代码仅拦 `>` 严格高于,允许 MEMBER 调用者把同级角色分配给别人,继而下一次 BindRoles 时
  655. // 由于同级权限集相同,可用后续 upgrade 路径放大;修复后变为 `<=`(含同级)拦截。
  656. // 本文件作为"同级也必须 403"的契约锚点。
  657. // ---------------------------------------------------------------------------
  658. // TC-0813: MEMBER 调用者不能分配与自己同 permsLevel 的角色。
  659. func TestBindRoles_EqualPermsLevel_Rejected(t *testing.T) {
  660. svcCtx := svc.NewServiceContext(testutil.GetTestConfig())
  661. conn := testutil.GetTestSqlConn()
  662. superCtx := ctxhelper.SuperAdminCtx()
  663. deptId, deptPath, cleanupDept := setupDeptForCaller(t, svcCtx)
  664. t.Cleanup(cleanupDept)
  665. productCode := "test_product"
  666. username := testutil.UniqueId()
  667. targetUserId := insertTestUserFull(t, superCtx, &userModel.SysUser{
  668. Username: username, Password: testutil.HashPassword("pass"),
  669. Nickname: "tgt_eq", DeptId: deptId,
  670. IsSuperAdmin: consts.IsSuperAdminNo, MustChangePassword: 2, Status: consts.StatusEnabled,
  671. })
  672. mId := insertTestMember(t, svcCtx, productCode, targetUserId)
  673. const callerLevel int64 = 50
  674. sameLevelRole := insertTestRoleWithLevel(t, svcCtx, productCode, consts.StatusEnabled, callerLevel)
  675. // 修复后 GuardRoleLevelAssignable 走 DB 强一致读取 caller 的 MinPermsLevel,
  676. // 因此需要在 DB 里为调用者落地真实的 user + role + user_role 关系链。
  677. callerUserId, callerCleanup := seedCallerWithRoleLevel(t, svcCtx, productCode, callerLevel)
  678. t.Cleanup(callerCleanup)
  679. t.Cleanup(func() {
  680. testutil.CleanTableByField(superCtx, conn, "`sys_user_role`", "userId", targetUserId)
  681. testutil.CleanTable(superCtx, conn, "`sys_product_member`", mId)
  682. testutil.CleanTable(superCtx, conn, "`sys_user`", targetUserId)
  683. testutil.CleanTable(superCtx, conn, "`sys_role`", sameLevelRole)
  684. })
  685. ctx := ctxhelper.CustomCtx(&loaders.UserDetails{
  686. UserId: callerUserId,
  687. Username: "member_eq_level",
  688. IsSuperAdmin: false,
  689. MemberType: consts.MemberTypeMember,
  690. Status: consts.StatusEnabled,
  691. ProductCode: productCode,
  692. DeptId: deptId,
  693. DeptPath: deptPath,
  694. MinPermsLevel: callerLevel,
  695. })
  696. err := NewBindRolesLogic(ctx, svcCtx).BindRoles(&types.BindRolesReq{
  697. UserId: targetUserId,
  698. RoleIds: []int64{sameLevelRole},
  699. })
  700. require.Error(t, err, "同级角色分配必须被拒绝(含同级)")
  701. var ce *response.CodeError
  702. require.True(t, errors.As(err, &ce))
  703. assert.Equal(t, 403, ce.Code())
  704. assert.Contains(t, ce.Error(), "不能分配权限级别高于自身的角色",
  705. "错误消息应当明确点出'含同级'的拦截语义")
  706. // 同时验证 DB 未产生任何 user-role 关系。
  707. rids, err := svcCtx.SysUserRoleModel.FindRoleIdsByUserIdForProduct(ctx, targetUserId, productCode)
  708. require.NoError(t, err)
  709. assert.Empty(t, rids, "被拒绝的 BindRoles 不得落地任何行")
  710. }