feat: Implement get flows API endpoint
This commit is contained in:
@@ -2,6 +2,7 @@ import { renderObject } from '../../../../helpers/renderer.js';
|
|||||||
|
|
||||||
export default async (request, response) => {
|
export default async (request, response) => {
|
||||||
const execution = await request.currentUser.authorizedExecutions
|
const execution = await request.currentUser.authorizedExecutions
|
||||||
|
.clone()
|
||||||
.withGraphFetched({
|
.withGraphFetched({
|
||||||
flow: {
|
flow: {
|
||||||
steps: true,
|
steps: true,
|
||||||
|
@@ -3,6 +3,7 @@ import paginateRest from '../../../../helpers/pagination-rest.js';
|
|||||||
|
|
||||||
export default async (request, response) => {
|
export default async (request, response) => {
|
||||||
const executionsQuery = request.currentUser.authorizedExecutions
|
const executionsQuery = request.currentUser.authorizedExecutions
|
||||||
|
.clone()
|
||||||
.withSoftDeleted()
|
.withSoftDeleted()
|
||||||
.orderBy('created_at', 'desc')
|
.orderBy('created_at', 'desc')
|
||||||
.withGraphFetched({
|
.withGraphFetched({
|
||||||
|
@@ -2,6 +2,7 @@ import { renderObject } from '../../../../helpers/renderer.js';
|
|||||||
|
|
||||||
export default async (request, response) => {
|
export default async (request, response) => {
|
||||||
const flow = await request.currentUser.authorizedFlows
|
const flow = await request.currentUser.authorizedFlows
|
||||||
|
.clone()
|
||||||
.withGraphJoined({ steps: true })
|
.withGraphJoined({ steps: true })
|
||||||
.orderBy('steps.position', 'asc')
|
.orderBy('steps.position', 'asc')
|
||||||
.findOne({ 'flows.id': request.params.flowId })
|
.findOne({ 'flows.id': request.params.flowId })
|
||||||
|
21
packages/backend/src/controllers/api/v1/flows/get-flows.js
Normal file
21
packages/backend/src/controllers/api/v1/flows/get-flows.js
Normal file
@@ -0,0 +1,21 @@
|
|||||||
|
import { renderObject } from '../../../../helpers/renderer.js';
|
||||||
|
import paginateRest from '../../../../helpers/pagination-rest.js';
|
||||||
|
|
||||||
|
export default async (request, response) => {
|
||||||
|
const flowsQuery = request.currentUser.authorizedFlows
|
||||||
|
.clone()
|
||||||
|
.withGraphFetched({
|
||||||
|
steps: true,
|
||||||
|
})
|
||||||
|
.where((builder) => {
|
||||||
|
if (request.query.name) {
|
||||||
|
builder.where('flows.name', 'ilike', `%${request.query.name}%`);
|
||||||
|
}
|
||||||
|
})
|
||||||
|
.orderBy('active', 'desc')
|
||||||
|
.orderBy('updated_at', 'desc');
|
||||||
|
|
||||||
|
const flows = await paginateRest(flowsQuery, request.query.page);
|
||||||
|
|
||||||
|
renderObject(response, flows);
|
||||||
|
};
|
118
packages/backend/src/controllers/api/v1/flows/get-flows.test.js
Normal file
118
packages/backend/src/controllers/api/v1/flows/get-flows.test.js
Normal file
@@ -0,0 +1,118 @@
|
|||||||
|
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';
|
||||||
|
import { createUser } from '../../../../../test/factories/user';
|
||||||
|
import { createFlow } from '../../../../../test/factories/flow';
|
||||||
|
import { createStep } from '../../../../../test/factories/step';
|
||||||
|
import { createPermission } from '../../../../../test/factories/permission';
|
||||||
|
import getFlowsMock from '../../../../../test/mocks/rest/api/v1/flows/get-flows.js';
|
||||||
|
|
||||||
|
describe('GET /api/v1/flows', () => {
|
||||||
|
let currentUser, currentUserRole, token;
|
||||||
|
|
||||||
|
beforeEach(async () => {
|
||||||
|
currentUser = await createUser();
|
||||||
|
currentUserRole = await currentUser.$relatedQuery('role');
|
||||||
|
|
||||||
|
token = createAuthTokenByUserId(currentUser.id);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('should return the flows data of current user', async () => {
|
||||||
|
const currentUserFlowOne = await createFlow({ userId: currentUser.id });
|
||||||
|
|
||||||
|
const triggerStepFlowOne = await createStep({
|
||||||
|
flowId: currentUserFlowOne.id,
|
||||||
|
type: 'trigger',
|
||||||
|
});
|
||||||
|
const actionStepFlowOne = await createStep({
|
||||||
|
flowId: currentUserFlowOne.id,
|
||||||
|
type: 'action',
|
||||||
|
});
|
||||||
|
|
||||||
|
const currentUserFlowTwo = await createFlow({ userId: currentUser.id });
|
||||||
|
|
||||||
|
const triggerStepFlowTwo = await createStep({
|
||||||
|
flowId: currentUserFlowTwo.id,
|
||||||
|
type: 'trigger',
|
||||||
|
});
|
||||||
|
const actionStepFlowTwo = await createStep({
|
||||||
|
flowId: currentUserFlowTwo.id,
|
||||||
|
type: 'action',
|
||||||
|
});
|
||||||
|
|
||||||
|
await createPermission({
|
||||||
|
action: 'read',
|
||||||
|
subject: 'Flow',
|
||||||
|
roleId: currentUserRole.id,
|
||||||
|
conditions: ['isCreator'],
|
||||||
|
});
|
||||||
|
|
||||||
|
const response = await request(app)
|
||||||
|
.get('/api/v1/flows')
|
||||||
|
.set('Authorization', token)
|
||||||
|
.expect(200);
|
||||||
|
|
||||||
|
const expectedPayload = await getFlowsMock(
|
||||||
|
[currentUserFlowTwo, currentUserFlowOne],
|
||||||
|
[
|
||||||
|
triggerStepFlowOne,
|
||||||
|
actionStepFlowOne,
|
||||||
|
triggerStepFlowTwo,
|
||||||
|
actionStepFlowTwo,
|
||||||
|
]
|
||||||
|
);
|
||||||
|
|
||||||
|
expect(response.body).toEqual(expectedPayload);
|
||||||
|
});
|
||||||
|
|
||||||
|
it('should return the flows data of another user', async () => {
|
||||||
|
const anotherUser = await createUser();
|
||||||
|
|
||||||
|
const anotherUserFlowOne = await createFlow({ userId: anotherUser.id });
|
||||||
|
|
||||||
|
const triggerStepFlowOne = await createStep({
|
||||||
|
flowId: anotherUserFlowOne.id,
|
||||||
|
type: 'trigger',
|
||||||
|
});
|
||||||
|
const actionStepFlowOne = await createStep({
|
||||||
|
flowId: anotherUserFlowOne.id,
|
||||||
|
type: 'action',
|
||||||
|
});
|
||||||
|
|
||||||
|
const anotherUserFlowTwo = await createFlow({ userId: anotherUser.id });
|
||||||
|
|
||||||
|
const triggerStepFlowTwo = await createStep({
|
||||||
|
flowId: anotherUserFlowTwo.id,
|
||||||
|
type: 'trigger',
|
||||||
|
});
|
||||||
|
const actionStepFlowTwo = await createStep({
|
||||||
|
flowId: anotherUserFlowTwo.id,
|
||||||
|
type: 'action',
|
||||||
|
});
|
||||||
|
|
||||||
|
await createPermission({
|
||||||
|
action: 'read',
|
||||||
|
subject: 'Flow',
|
||||||
|
roleId: currentUserRole.id,
|
||||||
|
conditions: [],
|
||||||
|
});
|
||||||
|
|
||||||
|
const response = await request(app)
|
||||||
|
.get('/api/v1/flows')
|
||||||
|
.set('Authorization', token)
|
||||||
|
.expect(200);
|
||||||
|
|
||||||
|
const expectedPayload = await getFlowsMock(
|
||||||
|
[anotherUserFlowTwo, anotherUserFlowOne],
|
||||||
|
[
|
||||||
|
triggerStepFlowOne,
|
||||||
|
actionStepFlowOne,
|
||||||
|
triggerStepFlowTwo,
|
||||||
|
actionStepFlowTwo,
|
||||||
|
]
|
||||||
|
);
|
||||||
|
|
||||||
|
expect(response.body).toEqual(expectedPayload);
|
||||||
|
});
|
||||||
|
});
|
@@ -11,6 +11,10 @@ const authorizationList = {
|
|||||||
action: 'read',
|
action: 'read',
|
||||||
subject: 'Flow',
|
subject: 'Flow',
|
||||||
},
|
},
|
||||||
|
'GET /api/v1/flows/': {
|
||||||
|
action: 'read',
|
||||||
|
subject: 'Flow',
|
||||||
|
},
|
||||||
'GET /api/v1/executions/:executionId': {
|
'GET /api/v1/executions/:executionId': {
|
||||||
action: 'read',
|
action: 'read',
|
||||||
subject: 'Execution',
|
subject: 'Execution',
|
||||||
|
@@ -2,10 +2,13 @@ import { Router } from 'express';
|
|||||||
import asyncHandler from 'express-async-handler';
|
import asyncHandler from 'express-async-handler';
|
||||||
import { authenticateUser } from '../../../helpers/authentication.js';
|
import { authenticateUser } from '../../../helpers/authentication.js';
|
||||||
import { authorizeUser } from '../../../helpers/authorization.js';
|
import { authorizeUser } from '../../../helpers/authorization.js';
|
||||||
|
import getFlowsAction from '../../../controllers/api/v1/flows/get-flows.js';
|
||||||
import getFlowAction from '../../../controllers/api/v1/flows/get-flow.js';
|
import getFlowAction from '../../../controllers/api/v1/flows/get-flow.js';
|
||||||
|
|
||||||
const router = Router();
|
const router = Router();
|
||||||
|
|
||||||
|
router.get('/', authenticateUser, authorizeUser, asyncHandler(getFlowsAction));
|
||||||
|
|
||||||
router.get(
|
router.get(
|
||||||
'/:flowId',
|
'/:flowId',
|
||||||
authenticateUser,
|
authenticateUser,
|
||||||
|
36
packages/backend/test/mocks/rest/api/v1/flows/get-flows.js
Normal file
36
packages/backend/test/mocks/rest/api/v1/flows/get-flows.js
Normal file
@@ -0,0 +1,36 @@
|
|||||||
|
const getFlowsMock = async (flows, steps) => {
|
||||||
|
const data = flows.map((flow) => {
|
||||||
|
const flowSteps = steps.filter((step) => step.flowId === flow.id);
|
||||||
|
|
||||||
|
return {
|
||||||
|
active: flow.active,
|
||||||
|
id: flow.id,
|
||||||
|
name: flow.name,
|
||||||
|
status: flow.active ? 'published' : 'draft',
|
||||||
|
steps: flowSteps.map((step) => ({
|
||||||
|
appKey: step.appKey,
|
||||||
|
iconUrl: step.iconUrl,
|
||||||
|
id: step.id,
|
||||||
|
key: step.key,
|
||||||
|
parameters: step.parameters,
|
||||||
|
position: step.position,
|
||||||
|
status: step.status,
|
||||||
|
type: step.type,
|
||||||
|
webhookUrl: step.webhookUrl,
|
||||||
|
})),
|
||||||
|
};
|
||||||
|
});
|
||||||
|
|
||||||
|
return {
|
||||||
|
data: data,
|
||||||
|
meta: {
|
||||||
|
count: data.length,
|
||||||
|
currentPage: 1,
|
||||||
|
isArray: true,
|
||||||
|
totalPages: 1,
|
||||||
|
type: 'Flow',
|
||||||
|
},
|
||||||
|
};
|
||||||
|
};
|
||||||
|
|
||||||
|
export default getFlowsMock;
|
Reference in New Issue
Block a user