yujunjun/strapi
1
0
Fork 0
mirror of https://github.com/strapi/strapi.git synced 2025-09-02 13:23:12 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add test for adding a compo and a dz

Browse Source
This commit is contained in:
soupette 2020-01-14 12:31:39 +01:00
parent 0a2dffc1d4
commit c577097994
2 changed files with 230 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
packages/strapi-plugin-content-type-builder/admin/src/containers/DataManagerProvider/reducer.js
View File

@ -75,6 +75,7 @@ const reducer = (state, action) => {
forTarget,
targetUid,
} = action;
console.log(action);
delete rest.createComponent;
const pathToDataToEdit = ['component', 'contentType'].includes(forTarget)

235
packages/strapi-plugin-content-type-builder/admin/src/containers/DataManagerProvider/tests/reducer_add_attribute_action.test.js
View File

@ -104,21 +104,244 @@ describe('CTB | containers | reducer | ADD_ATTRIBUTE', () => {
describe('Adding a component field attribute', () => {
it('Should create the component attribute and add the component to the modifiedData.components if the component is not in the object', () => {
expect(true).toBe(true);
const contentTypeUID = 'application::address.address';
const contentType = get(testData, ['contentTypes', contentTypeUID]);
const componentToAddUID = 'default.dish';
const state = initialState
.set('components', fromJS(testData.components))
.set('initialComponents', fromJS(testData.components))
.set('contentTypes', fromJS(testData.contentTypes))
.set('initialContentTypes', fromJS(testData.contentTypes))
.setIn(['modifiedData', 'contentType'], fromJS(contentType))
.setIn(['modifiedData', 'components'], fromJS({}));
const action = {
type: 'ADD_ATTRIBUTE',
attributeToSet: {
type: 'component',
repeatable: true,
name: 'compoField',
component: componentToAddUID,
required: true,
max: 2,
min: 1,
},
forTarget: 'contentType',
targetUid: 'application::address.address',
initialAttribute: {},
shouldAddComponentToData: true,
};
const expected = state
.setIn(
['modifiedData', 'components', componentToAddUID],
fromJS(testData.components[componentToAddUID])
)
.setIn(
['modifiedData', 'contentType', 'schema', 'attributes', 'compoField'],
fromJS({
type: 'component',
repeatable: true,
component: componentToAddUID,
required: true,
max: 2,
min: 1,
})
);
expect(reducer(state, action)).toEqual(expected);
});
it('Should create the component attribute and not add the component to the modifiedData.components if the component is already in the object to keep the modifications', () => {
expect(true).toBe(true);
it('Should create the component attribute and add the component to the modifiedData.components and its nested components if none of the added components are in the object', () => {
const contentTypeUID = 'application::address.address';
const contentType = get(testData, ['contentTypes', contentTypeUID]);
const componentToAddUID = 'default.closingperiod';
const state = initialState
.set('components', fromJS(testData.components))
.set('initialComponents', fromJS(testData.components))
.set('contentTypes', fromJS(testData.contentTypes))
.set('initialContentTypes', fromJS(testData.contentTypes))
.setIn(['modifiedData', 'contentType'], fromJS(contentType))
.setIn(['modifiedData', 'components'], fromJS({}));
const action = {
type: 'ADD_ATTRIBUTE',
attributeToSet: {
type: 'component',
repeatable: true,
name: 'compoField',
component: componentToAddUID,
required: true,
max: 2,
min: 1,
},
forTarget: 'contentType',
targetUid: 'application::address.address',
initialAttribute: {},
shouldAddComponentToData: true,
};
const expected = state
.setIn(
['modifiedData', 'components', componentToAddUID],
fromJS(testData.components[componentToAddUID])
)
.setIn(
['modifiedData', 'components', 'default.dish'],
fromJS(testData.components['default.dish'])
)
.setIn(
['modifiedData', 'contentType', 'schema', 'attributes', 'compoField'],
fromJS({
type: 'component',
repeatable: true,
component: componentToAddUID,
required: true,
max: 2,
min: 1,
})
);
expect(reducer(state, action)).toEqual(expected);
});
it('Should create the component correctly in case the component is created on the fly', () => {
expect(true).toBe(true);
it('Should create the component attribute and add the component to the modifiedData.components and only add the nested components that are not in components are in the object to keep previous the modifications', () => {
const contentTypeUID = 'application::address.address';
const contentType = get(testData, ['contentTypes', contentTypeUID]);
const componentToAddUID = 'default.closingperiod';
const state = initialState
.set('components', fromJS(testData.components))
.set('initialComponents', fromJS(testData.components))
.set('contentTypes', fromJS(testData.contentTypes))
.set('initialContentTypes', fromJS(testData.contentTypes))
.setIn(['modifiedData', 'contentType'], fromJS(contentType))
.setIn(
['modifiedData', 'components', 'default.dish'],
fromJS(testData.components['default.dish'])
);
const action = {
type: 'ADD_ATTRIBUTE',
attributeToSet: {
type: 'component',
repeatable: true,
name: 'compoField',
component: componentToAddUID,
required: true,
max: 2,
min: 1,
},
forTarget: 'contentType',
targetUid: contentTypeUID,
initialAttribute: {},
shouldAddComponentToData: true,
};
const expected = state
.setIn(
['modifiedData', 'components', componentToAddUID],
fromJS(testData.components[componentToAddUID])
)
.setIn(
['modifiedData', 'contentType', 'schema', 'attributes', 'compoField'],
fromJS({
type: 'component',
repeatable: true,
component: componentToAddUID,
required: true,
max: 2,
min: 1,
})
);
expect(reducer(state, action)).toEqual(expected);
});
it('Should create the component correctly in case of creating the component on the fly', () => {
const componentToCreateUID = 'default.new-compo';
const componentToCreate = {
uid: componentToCreateUID,
isTemporary: true,
category: 'default',
schema: {
name: 'newCompo',
icon: 'ad',
attributes: {},
},
};
const contentTypeUID = 'application::address.address';
const action = {
type: 'ADD_ATTRIBUTE',
attributeToSet: {
name: 'newCompo',
type: 'component',
repeatable: false,
component: componentToCreateUID,
},
forTarget: 'contentType',
targetUid: contentTypeUID,
initialAttribute: undefined,
shouldAddComponentToData: false,
};
const state = initialState
.setIn(['components', componentToCreateUID], fromJS(componentToCreate))
.setIn(
['modifiedData', 'components', componentToCreateUID],
fromJS(componentToCreate)
)
.setIn(
['modifiedData', 'contentType'],
fromJS(testData.contentTypes[contentTypeUID])
);
const expected = state.setIn(
['modifiedData', 'contentType', 'schema', 'attributes', 'newCompo'],
fromJS({
type: 'component',
repeatable: false,
component: componentToCreateUID,
})
);
expect(reducer(state, action)).toEqual(expected);
});
});
describe('Adding a dynamic zone', () => {
it('Should create the dynamiczone attribute correctly', () => {
expect(true).toBe(true);
const contentTypeUID = 'application::address.address';
const action = {
type: 'ADD_ATTRIBUTE',
attributeToSet: {
type: 'dynamiczone',
components: [],
name: 'dz',
},
forTarget: 'contentType',
targetUid: contentTypeUID,
initialAttribute: {},
shouldAddComponentToData: false,
};
const state = initialState.setIn(
['modifiedData', 'contentType'],
fromJS(testData.contentTypes[contentTypeUID])
);
const expected = state.setIn(
['modifiedData', 'contentType', 'schema', 'attributes', 'dz'],
fromJS({
type: 'dynamiczone',
components: [],
})
);
expect(reducer(state, action)).toEqual(expected);
});
});
});