feat: Add the ability to add new automation rule (#3459)

* Add automation modal

* Fix the v-model for automations

* Actions and Condition dropdowns for automations

* Fix merge conflicts

* Handle event change and confirmation

* Appends new action

* Removes actions

* Automation api integration

* Api integration for creating automations

* Registers vuex module to the global store

* Automations table

* Updarted labels and actions

* Integrate automation api

* Fixes the mutation error - removed the data key wrapper

* Fixed the automation condition models to work with respective event types

* Remove temporary fixes added to the api request

* Displa timestamp and automation status values

* Added the clone buton

* Removed uncessary helper method

* Specs for automations

* Handle WIP code

* Remove the payload wrap

* Fix the action query payload

* Removed unnecessary files

* Disabled Automations routes

* Ability to delete automations

* Fix specs

* Fixed merge conflicts

Co-authored-by: Fayaz Ahmed <15716057+fayazara@users.noreply.github.com>
Co-authored-by: fayazara <fayazara@gmail.com>
Co-authored-by: Nithin David Thomas <1277421+nithindavid@users.noreply.github.com>
Co-authored-by: Sivin Varghese <64252451+iamsivin@users.noreply.github.com>
This commit is contained in:
Muhsin Keloth
2022-02-02 18:46:07 +05:30
committed by GitHub
parent 1f3c5002b3
commit 91b9168fae
23 changed files with 1593 additions and 11 deletions

View File

@@ -0,0 +1,89 @@
import * as MutationHelpers from 'shared/helpers/vuex/mutationHelpers';
import types from '../mutation-types';
import AutomationAPI from '../../api/automation';
export const state = {
records: [],
uiFlags: {
isFetching: false,
isCreating: false,
isDeleting: false,
},
};
export const getters = {
getAutomations(_state) {
return _state.records;
},
getUIFlags(_state) {
return _state.uiFlags;
},
};
export const actions = {
get: async function getAutomations({ commit }) {
commit(types.SET_AUTOMATION_UI_FLAG, { isFetching: true });
try {
const response = await AutomationAPI.get();
commit(types.SET_AUTOMATIONS, response.data.payload);
} catch (error) {
// Ignore error
} finally {
commit(types.SET_AUTOMATION_UI_FLAG, { isFetching: false });
}
},
create: async function createAutomation({ commit }, automationObj) {
commit(types.SET_AUTOMATION_UI_FLAG, { isCreating: true });
try {
const response = await AutomationAPI.create(automationObj);
commit(types.ADD_AUTOMATION, response.data);
} catch (error) {
throw new Error(error);
} finally {
commit(types.SET_AUTOMATION_UI_FLAG, { isCreating: false });
}
},
update: async ({ commit }, { id, ...updateObj }) => {
commit(types.SET_AUTOMATION_UI_FLAG, { isUpdating: true });
try {
const response = await AutomationAPI.update(id, updateObj);
commit(types.EDIT_AUTOMATION, response.data);
} catch (error) {
throw new Error(error);
} finally {
commit(types.SET_AUTOMATION_UI_FLAG, { isUpdating: false });
}
},
delete: async ({ commit }, id) => {
commit(types.SET_AUTOMATION_UI_FLAG, { isDeleting: true });
try {
await AutomationAPI.delete(id);
commit(types.DELETE_AUTOMATION, id);
} catch (error) {
throw new Error(error);
} finally {
commit(types.SET_AUTOMATION_UI_FLAG, { isDeleting: false });
}
},
};
export const mutations = {
[types.SET_AUTOMATION_UI_FLAG](_state, data) {
_state.uiFlags = {
..._state.uiFlags,
...data,
};
},
[types.ADD_AUTOMATION]: MutationHelpers.create,
[types.SET_AUTOMATIONS]: MutationHelpers.set,
// [types.EDIT_AUTOMATION]: MutationHelpers.update,
[types.DELETE_AUTOMATION]: MutationHelpers.destroy,
};
export default {
namespaced: true,
actions,
state,
getters,
mutations,
};

View File

@@ -0,0 +1,94 @@
import axios from 'axios';
import { actions } from '../../automations';
import * as types from '../../../mutation-types';
import automationsList from './fixtures';
const commit = jest.fn();
global.axios = axios;
jest.mock('axios');
describe('#actions', () => {
describe('#get', () => {
it('sends correct actions if API is success', async () => {
axios.get.mockResolvedValue({ data: { payload: automationsList } });
await actions.get({ commit });
expect(commit.mock.calls).toEqual([
[types.default.SET_AUTOMATION_UI_FLAG, { isFetching: true }],
[types.default.SET_AUTOMATIONS, automationsList],
[types.default.SET_AUTOMATION_UI_FLAG, { isFetching: false }],
]);
});
it('sends correct actions if API is error', async () => {
axios.get.mockRejectedValue({ message: 'Incorrect header' });
await actions.get({ commit });
expect(commit.mock.calls).toEqual([
[types.default.SET_AUTOMATION_UI_FLAG, { isFetching: true }],
[types.default.SET_AUTOMATION_UI_FLAG, { isFetching: false }],
]);
});
});
describe('#create', () => {
it('sends correct actions if API is success', async () => {
axios.post.mockResolvedValue({ data: automationsList[0] });
await actions.create({ commit }, automationsList[0]);
expect(commit.mock.calls).toEqual([
[types.default.SET_AUTOMATION_UI_FLAG, { isCreating: true }],
[types.default.ADD_AUTOMATION, automationsList[0]],
[types.default.SET_AUTOMATION_UI_FLAG, { isCreating: false }],
]);
});
it('sends correct actions if API is error', async () => {
axios.post.mockRejectedValue({ message: 'Incorrect header' });
await expect(actions.create({ commit })).rejects.toThrow(Error);
expect(commit.mock.calls).toEqual([
[types.default.SET_AUTOMATION_UI_FLAG, { isCreating: true }],
[types.default.SET_AUTOMATION_UI_FLAG, { isCreating: false }],
]);
});
});
// API Work in progress
// describe('#update', () => {
// it('sends correct actions if API is success', async () => {
// axios.patch.mockResolvedValue({ data: automationsList[0] });
// await actions.update({ commit }, automationsList[0]);
// expect(commit.mock.calls).toEqual([
// [types.default.SET_AUTOMATION_UI_FLAG, { isUpdating: true }],
// [types.default.EDIT_AUTOMATION, automationsList[0]],
// [types.default.SET_AUTOMATION_UI_FLAG, { isUpdating: false }],
// ]);
// });
// it('sends correct actions if API is error', async () => {
// axios.patch.mockRejectedValue({ message: 'Incorrect header' });
// await expect(
// actions.update({ commit }, automationsList[0])
// ).rejects.toThrow(Error);
// expect(commit.mock.calls).toEqual([
// [types.default.SET_AUTOMATION_UI_FLAG, { isUpdating: true }],
// [types.default.SET_AUTOMATION_UI_FLAG, { isUpdating: false }],
// ]);
// });
// });
describe('#delete', () => {
it('sends correct actions if API is success', async () => {
axios.delete.mockResolvedValue({ data: automationsList[0] });
await actions.delete({ commit }, automationsList[0].id);
expect(commit.mock.calls).toEqual([
[types.default.SET_AUTOMATION_UI_FLAG, { isDeleting: true }],
[types.default.DELETE_AUTOMATION, automationsList[0].id],
[types.default.SET_AUTOMATION_UI_FLAG, { isDeleting: false }],
]);
});
it('sends correct actions if API is error', async () => {
axios.delete.mockRejectedValue({ message: 'Incorrect header' });
await expect(
actions.delete({ commit }, automationsList[0].id)
).rejects.toThrow(Error);
expect(commit.mock.calls).toEqual([
[types.default.SET_AUTOMATION_UI_FLAG, { isDeleting: true }],
[types.default.SET_AUTOMATION_UI_FLAG, { isDeleting: false }],
]);
});
});
});

View File

@@ -0,0 +1,64 @@
export default [
{
id: 12,
account_id: 1,
name: 'Test',
description: 'This is a test',
event_name: 'conversation_created',
conditions: [
{
values: ['open'],
attribute_key: 'status',
query_operator: null,
filter_operator: 'equal_to',
},
],
actions: [
{
action_name: 'add_label',
action_params: [{}],
},
],
created_on: '2022-01-14T09:17:55.689Z',
active: true,
},
{
id: 13,
account_id: 1,
name: 'Auto resolve conversation',
description: 'Auto resolves conversation',
event_name: 'conversation_updated',
conditions: [
{
values: ['resolved'],
attribute_key: 'status',
query_operator: null,
filter_operator: 'equal_to',
},
],
actions: [
{
action_name: 'add_label',
action_params: [{}],
},
],
created_on: '2022-01-14T13:06:31.843Z',
active: true,
},
{
id: 14,
account_id: 1,
name: 'Fayaz',
description: 'This is a test',
event_name: 'conversation_created',
conditions: {},
actions: [
{
action_name: 'add_label',
action_params: [{}],
},
],
created_on: '2022-01-17T06:46:08.098Z',
active: true,
},
];

View File

@@ -0,0 +1,25 @@
import { getters } from '../../automations';
import automations from './fixtures';
describe('#getters', () => {
it('getAutomations', () => {
const state = { records: automations };
expect(getters.getAutomations(state)).toEqual(automations);
});
it('getUIFlags', () => {
const state = {
uiFlags: {
isFetching: true,
isCreating: false,
isUpdating: false,
isDeleting: false,
},
};
expect(getters.getUIFlags(state)).toEqual({
isFetching: true,
isCreating: false,
isUpdating: false,
isDeleting: false,
});
});
});

View File

@@ -0,0 +1,58 @@
import types from '../../../mutation-types';
import { mutations } from '../../automations';
import automations from './fixtures';
describe('#mutations', () => {
describe('#SET_automations', () => {
it('set autonmation records', () => {
const state = { records: [] };
mutations[types.SET_AUTOMATIONS](state, automations);
expect(state.records).toEqual(automations);
});
});
describe('#ADD_AUTOMATION', () => {
it('push newly created automatuion to the store', () => {
const state = { records: [automations[0]] };
mutations[types.ADD_AUTOMATION](state, automations[1]);
expect(state.records).toEqual([automations[0], automations[1]]);
});
});
// describe('#EDIT_AUTOMATION', () => {
// it('update automation record', () => {
// const state = { records: [automations[0]] };
// mutations[types.EDIT_AUTOMATION](state, {
// id: 12,
// account_id: 1,
// name: 'Test Automation',
// description: 'This is a test',
// event_name: 'conversation_created',
// conditions: [
// {
// values: ['open'],
// attribute_key: 'status',
// query_operator: null,
// filter_operator: 'equal_to',
// },
// ],
// actions: [
// {
// action_name: 'add_label',
// action_params: [{}],
// },
// ],
// created_on: '2022-01-14T09:17:55.689Z',
// active: true,
// });
// expect(state.records[0].name).toEqual('Test Automation');
// });
// });
describe('#DELETE_AUTOMATION', () => {
it('delete automation record', () => {
const state = { records: [automations[0]] };
mutations[types.DELETE_AUTOMATION](state, 12);
expect(state.records).toEqual([]);
});
});
});