mudhorn/n8n
1
0
Fork 0
mirror of https://github.com/n8n-io/n8n.git synced 2025-01-23 10:32:17 -08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

test(core): Align test names with route names (no-changelog) (#9518)

Browse source
This commit is contained in:
Danny Martini 2024-05-27 20:41:34 +02:00 committed by GitHub
parent 6ed9ef0b60
commit e07de837b9
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 29 additions and 29 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

50
packages/cli/test/integration/workflows/workflows.controller.ee.test.ts
View file

@ -104,8 +104,8 @@ describe('router should switch based on flag', () => {
});
});
describe('PUT /workflows/:id', () => {
test('PUT /workflows/:id/share should save sharing with new users', async () => {
describe('PUT /workflows/:workflowId/share', () => {
test('should save sharing with new users', async () => {
const workflow = await createWorkflow({}, owner);
const response = await authOwnerAgent
@ -126,7 +126,7 @@ describe('PUT /workflows/:id', () => {
);
});
test('PUT /workflows/:id/share should succeed when sharing with invalid user-id', async () => {
test('should succeed when sharing with invalid user-id', async () => {
const workflow = await createWorkflow({}, owner);
const response = await authOwnerAgent
@ -139,7 +139,7 @@ describe('PUT /workflows/:id', () => {
expect(sharedWorkflows).toHaveLength(1);
});
test('PUT /workflows/:id/share should allow sharing with pending users', async () => {
test('should allow sharing with pending users', async () => {
const workflow = await createWorkflow({}, owner);
const memberShell = await createUserShell('global:member');
const memberShellPersonalProject = await projectRepository.getPersonalProjectForUserOrFail(
@ -157,7 +157,7 @@ describe('PUT /workflows/:id', () => {
expect(mailer.notifyWorkflowShared).toHaveBeenCalledTimes(1);
});
test('PUT /workflows/:id/share should allow sharing with multiple users', async () => {
test('should allow sharing with multiple users', async () => {
const workflow = await createWorkflow({}, owner);
const response = await authOwnerAgent
@ -171,7 +171,7 @@ describe('PUT /workflows/:id', () => {
expect(mailer.notifyWorkflowShared).toHaveBeenCalledTimes(1);
});
test('PUT /workflows/:id/share should override sharing', async () => {
test('should override sharing', async () => {
const workflow = await createWorkflow({}, owner);
const response = await authOwnerAgent
@ -193,7 +193,7 @@ describe('PUT /workflows/:id', () => {
expect(mailer.notifyWorkflowShared).toHaveBeenCalledTimes(2);
});
test('PUT /workflows/:id/share should allow sharing by the owner of the workflow', async () => {
test('should allow sharing by the owner of the workflow', async () => {
const workflow = await createWorkflow({}, member);
const response = await authMemberAgent
@ -207,7 +207,7 @@ describe('PUT /workflows/:id', () => {
expect(mailer.notifyWorkflowShared).toHaveBeenCalledTimes(1);
});
test('PUT /workflows/:id/share should allow sharing by the instance owner', async () => {
test('should allow sharing by the instance owner', async () => {
const workflow = await createWorkflow({}, member);
const response = await authOwnerAgent
@ -221,7 +221,7 @@ describe('PUT /workflows/:id', () => {
expect(mailer.notifyWorkflowShared).toHaveBeenCalledTimes(1);
});
test('PUT /workflows/:id/share should not allow sharing by another shared member', async () => {
test('should not allow sharing by another shared member', async () => {
const workflow = await createWorkflow({}, member);
await shareWorkflowWithUsers(workflow, [anotherMember]);
@ -237,7 +237,7 @@ describe('PUT /workflows/:id', () => {
expect(mailer.notifyWorkflowShared).toHaveBeenCalledTimes(0);
});
test('PUT /workflows/:id/share should not allow sharing with self by another non-shared member', async () => {
test('should not allow sharing with self by another non-shared member', async () => {
const workflow = await createWorkflow({}, member);
const response = await authAnotherMemberAgent
@ -251,7 +251,7 @@ describe('PUT /workflows/:id', () => {
expect(mailer.notifyWorkflowShared).toHaveBeenCalledTimes(0);
});
test('PUT /workflows/:id/share should not allow sharing by another non-shared member', async () => {
test('should not allow sharing by another non-shared member', async () => {
const workflow = await createWorkflow({}, member);
const tempUser = await createUser({ role: 'global:member' });
@ -302,20 +302,20 @@ describe('GET /workflows/new', () => {
});
});
describe('GET /workflows/:id', () => {
test('GET should fail with invalid id due to route rule', async () => {
describe('GET /workflows/:workflowId', () => {
test('should fail with invalid id due to route rule', async () => {
const response = await authOwnerAgent.get('/workflows/potatoes');
expect(response.statusCode).toBe(404);
});
test('GET should return 404 for non existing workflow', async () => {
test('should return 404 for non existing workflow', async () => {
const response = await authOwnerAgent.get('/workflows/9001');
expect(response.statusCode).toBe(404);
});
test('GET should return a workflow with owner', async () => {
test('should return a workflow with owner', async () => {
const workflow = await createWorkflow({}, owner);
const response = await authOwnerAgent.get(`/workflows/${workflow.id}`).expect(200);
@ -343,7 +343,7 @@ describe('GET /workflows/:id', () => {
});
});
test('GET should return shared workflow with user data', async () => {
test('should return shared workflow with user data', async () => {
const workflow = await createWorkflow({}, owner);
await shareWorkflowWithUsers(workflow, [member]);
@ -364,7 +364,7 @@ describe('GET /workflows/:id', () => {
});
});
test('GET should return all sharees', async () => {
test('should return all sharees', async () => {
const workflow = await createWorkflow({}, owner);
await shareWorkflowWithUsers(workflow, [member, anotherMember]);
@ -380,7 +380,7 @@ describe('GET /workflows/:id', () => {
expect(responseWorkflow.sharedWithProjects).toHaveLength(2);
});
test('GET should return workflow with credentials owned by user', async () => {
test('should return workflow with credentials owned by user', async () => {
const savedCredential = await saveCredential(randomCredentialPayload(), { user: owner });
const workflowPayload = makeWorkflow({
@ -404,7 +404,7 @@ describe('GET /workflows/:id', () => {
expect(responseWorkflow.sharedWithProjects).toHaveLength(0);
});
test('GET should return workflow with credentials saying owner does not have access when not shared', async () => {
test('should return workflow with credentials saying owner does not have access when not shared', async () => {
const savedCredential = await saveCredential(randomCredentialPayload(), { user: member });
const workflowPayload = makeWorkflow({
@ -427,7 +427,7 @@ describe('GET /workflows/:id', () => {
expect(responseWorkflow.sharedWithProjects).toHaveLength(0);
});
test('GET should return workflow with credentials for all users with or without access', async () => {
test('should return workflow with credentials for all users with or without access', async () => {
const savedCredential = await saveCredential(randomCredentialPayload(), { user: member });
const workflowPayload = makeWorkflow({
@ -464,7 +464,7 @@ describe('GET /workflows/:id', () => {
expect(member2Workflow.sharedWithProjects).toHaveLength(1);
});
test('GET should return workflow with credentials for all users with access', async () => {
test('should return workflow with credentials for all users with access', async () => {
const savedCredential = await saveCredential(randomCredentialPayload(), { user: member });
// Both users have access to the credential (none is owner)
await shareCredentialWithUsers(savedCredential, [anotherMember]);
@ -659,7 +659,7 @@ describe('POST /workflows', () => {
});
});
describe('PATCH /workflows/:id - validate credential permissions to user', () => {
describe('PATCH /workflows/:workflowId - validate credential permissions to user', () => {
it('Should succeed when saving unchanged workflow nodes', async () => {
const savedCredential = await saveCredential(randomCredentialPayload(), { user: owner });
const workflow = {
@ -891,7 +891,7 @@ describe('PATCH /workflows/:id - validate credential permissions to user', () =>
});
});
describe('PATCH /workflows/:id - validate interim updates', () => {
describe('PATCH /workflows/:workflowId - validate interim updates', () => {
it('should block owner updating workflow nodes on interim update by member', async () => {
// owner creates and shares workflow
@ -1080,7 +1080,7 @@ describe('PATCH /workflows/:id - validate interim updates', () => {
});
});
describe('PATCH /workflows/:id - workflow history', () => {
describe('PATCH /workflows/:workflowId - workflow history', () => {
test('Should create workflow history version when licensed', async () => {
license.enable('feat:workflowHistory');
const workflow = await createWorkflow({}, owner);
@ -1190,7 +1190,7 @@ describe('PATCH /workflows/:id - workflow history', () => {
});
});
describe('PATCH /workflows/:id - activate workflow', () => {
describe('PATCH /workflows/:workflowId - activate workflow', () => {
test('should activate workflow without changing version ID', async () => {
license.disable('feat:workflowHistory');
const workflow = await createWorkflow({}, owner);

8
packages/cli/test/integration/workflows/workflows.controller.test.ts
View file

@ -359,7 +359,7 @@ describe('POST /workflows', () => {
});
});
describe('GET /workflows/:id', () => {
describe('GET /workflows/:workflowId', () => {
test('should return pin data', async () => {
const workflow = makeWorkflow({ withPinData: true });
const workflowCreationResponse = await authOwnerAgent.post('/workflows').send(workflow);
@ -823,7 +823,7 @@ describe('GET /workflows', () => {
});
});
describe('PATCH /workflows/:id', () => {
describe('PATCH /workflows/:workflowId', () => {
test('should create workflow history version when licensed', async () => {
license.enable('feat:workflowHistory');
const workflow = await createWorkflow({}, owner);
@ -997,7 +997,7 @@ describe('PATCH /workflows/:id', () => {
});
});
describe('POST /workflows/run', () => {
describe('POST /workflows/:workflowId/run', () => {
let sharingSpy: jest.SpyInstance;
let tamperingSpy: jest.SpyInstance;
let workflow: WorkflowEntity;
@ -1028,7 +1028,7 @@ describe('POST /workflows/run', () => {
});
});
describe('DELETE /workflows/:id', () => {
describe('DELETE /workflows/:workflowId', () => {
test('deletes a workflow owned by the user', async () => {
const workflow = await createWorkflow({}, owner);