mirror of
https://github.com/n8n-io/n8n.git
synced 2026-05-26 14:25:35 +02:00
Co-authored-by: Declan Carroll <declan@n8n.io> Co-authored-by: Milorad FIlipović <milorad@n8n.io> Co-authored-by: Ricardo Espinoza <ricardo@n8n.io> Co-authored-by: Michael Kret <88898367+michael-radency@users.noreply.github.com> Co-authored-by: Charlie Kolb <charlie@n8n.io> Co-authored-by: Mutasem Aldmour <4711238+mutdmour@users.noreply.github.com> Co-authored-by: Ali Elkhateeb <ali.elkhateeb@n8n.io> Co-authored-by: Matsu <huhta.matias@gmail.com> Co-authored-by: Guillaume Jacquart <jacquart.guillaume@gmail.com> Co-authored-by: yehorkardash <yehor.kardash@n8n.io> Co-authored-by: Rob Hough <robhough180@gmail.com> Co-authored-by: Svetoslav Dekov <svetoslav.dekov@n8n.io> Co-authored-by: James Gee <1285296+geemanjs@users.noreply.github.com> Co-authored-by: Alex Grozav <alex@grozav.com> Co-authored-by: Dawid Myslak <dawid.myslak@gmail.com> Co-authored-by: Stephen Wright <sjw948@gmail.com> Co-authored-by: Andreas Fitzek <andreas.fitzek@n8n.io> Co-authored-by: Albert Alises <albert.alises@gmail.com> Co-authored-by: Dimitri Lavrenük <20122620+dlavrenuek@users.noreply.github.com> Co-authored-by: Danny Martini <danny@n8n.io> Co-authored-by: Daria <daria.staferova@n8n.io> Co-authored-by: peteawood <pete.a.wood@gmail.com> Co-authored-by: Roman Davydchuk <roman.davydchuk@n8n.io> Co-authored-by: Jaakko Husso <jaakko@n8n.io> Co-authored-by: Elias Meire <elias@meire.dev> Co-authored-by: Cursor <cursoragent@cursor.com> Co-authored-by: Eugene <eugene@n8n.io> Co-authored-by: Benjamin Schroth <benjamin@n8n.io> Co-authored-by: Sandra Zollner <sandra.zollner@n8n.io> Co-authored-by: Irénée <irenee.ajeneza@n8n.io> Co-authored-by: Benjamin Schroth <68321970+schrothbn@users.noreply.github.com> Co-authored-by: oleg <me@olegivaniv.com> Co-authored-by: Konstantin Tieber <46342664+konstantintieber@users.noreply.github.com> Co-authored-by: Nikhil Kuriakose <nikhilkuria@gmail.com> Co-authored-by: Romeo Balta <7095569+romeobalta@users.noreply.github.com> Co-authored-by: Tuukka Kantola <Tuukkaa@users.noreply.github.com> Co-authored-by: Kaito Horiuchi <horiyee.ka@gmail.com> Co-authored-by: Michael Drury <me@michaeldrury.co.uk> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
254 lines
8.1 KiB
TypeScript
254 lines
8.1 KiB
TypeScript
import type { CreateRoleDto, UpdateRoleDto } from '@n8n/api-types';
|
|
import { createTeamProject, linkUserToProject, testDb } from '@n8n/backend-test-utils';
|
|
import {
|
|
PROJECT_ADMIN_ROLE,
|
|
PROJECT_EDITOR_ROLE,
|
|
PROJECT_OWNER_ROLE,
|
|
PROJECT_VIEWER_ROLE,
|
|
RoleRepository,
|
|
} from '@n8n/db';
|
|
import type { User } from '@n8n/db';
|
|
import { Container } from '@n8n/di';
|
|
|
|
import { cleanupRolesAndScopes } from '../shared/db/roles';
|
|
import { createMember, createOwner } from '../shared/db/users';
|
|
import type { SuperAgentTest } from '../shared/types';
|
|
import { setupTestServer } from '../shared/utils';
|
|
|
|
describe('RoleController - Integration Tests', () => {
|
|
const testServer = setupTestServer({ endpointGroups: ['role'] });
|
|
let ownerAgent: SuperAgentTest;
|
|
let memberAgent: SuperAgentTest;
|
|
let owner: User;
|
|
let member: User;
|
|
|
|
beforeAll(async () => {
|
|
await testDb.init();
|
|
owner = await createOwner();
|
|
member = await createMember();
|
|
ownerAgent = testServer.authAgentFor(owner);
|
|
memberAgent = testServer.authAgentFor(member);
|
|
});
|
|
|
|
beforeEach(() => {
|
|
testServer.license.enable('feat:customRoles');
|
|
});
|
|
|
|
afterEach(async () => {
|
|
await cleanupRolesAndScopes();
|
|
await Container.get(RoleRepository).delete({ systemRole: false });
|
|
});
|
|
|
|
afterAll(async () => {
|
|
await testDb.terminate();
|
|
});
|
|
|
|
describe('GET /roles/:slug', () => {
|
|
const staticRoles = [PROJECT_ADMIN_ROLE, PROJECT_EDITOR_ROLE, PROJECT_VIEWER_ROLE];
|
|
|
|
it.each(staticRoles)('should return 200 and the role data for role $slug', async (role) => {
|
|
const response = await memberAgent.get(`/roles/${role.slug}`).expect(200);
|
|
|
|
response.body.data.scopes.sort();
|
|
expect(response.body).toEqual({
|
|
data: {
|
|
slug: role.slug,
|
|
displayName: role.displayName,
|
|
description: role.description,
|
|
systemRole: role.systemRole,
|
|
roleType: role.roleType,
|
|
scopes: role.scopes.map((scope) => scope.slug).sort(),
|
|
licensed: expect.any(Boolean),
|
|
createdAt: expect.any(String),
|
|
updatedAt: expect.any(String),
|
|
},
|
|
});
|
|
});
|
|
|
|
it('should return 200 and the role data for PROJECT_OWNER_ROLE with dynamic scopes', async () => {
|
|
// PROJECT_OWNER_ROLE has conditional scopes based on security settings.
|
|
// The workflow:publish scope is dynamically added/removed based on the
|
|
// personal space publishing setting. We fetch the actual role from the
|
|
// database to get the current scopes.
|
|
const roleRepository = Container.get(RoleRepository);
|
|
const dbRole = await roleRepository.findBySlug(PROJECT_OWNER_ROLE.slug);
|
|
expect(dbRole).not.toBeNull();
|
|
|
|
const response = await memberAgent.get(`/roles/${PROJECT_OWNER_ROLE.slug}`).expect(200);
|
|
|
|
response.body.data.scopes.sort();
|
|
const expectedScopes = dbRole!.scopes.map((scope) => scope.slug).sort();
|
|
|
|
expect(response.body).toEqual({
|
|
data: {
|
|
slug: PROJECT_OWNER_ROLE.slug,
|
|
displayName: PROJECT_OWNER_ROLE.displayName,
|
|
description: PROJECT_OWNER_ROLE.description,
|
|
systemRole: PROJECT_OWNER_ROLE.systemRole,
|
|
roleType: PROJECT_OWNER_ROLE.roleType,
|
|
scopes: expectedScopes,
|
|
licensed: expect.any(Boolean),
|
|
createdAt: expect.any(String),
|
|
updatedAt: expect.any(String),
|
|
},
|
|
});
|
|
});
|
|
});
|
|
|
|
describe('POST /roles', () => {
|
|
it('should create a custom role', async () => {
|
|
const createRoleDto: CreateRoleDto = {
|
|
displayName: 'Custom Project Role',
|
|
description: 'A custom role for project management',
|
|
roleType: 'project',
|
|
scopes: ['workflow:create', 'workflow:read'].sort(),
|
|
};
|
|
|
|
const response = await ownerAgent.post('/roles').send(createRoleDto).expect(200);
|
|
|
|
response.body.data.scopes.sort();
|
|
expect(response.body).toEqual({
|
|
data: {
|
|
...createRoleDto,
|
|
slug: expect.any(String),
|
|
licensed: expect.any(Boolean),
|
|
systemRole: false,
|
|
createdAt: expect.any(String),
|
|
updatedAt: expect.any(String),
|
|
},
|
|
});
|
|
|
|
const availableRole = await memberAgent.get(`/roles/${response.body.data.slug}`).expect(200);
|
|
|
|
availableRole.body.data.scopes.sort();
|
|
expect(availableRole.body).toEqual({
|
|
data: {
|
|
...createRoleDto,
|
|
slug: response.body.data.slug,
|
|
licensed: expect.any(Boolean),
|
|
systemRole: false,
|
|
createdAt: expect.any(String),
|
|
updatedAt: expect.any(String),
|
|
},
|
|
});
|
|
});
|
|
});
|
|
|
|
describe('PATCH /roles/:slug', () => {
|
|
it('should update a custom role', async () => {
|
|
const createRoleDto: CreateRoleDto = {
|
|
displayName: 'Custom Project Role',
|
|
description: 'A custom role for project management',
|
|
roleType: 'project',
|
|
scopes: ['workflow:create', 'workflow:read'].sort(),
|
|
};
|
|
|
|
const createResponse = await ownerAgent.post('/roles').send(createRoleDto).expect(200);
|
|
|
|
expect(createResponse.body?.data?.slug).toBeDefined();
|
|
const generatedRoleSlug = createResponse.body.data.slug;
|
|
|
|
const updateRoleDto: UpdateRoleDto = {
|
|
displayName: 'Custom Project Role Updated',
|
|
description: 'A custom role for project management - updated',
|
|
};
|
|
|
|
const response = await ownerAgent
|
|
.patch(`/roles/${generatedRoleSlug}`)
|
|
.send(updateRoleDto)
|
|
.expect(200);
|
|
|
|
response.body.data.scopes.sort();
|
|
expect(response.body).toEqual({
|
|
data: {
|
|
...updateRoleDto,
|
|
scopes: ['workflow:create', 'workflow:read'].sort(),
|
|
slug: generatedRoleSlug,
|
|
roleType: 'project',
|
|
licensed: expect.any(Boolean),
|
|
systemRole: false,
|
|
createdAt: expect.any(String),
|
|
updatedAt: expect.any(String),
|
|
},
|
|
});
|
|
|
|
const availableRole = await memberAgent.get(`/roles/${response.body.data.slug}`).expect(200);
|
|
|
|
availableRole.body.data.scopes.sort();
|
|
expect(availableRole.body).toEqual({
|
|
data: {
|
|
...updateRoleDto,
|
|
scopes: ['workflow:create', 'workflow:read'].sort(),
|
|
slug: generatedRoleSlug,
|
|
roleType: 'project',
|
|
licensed: expect.any(Boolean),
|
|
systemRole: false,
|
|
createdAt: expect.any(String),
|
|
updatedAt: expect.any(String),
|
|
},
|
|
});
|
|
});
|
|
});
|
|
|
|
describe('GET /roles/:slug/assignments', () => {
|
|
it('should return projects where the role is assigned', async () => {
|
|
const project = await createTeamProject('Test Project', owner);
|
|
await linkUserToProject(member, project, 'project:editor');
|
|
|
|
const response = await ownerAgent
|
|
.get(`/roles/${PROJECT_EDITOR_ROLE.slug}/assignments`)
|
|
.expect(200);
|
|
|
|
expect(response.body.data.totalProjects).toBeGreaterThanOrEqual(1);
|
|
const projectNames = response.body.data.projects.map(
|
|
(p: { projectName: string }) => p.projectName,
|
|
);
|
|
expect(projectNames).toContain('Test Project');
|
|
|
|
const testProject = response.body.data.projects.find(
|
|
(p: { projectName: string }) => p.projectName === 'Test Project',
|
|
);
|
|
expect(testProject.memberCount).toBe(1);
|
|
expect(testProject.projectId).toBe(project.id);
|
|
});
|
|
|
|
it('should return empty when role has no assignments', async () => {
|
|
const response = await ownerAgent
|
|
.get(`/roles/${PROJECT_VIEWER_ROLE.slug}/assignments`)
|
|
.expect(200);
|
|
|
|
expect(response.body.data.totalProjects).toBe(0);
|
|
expect(response.body.data.projects).toEqual([]);
|
|
});
|
|
|
|
it('should require role:manage scope (deny member)', async () => {
|
|
await memberAgent.get(`/roles/${PROJECT_EDITOR_ROLE.slug}/assignments`).expect(403);
|
|
});
|
|
});
|
|
|
|
describe('GET /roles/:slug/assignments/:projectId/members', () => {
|
|
it('should return only members with the specified role', async () => {
|
|
const project = await createTeamProject('Members Test', owner);
|
|
await linkUserToProject(member, project, 'project:editor');
|
|
// owner is project:admin via createTeamProject
|
|
|
|
const response = await ownerAgent
|
|
.get(`/roles/${PROJECT_EDITOR_ROLE.slug}/assignments/${project.id}/members`)
|
|
.expect(200);
|
|
|
|
// Should only include the editor, not the admin
|
|
expect(response.body.data.members).toHaveLength(1);
|
|
expect(response.body.data.members[0].email).toBe(member.email);
|
|
expect(response.body.data.members[0].role).toBe('project:editor');
|
|
});
|
|
|
|
it('should require role:manage scope (deny member)', async () => {
|
|
const project = await createTeamProject('Auth Test');
|
|
|
|
await memberAgent
|
|
.get(`/roles/${PROJECT_EDITOR_ROLE.slug}/assignments/${project.id}/members`)
|
|
.expect(403);
|
|
});
|
|
});
|
|
});
|