| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183 |
- package dept
- import (
- "context"
- "testing"
- "perms-system-server/internal/consts"
- "perms-system-server/internal/loaders"
- "perms-system-server/internal/middleware"
- deptModel "perms-system-server/internal/model/dept"
- "perms-system-server/internal/testutil/mocks"
- "github.com/stretchr/testify/assert"
- "github.com/stretchr/testify/require"
- "go.uber.org/mock/gomock"
- )
- // ---------------------------------------------------------------------------
- // 覆盖目标:只有 SuperAdmin 才能拿到完整部门树;产品 ADMIN / DEVELOPER / MEMBER 一律
- // 按 caller.DeptPath 前缀过滤,仅返回以其为根的子树。避免:
- // * MEMBER 级账号枚举全公司组织结构;
- // * 定位 DEV 部门再针对性申请权限;
- // * 审计 L-R15-2:小产品 ADMIN 借 fullAccess 侦察大产品的 DEV/HR/财务部门命名,
- // 为针对性社工 / 撞库提供前置输入——sys_dept 是全局命名空间,ADMIN 在产品 P1
- // 的授权范围不应扩散到 P2 的组织结构视图。
- //
- // 测试数据:一棵 "/100/" 根下挂 "/100/1/"、"/100/1/5/",以及一个平行分支 "/200/"。
- // 期望:caller DeptPath="/100/1/" 只能看到 "/100/1/" 和 "/100/1/5/";
- // SuperAdmin 看到完整的 `/100/` + `/200/` 两个根。
- // ---------------------------------------------------------------------------
- var allDepts = []*deptModel.SysDept{
- {Id: 100, ParentId: 0, Path: "/100/", Name: "root"},
- {Id: 1, ParentId: 100, Path: "/100/1/", Name: "childA"},
- {Id: 5, ParentId: 1, Path: "/100/1/5/", Name: "grandchild"},
- {Id: 2, ParentId: 100, Path: "/100/2/", Name: "childB"},
- {Id: 200, ParentId: 0, Path: "/200/", Name: "siblingRoot"},
- }
- func ctxWith(caller *loaders.UserDetails) context.Context {
- return middleware.WithUserDetails(context.Background(), caller)
- }
- // TC-0855: MEMBER DeptPath="/100/1/" 应只看到 "/100/1/" 和 "/100/1/5/",且局部根就是 "/100/1/"。
- func TestDeptTree_Member_PrunedToSubtree(t *testing.T) {
- ctrl := gomock.NewController(t)
- t.Cleanup(ctrl.Finish)
- deptMock := mocks.NewMockSysDeptModel(ctrl)
- deptMock.EXPECT().FindAll(gomock.Any()).Return(allDepts, nil)
- svcCtx := mocks.NewMockServiceContext(mocks.MockModels{Dept: deptMock})
- caller := &loaders.UserDetails{
- UserId: 42, IsSuperAdmin: false, MemberType: consts.MemberTypeMember,
- DeptId: 1, DeptPath: "/100/1/", ProductCode: "pA",
- }
- tree, err := NewDeptTreeLogic(ctxWith(caller), svcCtx).DeptTree()
- require.NoError(t, err)
- // 剪枝后只剩 2 个节点;根仍应只有一个(id=1,grandchild 挂在其下)。
- require.Len(t, tree, 1, "剪枝后根只剩 1 个(caller 所在部门)")
- assert.Equal(t, int64(1), tree[0].Id, "局部根必须是 /100/1/,不得把 /100/ 也暴露")
- assert.Equal(t, "/100/1/", tree[0].Path)
- require.Len(t, tree[0].Children, 1, "grandchild 必须挂在局部根下")
- assert.Equal(t, int64(5), tree[0].Children[0].Id)
- }
- // TC-0856: MEMBER DeptPath="" —— 返回空切片,即使 DB 有数据。
- func TestDeptTree_OrphanMember_ReturnsEmpty(t *testing.T) {
- ctrl := gomock.NewController(t)
- t.Cleanup(ctrl.Finish)
- deptMock := mocks.NewMockSysDeptModel(ctrl)
- // 注意:当前实现会先 FindAll 再剪枝,空 DeptPath 直接走空返回,但 FindAll 仍被调用一次(有些成本),
- // 这里设置 AnyTimes 适配 "就算调用一次也可以"。
- deptMock.EXPECT().FindAll(gomock.Any()).Return(allDepts, nil).AnyTimes()
- svcCtx := mocks.NewMockServiceContext(mocks.MockModels{Dept: deptMock})
- caller := &loaders.UserDetails{
- UserId: 43, IsSuperAdmin: false, MemberType: consts.MemberTypeMember,
- DeptId: 0, DeptPath: "", ProductCode: "pA",
- }
- tree, err := NewDeptTreeLogic(ctxWith(caller), svcCtx).DeptTree()
- require.NoError(t, err)
- assert.Len(t, tree, 0, "DeptPath 为空必须返回空树,不能泄露组织结构")
- }
- // TC-0857(L-R15-2 后契约反转):产品 ADMIN 不再拥有 fullAccess——只能看到以 DeptPath
- // 为根的子树,与 MEMBER / DEVELOPER 同路径。
- //
- // 关键断言不只是"根只剩 1 个",还要显式证明 "平行分支 /200/ 对 ADMIN 不可见",
- // 以免未来如果有人把条件从 `caller.IsSuperAdmin` 又放宽回 `|| MemberType == Admin`,
- // 本测试能在第一次 run 立刻飘红。
- func TestDeptTree_Admin_PrunedToSubtree(t *testing.T) {
- ctrl := gomock.NewController(t)
- t.Cleanup(ctrl.Finish)
- deptMock := mocks.NewMockSysDeptModel(ctrl)
- deptMock.EXPECT().FindAll(gomock.Any()).Return(allDepts, nil)
- svcCtx := mocks.NewMockServiceContext(mocks.MockModels{Dept: deptMock})
- caller := &loaders.UserDetails{
- UserId: 2, IsSuperAdmin: false, MemberType: consts.MemberTypeAdmin,
- DeptId: 1, DeptPath: "/100/1/", ProductCode: "pA",
- }
- tree, err := NewDeptTreeLogic(ctxWith(caller), svcCtx).DeptTree()
- require.NoError(t, err)
- require.Len(t, tree, 1,
- "L-R15-2:产品 ADMIN 不再享有 fullAccess,剪枝后局部根只有 1 个(caller 自己的 DeptPath)")
- assert.Equal(t, int64(1), tree[0].Id,
- "局部根必须是 /100/1/,父部门 /100/ 不应暴露给 ADMIN")
- assert.Equal(t, "/100/1/", tree[0].Path)
- require.Len(t, tree[0].Children, 1, "grandchild /100/1/5/ 必须挂在局部根下")
- assert.Equal(t, int64(5), tree[0].Children[0].Id)
- for _, r := range tree {
- assert.NotEqual(t, int64(200), r.Id,
- "平行分支 /200/ 对产品 ADMIN 必须不可见——若回归到旧 fullAccess,这里立刻飘红")
- assert.NotEqual(t, int64(100), r.Id,
- "父部门 /100/ 也不应暴露给 ADMIN(同上)")
- }
- }
- // TC-1128:SuperAdmin 仍然享有 fullAccess,完整树返回两个根(/100/ + /200/)。
- // 这一条是 L-R15-2 收敛范围的正向回归:确认 `fullAccess = caller.IsSuperAdmin`
- // 的 true 分支未被连带削弱(否则超管运营视角会瘫痪)。
- func TestDeptTree_SuperAdmin_FullTree(t *testing.T) {
- ctrl := gomock.NewController(t)
- t.Cleanup(ctrl.Finish)
- deptMock := mocks.NewMockSysDeptModel(ctrl)
- deptMock.EXPECT().FindAll(gomock.Any()).Return(allDepts, nil)
- svcCtx := mocks.NewMockServiceContext(mocks.MockModels{Dept: deptMock})
- // 即便 DeptPath 指定成某具体子部门,SuperAdmin 也必须拿到全局树——
- // fullAccess 判定是 IsSuperAdmin 而不是 DeptPath。
- caller := &loaders.UserDetails{
- UserId: 1, IsSuperAdmin: true, MemberType: consts.MemberTypeAdmin,
- DeptId: 100, DeptPath: "/100/", ProductCode: "pA",
- }
- tree, err := NewDeptTreeLogic(ctxWith(caller), svcCtx).DeptTree()
- require.NoError(t, err)
- require.Len(t, tree, 2, "SuperAdmin 必须看到完整树,包含所有平行根")
- var rootIds []int64
- for _, r := range tree {
- rootIds = append(rootIds, r.Id)
- }
- assert.ElementsMatch(t, []int64{100, 200}, rootIds,
- "SuperAdmin 走 fullAccess 分支,平行根 /100/ 与 /200/ 必须同时出现")
- }
- // TC-1129:产品 DEVELOPER 与 MEMBER 同路径——只能看自己 DeptPath 的子树。
- // 这一条补齐 L-R15-2 的"角色对称"覆盖:fullAccess 判定只认 SuperAdmin,
- // ADMIN / DEVELOPER / MEMBER 三者剪枝语义必须完全一致。
- func TestDeptTree_Developer_PrunedToSubtree(t *testing.T) {
- ctrl := gomock.NewController(t)
- t.Cleanup(ctrl.Finish)
- deptMock := mocks.NewMockSysDeptModel(ctrl)
- deptMock.EXPECT().FindAll(gomock.Any()).Return(allDepts, nil)
- svcCtx := mocks.NewMockServiceContext(mocks.MockModels{Dept: deptMock})
- caller := &loaders.UserDetails{
- UserId: 3, IsSuperAdmin: false, MemberType: consts.MemberTypeDeveloper,
- DeptId: 1, DeptPath: "/100/1/", ProductCode: "pA",
- }
- tree, err := NewDeptTreeLogic(ctxWith(caller), svcCtx).DeptTree()
- require.NoError(t, err)
- require.Len(t, tree, 1, "DEVELOPER 同样不享有 fullAccess,只能看到自己子树")
- assert.Equal(t, int64(1), tree[0].Id, "局部根必须是 /100/1/")
- for _, r := range tree {
- assert.NotEqual(t, int64(200), r.Id,
- "平行分支 /200/ 对 DEVELOPER 同样不可见")
- }
- }
|