[ADF-4273] Remove description property from Process Instance Cloud Model (#4568)

* [ADF-4273] Remove description property from Process Instance Cloud Model

* [ADF-4273] Remove from e2e tests
This commit is contained in:
davidcanonieto
2019-04-09 16:49:44 +01:00
committed by Eugenio Romano
parent ce0775c525
commit 2753771d29
9 changed files with 1 additions and 46 deletions

View File

@@ -36,7 +36,7 @@ The component populates an internal array of
By default all properties are displayed: By default all properties are displayed:
**_id_**, **_name_**, **_description_**, **_status_**, **_initiator_**, **_startDate_**, **_lastModified_**, **_parentId_**, **_businessKey_**. **_id_**, **_name_**, **_status_**, **_initiator_**, **_startDate_**, **_lastModified_**, **_parentId_**, **_businessKey_**.
However, you can also choose which properties to show using a configuration in `app.config.json`: However, you can also choose which properties to show using a configuration in `app.config.json`:

View File

@@ -87,7 +87,6 @@ describe('Process list cloud', () => {
processCloudDemoPage.processListCloudComponent().getDataTable().checkColumnIsDisplayed('startDate'); processCloudDemoPage.processListCloudComponent().getDataTable().checkColumnIsDisplayed('startDate');
processCloudDemoPage.processListCloudComponent().getDataTable().checkColumnIsDisplayed('appName'); processCloudDemoPage.processListCloudComponent().getDataTable().checkColumnIsDisplayed('appName');
processCloudDemoPage.processListCloudComponent().getDataTable().checkColumnIsDisplayed('businessKey'); processCloudDemoPage.processListCloudComponent().getDataTable().checkColumnIsDisplayed('businessKey');
processCloudDemoPage.processListCloudComponent().getDataTable().checkColumnIsDisplayed('description');
processCloudDemoPage.processListCloudComponent().getDataTable().checkColumnIsDisplayed('initiator'); processCloudDemoPage.processListCloudComponent().getDataTable().checkColumnIsDisplayed('initiator');
processCloudDemoPage.processListCloudComponent().getDataTable().checkColumnIsDisplayed('lastModified'); processCloudDemoPage.processListCloudComponent().getDataTable().checkColumnIsDisplayed('lastModified');
processCloudDemoPage.processListCloudComponent().getDataTable().checkColumnIsDisplayed('processName'); processCloudDemoPage.processListCloudComponent().getDataTable().checkColumnIsDisplayed('processName');

View File

@@ -58,12 +58,6 @@ export class ProcessListCloudConfiguration {
'title': 'ADF_CLOUD_PROCESS_LIST.PROPERTIES.BUSINESS_KEY', 'title': 'ADF_CLOUD_PROCESS_LIST.PROPERTIES.BUSINESS_KEY',
'sortable': true 'sortable': true
}, },
{
'key': 'entry.description',
'type': 'text',
'title': 'ADF_CLOUD_PROCESS_LIST.PROPERTIES.DESCRIPTION',
'sortable': true
},
{ {
'key': 'entry.initiator', 'key': 'entry.initiator',
'type': 'text', 'type': 'text',

View File

@@ -27,7 +27,6 @@ import { ProcessHeaderCloudService } from '../services/process-header-cloud.serv
const processInstanceDetailsCloudMock = { const processInstanceDetailsCloudMock = {
appName: 'app-form-mau', appName: 'app-form-mau',
businessKey: 'MyBusinessKey', businessKey: 'MyBusinessKey',
description: 'new desc',
id: '00fcc4ab-4290-11e9-b133-0a586460016a', id: '00fcc4ab-4290-11e9-b133-0a586460016a',
initiator: 'devopsuser', initiator: 'devopsuser',
lastModified: 1552152187081, lastModified: 1552152187081,
@@ -91,16 +90,6 @@ describe('ProcessHeaderCloudComponent', () => {
}); });
})); }));
it('should display description', async(() => {
component.ngOnChanges();
fixture.detectChanges();
fixture.whenStable().then(() => {
const formNameEl = fixture.debugElement.query(By.css('[data-automation-id="card-textitem-value-description"] span'));
expect(formNameEl.nativeElement.innerText).toBe('new desc');
});
}));
it('should display placeholder if no name is available', async(() => { it('should display placeholder if no name is available', async(() => {
processInstanceDetailsCloudMock.name = null; processInstanceDetailsCloudMock.name = null;
component.ngOnChanges(); component.ngOnChanges();
@@ -113,18 +102,6 @@ describe('ProcessHeaderCloudComponent', () => {
})); }));
it('should display placeholder if no description is available', async(() => {
processInstanceDetailsCloudMock.description = null;
component.ngOnChanges();
fixture.detectChanges();
fixture.whenStable().then(() => {
const valueEl = fixture.debugElement.query(By.css('[data-automation-id="card-textitem-value-description"] span'));
expect(valueEl.nativeElement.innerText).toBe('ADF_CLOUD_PROCESS_HEADER.PROPERTIES.DESCRIPTION_DEFAULT');
});
}));
it('should display status', async(() => { it('should display status', async(() => {
component.ngOnChanges(); component.ngOnChanges();
fixture.detectChanges(); fixture.detectChanges();

View File

@@ -86,13 +86,6 @@ export class ProcessHeaderCloudComponent implements OnChanges {
key: 'name', key: 'name',
default: this.translationService.instant('ADF_CLOUD_PROCESS_HEADER.PROPERTIES.NAME_DEFAULT') default: this.translationService.instant('ADF_CLOUD_PROCESS_HEADER.PROPERTIES.NAME_DEFAULT')
}), }),
new CardViewTextItemModel(
{
label: 'ADF_CLOUD_PROCESS_HEADER.PROPERTIES.DESCRIPTION',
value: this.processInstanceDetails.description,
key: 'description',
default: this.translationService.instant('ADF_CLOUD_PROCESS_HEADER.PROPERTIES.DESCRIPTION_DEFAULT')
}),
new CardViewTextItemModel( new CardViewTextItemModel(
{ {
label: 'ADF_CLOUD_PROCESS_HEADER.PROPERTIES.STATUS', label: 'ADF_CLOUD_PROCESS_HEADER.PROPERTIES.STATUS',

View File

@@ -133,7 +133,6 @@ describe('ProcessListCloudComponent', () => {
expect(component.rows[0].entry['appVersion']).toBe(''); expect(component.rows[0].entry['appVersion']).toBe('');
expect(component.rows[0].entry['id']).toBe('69eddfa7-d781-11e8-ae24-0a58646001fa'); expect(component.rows[0].entry['id']).toBe('69eddfa7-d781-11e8-ae24-0a58646001fa');
expect(component.rows[0].entry['name']).toEqual('starring'); expect(component.rows[0].entry['name']).toEqual('starring');
expect(component.rows[0].entry['description']).toBeNull();
expect(component.rows[0].entry['processDefinitionId']).toBe('BasicProcess:1:d05062f1-c6fb-11e8-ae24-0a58646001fa'); expect(component.rows[0].entry['processDefinitionId']).toBe('BasicProcess:1:d05062f1-c6fb-11e8-ae24-0a58646001fa');
expect(component.rows[0].entry['processDefinitionKey']).toBe('BasicProcess'); expect(component.rows[0].entry['processDefinitionKey']).toBe('BasicProcess');
expect(component.rows[0].entry['initiator']).toBe('devopsuser'); expect(component.rows[0].entry['initiator']).toBe('devopsuser');

View File

@@ -26,7 +26,6 @@ export const fakeProcessCloudList = {
appVersion: '', appVersion: '',
id: '69eddfa7-d781-11e8-ae24-0a58646001fa', id: '69eddfa7-d781-11e8-ae24-0a58646001fa',
name: 'starring', name: 'starring',
description: null,
processDefinitionId: 'BasicProcess:1:d05062f1-c6fb-11e8-ae24-0a58646001fa', processDefinitionId: 'BasicProcess:1:d05062f1-c6fb-11e8-ae24-0a58646001fa',
processDefinitionKey: 'BasicProcess', processDefinitionKey: 'BasicProcess',
initiator: 'devopsuser', initiator: 'devopsuser',
@@ -44,7 +43,6 @@ export const fakeProcessCloudList = {
appVersion: '', appVersion: '',
id: '8b3f625f-d781-11e8-ae24-0a58646001fa', id: '8b3f625f-d781-11e8-ae24-0a58646001fa',
name: null, name: null,
description: null,
processDefinitionId: 'BasicProcess:1:d05062f1-c6fb-11e8-ae24-0a58646001fa', processDefinitionId: 'BasicProcess:1:d05062f1-c6fb-11e8-ae24-0a58646001fa',
processDefinitionKey: 'BasicProcess', processDefinitionKey: 'BasicProcess',
initiator: 'devopsuser', initiator: 'devopsuser',
@@ -62,7 +60,6 @@ export const fakeProcessCloudList = {
appVersion: '', appVersion: '',
id: '87c12637-d783-11e8-ae24-0a58646001fa', id: '87c12637-d783-11e8-ae24-0a58646001fa',
name: null, name: null,
description: null,
processDefinitionId: 'BasicProcess:1:d05062f1-c6fb-11e8-ae24-0a58646001fa', processDefinitionId: 'BasicProcess:1:d05062f1-c6fb-11e8-ae24-0a58646001fa',
processDefinitionKey: 'BasicProcess', processDefinitionKey: 'BasicProcess',
initiator: 'superadminuser', initiator: 'superadminuser',

View File

@@ -19,7 +19,6 @@ import { ProcessListCloudSortingModel } from './process-list-sorting.model';
export class ProcessQueryCloudRequestModel { export class ProcessQueryCloudRequestModel {
appName: string; appName: string;
description?: string;
initiator?: null; initiator?: null;
id?: string; id?: string;
name?: string; name?: string;
@@ -37,7 +36,6 @@ export class ProcessQueryCloudRequestModel {
constructor(obj?: any) { constructor(obj?: any) {
if (obj) { if (obj) {
this.appName = obj.appName; this.appName = obj.appName;
this.description = obj.description;
this.initiator = obj.initiator; this.initiator = obj.initiator;
this.id = obj.id; this.id = obj.id;
this.name = obj.name; this.name = obj.name;

View File

@@ -19,7 +19,6 @@ export class ProcessInstanceCloud {
appName: string; appName: string;
id: string; id: string;
name: string; name: string;
description: string;
startDate: Date; startDate: Date;
initiator: string; initiator: string;
status: string; status: string;
@@ -33,7 +32,6 @@ export class ProcessInstanceCloud {
this.appName = obj && obj.appName || null; this.appName = obj && obj.appName || null;
this.id = obj && obj.id || null; this.id = obj && obj.id || null;
this.name = obj && obj.name || null; this.name = obj && obj.name || null;
this.description = obj && obj.description || null;
this.startDate = obj && obj.startDate || null; this.startDate = obj && obj.startDate || null;
this.initiator = obj && obj.initiator || null; this.initiator = obj && obj.initiator || null;
this.status = obj && obj.status || null; this.status = obj && obj.status || null;