tsai
/
grapesjs
mirror of https://github.com/artf/grapesjs.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity
Browse Source

Format

pull/6419/head
mohamedsalem401 2 years ago
parent
72f4d81581
commit
e5fcba3eb8
2 changed files with 24 additions and 29 deletions
Whitespace
Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL
Split View
Diff Options
Show Stats Download Patch File Download Diff File
  1. 16
      packages/core/src/data_sources/model/data_collection/ComponentDataCollection.ts
  2. 37
      packages/core/test/specs/data_sources/model/data_collection/ComponentDataCollection.getters-setters.ts

16
packages/core/src/data_sources/model/data_collection/ComponentDataCollection.ts
View File

@ -8,19 +8,14 @@ import DataResolverListener from '../DataResolverListener';
import DataSource from '../DataSource';
import DataVariable, { DataVariableProps, DataVariableType } from '../DataVariable';
import { isDataVariable } from '../utils';
import {
DataCollectionType,
keyCollectionDefinition,
keyCollectionsStateMap,
keyIsCollectionItem
} from './constants';
import { DataCollectionType, keyCollectionDefinition, keyCollectionsStateMap, keyIsCollectionItem } from './constants';
import {
ComponentDataCollectionProps,
DataCollectionConfig,
DataCollectionDataSource,
DataCollectionProps,
DataCollectionState,
DataCollectionStateMap
DataCollectionStateMap,
} from './types';
import { getSymbolsToUpdate } from '../../../dom_components/model/SymbolUtils';
import { StyleProps, UpdateStyleOptions } from '../../../domain_abstract/model/StyleableModel';
@ -113,13 +108,12 @@ export default class ComponentDataCollection extends Component {
setDataSource(dataSource: DataCollectionDataSource) {
this.set(keyCollectionDefinition, {
...this.collectionDef,
collectionConfig: { ...this.collectionConfig, dataSource }
collectionConfig: { ...this.collectionConfig, dataSource },
});
}
private getDataSourceItems() {
return this.collectionDef?.collectionConfig ?
getDataSourceItems(this.collectionConfig.dataSource, this.em) : [];
return this.collectionDef?.collectionConfig ? getDataSourceItems(this.collectionConfig.dataSource, this.em) : [];
}
private getCollectionStateMap() {
@ -153,7 +147,7 @@ export default class ComponentDataCollection extends Component {
this.dataSourceWatcher = new DataResolverListener({
em,
resolver: new DataVariable({ type: DataVariableType, path }, { em }),
onUpdate: this.rebuildChildrenFromCollection
onUpdate: this.rebuildChildrenFromCollection,
});
}

37
packages/core/test/specs/data_sources/model/data_collection/ComponentDataCollection.getters-setters.ts
View File

@ -3,7 +3,6 @@ import { DataVariableType } from '../../../../../src/data_sources/model/DataVari
import {
DataCollectionType,
DataCollectionVariableType,
} from '../../../../../src/data_sources/model/data_collection/constants';
import { DataCollectionStateVariableType } from '../../../../../src/data_sources/model/data_collection/types';
import EditorModel from '../../../../../src/editor/model/Editor';
@ -12,7 +11,6 @@ import ComponentDataCollection from '../../../../../src/data_sources/model/data_
describe('Collection component getters and setters', () => {
let em: EditorModel;
let editor: Editor;
let dsm: DataSourceManager;
let dataSource: DataSource;
let wrapper: Component;
@ -20,7 +18,7 @@ describe('Collection component getters and setters', () => {
let secondRecord: DataRecord;
beforeEach(() => {
({ em, editor, dsm } = setupTestEditor());
({ em, dsm } = setupTestEditor());
wrapper = em.getWrapper()!;
dataSource = dsm.add({
id: 'my_data_source_id',
@ -60,7 +58,6 @@ describe('Collection component getters and setters', () => {
},
},
},
],
},
collectionConfig: {
@ -90,6 +87,7 @@ describe('Collection component getters and setters', () => {
test('getComponentDef should return the correct component definition', () => {
const componentDef = cmp.getComponentDef();
expect(componentDef.type).toBe('default');
expect(componentDef.components).toHaveLength(1);
expect(componentDef?.components?.[0].attributes?.['dataUser']).toEqual({
@ -102,10 +100,10 @@ describe('Collection component getters and setters', () => {
test('getDataSource should return the correct data source', () => {
const ds = cmp.getDataSource();
expect(ds).toEqual({
type: DataVariableType,
path: 'my_data_source_id',
});
});
@ -122,12 +120,14 @@ describe('Collection component getters and setters', () => {
test('getConfigStartIndex should handle zero as a valid start index', () => {
cmp.setStartIndex(0);
expect(cmp.getConfigStartIndex()).toBe(0);
expect(cmp.getItemsCount()).toBe(3);
});
test('getConfigEndIndex should handle zero as a valid end index', () => {
cmp.setEndIndex(0);
expect(cmp.getConfigStartIndex()).toBe(1);
expect(cmp.getConfigEndIndex()).toBe(0);
expect(cmp.getItemsCount()).toBe(0);
@ -153,11 +153,9 @@ describe('Collection component getters and setters', () => {
variableType: DataCollectionStateVariableType.currentItem,
collectionId: 'my_collection',
path: 'user',
},
},
},
],
},
collectionConfig: {
@ -167,7 +165,6 @@ describe('Collection component getters and setters', () => {
dataSource: {
type: DataVariableType,
path: 'my_data_source_id',
},
},
},
@ -187,14 +184,13 @@ describe('Collection component getters and setters', () => {
variableType: DataCollectionStateVariableType.currentItem,
collectionId: 'my_collection',
path: 'firstName',
},
},
},
],
};
cmp.setComponentDef(newComponentDef);
const children = cmp.components();
expect(children).toHaveLength(2);
expect(children.at(0).get('type')).toBe('newType');
@ -205,6 +201,7 @@ describe('Collection component getters and setters', () => {
test('setStartIndex should update the start index and reflect in children', () => {
cmp.setStartIndex(0);
expect(cmp.getConfigStartIndex()).toBe(0);
const children = cmp.components();
expect(children).toHaveLength(3);
expect(children.at(0).components().at(0).getAttributes()['dataUser']).toBe('user1');
@ -215,6 +212,7 @@ describe('Collection component getters and setters', () => {
test('setEndIndex should update the end index and reflect in children', () => {
cmp.setEndIndex(3);
expect(cmp.getConfigEndIndex()).toBe(3);
const children = cmp.components();
expect(children).toHaveLength(2);
expect(children.at(0).components().at(0).getAttributes()['dataUser']).toBe('user2');
@ -226,7 +224,7 @@ describe('Collection component getters and setters', () => {
id: 'new_data_source_id',
records: [
{ id: 'user4', user: 'user4', firstName: 'Name4', age: '20' },
{ id: 'user5', user: 'user5', firstName: 'Name5', age: '21' }
{ id: 'user5', user: 'user5', firstName: 'Name5', age: '21' },
],
});
@ -242,6 +240,7 @@ describe('Collection component getters and setters', () => {
test('setStartIndex with zero should include the first record', () => {
cmp.setStartIndex(0);
const children = cmp.components();
expect(children).toHaveLength(3);
expect(children.at(0).components().at(0).getAttributes()['dataUser']).toBe('user1');
@ -249,6 +248,7 @@ describe('Collection component getters and setters', () => {
test('setEndIndex with zero should result in no children', () => {
cmp.setEndIndex(0);
const children = cmp.components();
expect(children).toHaveLength(0);
});
@ -257,13 +257,13 @@ describe('Collection component getters and setters', () => {
dsm.add({
id: 'empty_data_source_id',
records: [],
});
cmp.setDataSource({
type: DataVariableType,
path: 'empty_data_source_id',
});
const children = cmp.components();
expect(children).toHaveLength(0);
});
@ -271,6 +271,7 @@ describe('Collection component getters and setters', () => {
describe('Impact on HTML output', () => {
let cmp: ComponentDataCollection;
beforeEach(() => {
cmp = wrapper.components({
type: DataCollectionType,
@ -287,11 +288,9 @@ describe('Collection component getters and setters', () => {
variableType: DataCollectionStateVariableType.currentItem,
collectionId: 'my_collection',
path: 'user',
},
},
},
],
},
collectionConfig: {
@ -301,7 +300,6 @@ describe('Collection component getters and setters', () => {
dataSource: {
type: DataVariableType,
path: 'my_data_source_id',
},
},
},
@ -310,6 +308,7 @@ describe('Collection component getters and setters', () => {
test('HTML output should reflect changes in startIndex', () => {
cmp.setStartIndex(0);
const html = cmp.toHTML();
expect(html).toContain('dataUser="user1"');
expect(html).toContain('dataUser="user2"');
@ -318,6 +317,7 @@ describe('Collection component getters and setters', () => {
test('HTML output should reflect changes in endIndex', () => {
cmp.setEndIndex(3);
const html = cmp.toHTML();
expect(html).toContain('dataUser="user2"');
expect(html).toContain('dataUser="user3"');
@ -328,7 +328,7 @@ describe('Collection component getters and setters', () => {
id: 'new_data_source_id',
records: [
{ id: 'user4', user: 'user4', firstName: 'Name4', age: '20' },
{ id: 'user5', user: 'user5', firstName: 'Name5', age: '21' }
{ id: 'user5', user: 'user5', firstName: 'Name5', age: '21' },
],
});
cmp.setDataSource({
@ -342,8 +342,9 @@ describe('Collection component getters and setters', () => {
test('HTML output should be empty when endIndex is zero', () => {
cmp.setEndIndex(0);
const html = cmp.toHTML();
expect(html).not.toContain('dataUser');
});
});
});
});

Write Preview
Loading…
Cancel
Save