feat: Implement import flow API endpoint
This commit is contained in:
@@ -42,7 +42,7 @@ describe('POST /api/v1/flows/:flowId/export', () => {
|
||||
key: 'text',
|
||||
name: 'Text',
|
||||
parameters: {
|
||||
input: `hello {{step.${triggerStep.id}.query.sample}} deneme`,
|
||||
input: `hello {{step.${triggerStep.id}.query.sample}} world`,
|
||||
transform: 'capitalize',
|
||||
},
|
||||
position: 2,
|
||||
@@ -99,7 +99,7 @@ describe('POST /api/v1/flows/:flowId/export', () => {
|
||||
key: 'text',
|
||||
name: 'Text',
|
||||
parameters: {
|
||||
input: `hello {{step.${triggerStep.id}.query.sample}} deneme`,
|
||||
input: `hello {{step.${triggerStep.id}.query.sample}} world`,
|
||||
transform: 'capitalize',
|
||||
},
|
||||
position: 2,
|
||||
|
||||
25
packages/backend/src/controllers/api/v1/flows/import-flow.js
Normal file
25
packages/backend/src/controllers/api/v1/flows/import-flow.js
Normal file
@@ -0,0 +1,25 @@
|
||||
import { renderObject } from '../../../../helpers/renderer.js';
|
||||
import importFlow from '../../../../helpers/import-flow.js';
|
||||
|
||||
export default async (request, response) => {
|
||||
const flow = await importFlow(request.currentUser, flowParams(request));
|
||||
|
||||
return renderObject(response, flow, { status: 201 });
|
||||
};
|
||||
|
||||
const flowParams = (request) => {
|
||||
return {
|
||||
id: request.body.id,
|
||||
name: request.body.name,
|
||||
steps: request.body.steps.map((step) => ({
|
||||
id: step.id,
|
||||
key: step.key,
|
||||
name: step.name,
|
||||
appKey: step.appKey,
|
||||
type: step.type,
|
||||
parameters: step.parameters,
|
||||
position: step.position,
|
||||
webhookPath: step.webhookPath,
|
||||
})),
|
||||
};
|
||||
};
|
||||
@@ -0,0 +1,328 @@
|
||||
import { describe, it, expect, beforeEach } from 'vitest';
|
||||
import request from 'supertest';
|
||||
import app from '../../../../app.js';
|
||||
import createAuthTokenByUserId from '../../../../helpers/create-auth-token-by-user-id.js';
|
||||
import { createUser } from '../../../../../test/factories/user.js';
|
||||
import { createFlow } from '../../../../../test/factories/flow.js';
|
||||
import { createStep } from '../../../../../test/factories/step.js';
|
||||
import { createPermission } from '../../../../../test/factories/permission.js';
|
||||
import importFlowMock from '../../../../../test/mocks/rest/api/v1/flows/import-flow.js';
|
||||
|
||||
describe('POST /api/v1/flows/import', () => {
|
||||
let currentUser, currentUserRole, token;
|
||||
|
||||
beforeEach(async () => {
|
||||
currentUser = await createUser();
|
||||
currentUserRole = await currentUser.$relatedQuery('role');
|
||||
|
||||
token = await createAuthTokenByUserId(currentUser.id);
|
||||
});
|
||||
|
||||
it('should import the flow data', async () => {
|
||||
const currentUserFlow = await createFlow({ userId: currentUser.id });
|
||||
|
||||
const triggerStep = await createStep({
|
||||
flowId: currentUserFlow.id,
|
||||
type: 'trigger',
|
||||
appKey: 'webhook',
|
||||
key: 'catchRawWebhook',
|
||||
name: 'Catch raw webhook',
|
||||
parameters: {
|
||||
workSynchronously: true,
|
||||
},
|
||||
position: 1,
|
||||
webhookPath: `/webhooks/flows/${currentUserFlow.id}/sync`,
|
||||
});
|
||||
|
||||
const actionStep = await createStep({
|
||||
flowId: currentUserFlow.id,
|
||||
type: 'action',
|
||||
appKey: 'formatter',
|
||||
key: 'text',
|
||||
name: 'Text',
|
||||
parameters: {
|
||||
input: `hello {{step.${triggerStep.id}.query.sample}} world`,
|
||||
transform: 'capitalize',
|
||||
},
|
||||
position: 2,
|
||||
});
|
||||
|
||||
await createPermission({
|
||||
action: 'create',
|
||||
subject: 'Flow',
|
||||
roleId: currentUserRole.id,
|
||||
conditions: ['isCreator'],
|
||||
});
|
||||
|
||||
const importFlowData = {
|
||||
id: currentUserFlow.id,
|
||||
name: currentUserFlow.name,
|
||||
steps: [
|
||||
{
|
||||
id: triggerStep.id,
|
||||
key: triggerStep.key,
|
||||
name: triggerStep.name,
|
||||
appKey: triggerStep.appKey,
|
||||
type: triggerStep.type,
|
||||
parameters: triggerStep.parameters,
|
||||
position: triggerStep.position,
|
||||
webhookPath: triggerStep.webhookPath,
|
||||
},
|
||||
{
|
||||
id: actionStep.id,
|
||||
key: actionStep.key,
|
||||
name: actionStep.name,
|
||||
appKey: actionStep.appKey,
|
||||
type: actionStep.type,
|
||||
parameters: actionStep.parameters,
|
||||
position: actionStep.position,
|
||||
},
|
||||
],
|
||||
};
|
||||
|
||||
const response = await request(app)
|
||||
.post('/api/v1/flows/import')
|
||||
.set('Authorization', token)
|
||||
.send(importFlowData)
|
||||
.expect(201);
|
||||
|
||||
const expectedPayload = await importFlowMock(currentUserFlow, [
|
||||
triggerStep,
|
||||
actionStep,
|
||||
]);
|
||||
|
||||
expect(response.body).toMatchObject(expectedPayload);
|
||||
});
|
||||
|
||||
it('should have correct parameters of the steps', async () => {
|
||||
const currentUserFlow = await createFlow({ userId: currentUser.id });
|
||||
|
||||
const triggerStep = await createStep({
|
||||
flowId: currentUserFlow.id,
|
||||
type: 'trigger',
|
||||
appKey: 'webhook',
|
||||
key: 'catchRawWebhook',
|
||||
name: 'Catch raw webhook',
|
||||
parameters: {
|
||||
workSynchronously: true,
|
||||
},
|
||||
position: 1,
|
||||
webhookPath: `/webhooks/flows/${currentUserFlow.id}/sync`,
|
||||
});
|
||||
|
||||
const actionStep = await createStep({
|
||||
flowId: currentUserFlow.id,
|
||||
type: 'action',
|
||||
appKey: 'formatter',
|
||||
key: 'text',
|
||||
name: 'Text',
|
||||
parameters: {
|
||||
input: `hello {{step.${triggerStep.id}.query.sample}} world`,
|
||||
transform: 'capitalize',
|
||||
},
|
||||
position: 2,
|
||||
});
|
||||
|
||||
await createPermission({
|
||||
action: 'create',
|
||||
subject: 'Flow',
|
||||
roleId: currentUserRole.id,
|
||||
conditions: ['isCreator'],
|
||||
});
|
||||
|
||||
const importFlowData = {
|
||||
id: currentUserFlow.id,
|
||||
name: currentUserFlow.name,
|
||||
steps: [
|
||||
{
|
||||
id: triggerStep.id,
|
||||
key: triggerStep.key,
|
||||
name: triggerStep.name,
|
||||
appKey: triggerStep.appKey,
|
||||
type: triggerStep.type,
|
||||
parameters: triggerStep.parameters,
|
||||
position: triggerStep.position,
|
||||
webhookPath: triggerStep.webhookPath,
|
||||
},
|
||||
{
|
||||
id: actionStep.id,
|
||||
key: actionStep.key,
|
||||
name: actionStep.name,
|
||||
appKey: actionStep.appKey,
|
||||
type: actionStep.type,
|
||||
parameters: actionStep.parameters,
|
||||
position: actionStep.position,
|
||||
},
|
||||
],
|
||||
};
|
||||
|
||||
const response = await request(app)
|
||||
.post('/api/v1/flows/import')
|
||||
.set('Authorization', token)
|
||||
.send(importFlowData)
|
||||
.expect(201);
|
||||
|
||||
const newTriggerParameters = response.body.data.steps[0].parameters;
|
||||
const newActionParameters = response.body.data.steps[1].parameters;
|
||||
const newTriggerStepId = response.body.data.steps[0].id;
|
||||
|
||||
expect(newTriggerParameters).toMatchObject({
|
||||
workSynchronously: true,
|
||||
});
|
||||
|
||||
expect(newActionParameters).toMatchObject({
|
||||
input: `hello {{step.${newTriggerStepId}.query.sample}} world`,
|
||||
transform: 'capitalize',
|
||||
});
|
||||
});
|
||||
|
||||
it('should have the new flow id in the new webhook url', async () => {
|
||||
const currentUserFlow = await createFlow({ userId: currentUser.id });
|
||||
|
||||
const triggerStep = await createStep({
|
||||
flowId: currentUserFlow.id,
|
||||
type: 'trigger',
|
||||
appKey: 'webhook',
|
||||
key: 'catchRawWebhook',
|
||||
name: 'Catch raw webhook',
|
||||
parameters: {
|
||||
workSynchronously: true,
|
||||
},
|
||||
position: 1,
|
||||
webhookPath: `/webhooks/flows/${currentUserFlow.id}/sync`,
|
||||
});
|
||||
|
||||
const actionStep = await createStep({
|
||||
flowId: currentUserFlow.id,
|
||||
type: 'action',
|
||||
appKey: 'formatter',
|
||||
key: 'text',
|
||||
name: 'Text',
|
||||
parameters: {
|
||||
input: `hello {{step.${triggerStep.id}.query.sample}} world`,
|
||||
transform: 'capitalize',
|
||||
},
|
||||
position: 2,
|
||||
});
|
||||
|
||||
await createPermission({
|
||||
action: 'create',
|
||||
subject: 'Flow',
|
||||
roleId: currentUserRole.id,
|
||||
conditions: ['isCreator'],
|
||||
});
|
||||
|
||||
const importFlowData = {
|
||||
id: currentUserFlow.id,
|
||||
name: currentUserFlow.name,
|
||||
steps: [
|
||||
{
|
||||
id: triggerStep.id,
|
||||
key: triggerStep.key,
|
||||
name: triggerStep.name,
|
||||
appKey: triggerStep.appKey,
|
||||
type: triggerStep.type,
|
||||
parameters: triggerStep.parameters,
|
||||
position: triggerStep.position,
|
||||
webhookPath: triggerStep.webhookPath,
|
||||
},
|
||||
{
|
||||
id: actionStep.id,
|
||||
key: actionStep.key,
|
||||
name: actionStep.name,
|
||||
appKey: actionStep.appKey,
|
||||
type: actionStep.type,
|
||||
parameters: actionStep.parameters,
|
||||
position: actionStep.position,
|
||||
},
|
||||
],
|
||||
};
|
||||
|
||||
const response = await request(app)
|
||||
.post('/api/v1/flows/import')
|
||||
.set('Authorization', token)
|
||||
.send(importFlowData)
|
||||
.expect(201);
|
||||
|
||||
const newWebhookUrl = response.body.data.steps[0].webhookUrl;
|
||||
|
||||
expect(newWebhookUrl).toContain(`/webhooks/flows/${response.body.data.id}`);
|
||||
});
|
||||
|
||||
it('should have the first step id in the input parameter of the second step', async () => {
|
||||
const currentUserFlow = await createFlow({ userId: currentUser.id });
|
||||
|
||||
const triggerStep = await createStep({
|
||||
flowId: currentUserFlow.id,
|
||||
type: 'trigger',
|
||||
appKey: 'webhook',
|
||||
key: 'catchRawWebhook',
|
||||
name: 'Catch raw webhook',
|
||||
parameters: {
|
||||
workSynchronously: true,
|
||||
},
|
||||
position: 1,
|
||||
webhookPath: `/webhooks/flows/${currentUserFlow.id}/sync`,
|
||||
});
|
||||
|
||||
const actionStep = await createStep({
|
||||
flowId: currentUserFlow.id,
|
||||
type: 'action',
|
||||
appKey: 'formatter',
|
||||
key: 'text',
|
||||
name: 'Text',
|
||||
parameters: {
|
||||
input: `hello {{step.${triggerStep.id}.query.sample}} world`,
|
||||
transform: 'capitalize',
|
||||
},
|
||||
position: 2,
|
||||
});
|
||||
|
||||
await createPermission({
|
||||
action: 'create',
|
||||
subject: 'Flow',
|
||||
roleId: currentUserRole.id,
|
||||
conditions: ['isCreator'],
|
||||
});
|
||||
|
||||
const importFlowData = {
|
||||
id: currentUserFlow.id,
|
||||
name: currentUserFlow.name,
|
||||
steps: [
|
||||
{
|
||||
id: triggerStep.id,
|
||||
key: triggerStep.key,
|
||||
name: triggerStep.name,
|
||||
appKey: triggerStep.appKey,
|
||||
type: triggerStep.type,
|
||||
parameters: triggerStep.parameters,
|
||||
position: triggerStep.position,
|
||||
webhookPath: triggerStep.webhookPath,
|
||||
},
|
||||
{
|
||||
id: actionStep.id,
|
||||
key: actionStep.key,
|
||||
name: actionStep.name,
|
||||
appKey: actionStep.appKey,
|
||||
type: actionStep.type,
|
||||
parameters: actionStep.parameters,
|
||||
position: actionStep.position,
|
||||
},
|
||||
],
|
||||
};
|
||||
|
||||
const response = await request(app)
|
||||
.post('/api/v1/flows/import')
|
||||
.set('Authorization', token)
|
||||
.send(importFlowData)
|
||||
.expect(201);
|
||||
|
||||
const newTriggerStepId = response.body.data.steps[0].id;
|
||||
const newActionStepInputParameter =
|
||||
response.body.data.steps[1].parameters.input;
|
||||
|
||||
expect(newActionStepInputParameter).toContain(
|
||||
`{{step.${newTriggerStepId}.query.sample}}`
|
||||
);
|
||||
});
|
||||
});
|
||||
Reference in New Issue
Block a user