[AAE-5343] add update person api (#7426)

* add update person api * fix e2e tests
Alfresco · Dec 16, 2021 · 65a9d2b · 65a9d2b
1 parent 75adf21
commit 65a9d2b
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 3 deletions.
diff --git a/e2e/process-services/form/apps-section.e2e.ts b/e2e/process-services/form/apps-section.e2e.ts
@@ -132,7 +132,9 @@ describe('Modify applications', () => {
                     'models': [firstApp.definition.models[0]], 'theme': 'theme-4',
                     'icon': 'glyphicon-user'
                 }
-            }, 'publish': true
+            },
+            'publish': true,
+            'force': true
         };
 
         await appsApi.updateAppDefinition(appVersionToBeDeleted.id, appDefinition);

diff --git a/lib/core/services/people-content.service.ts b/lib/core/services/people-content.service.ts
@@ -19,7 +19,7 @@ import { Injectable } from '@angular/core';
 import { Observable, from, throwError } from 'rxjs';
 import { AlfrescoApiService } from './alfresco-api.service';
 import { catchError, map } from 'rxjs/operators';
-import { PersonEntry, PeopleApi, PersonBodyCreate, Pagination } from '@alfresco/js-api';
+import { PersonEntry, PeopleApi, PersonBodyCreate, Pagination, PersonBodyUpdate } from '@alfresco/js-api';
 import { EcmUserModel } from '../models/ecm-user.model';
 import { LogService } from './log.service';
 
@@ -117,6 +117,20 @@ export class PeopleContentService {
         );
     }
 
+    /**
+     * Updates the person details
+     * @param personId The identifier of a person
+     * @param details The person details
+     * @param opts Optional parameters
+     * @returns Updated person model
+     */
+    updatePerson(personId: string, details: PersonBodyUpdate, opts?: any): Observable<EcmUserModel> {
+        return from(this.peopleApi.updatePerson(personId, details, opts)).pipe(
+            map((res: PersonEntry) => <EcmUserModel> res?.entry),
+            catchError((error) => this.handleError(error))
+        );
+    }
+
     async isContentAdmin(): Promise<boolean> {
         if (!this.hasCheckedIsContentAdmin) {
             const user: PersonEntry = await this.getCurrentPerson().toPromise();

diff --git a/...c/lib/protractor/process-services-cloud/pages/edit-process-filter-cloud-component.page.ts b/...c/lib/protractor/process-services-cloud/pages/edit-process-filter-cloud-component.page.ts
@@ -113,14 +113,15 @@ export class EditProcessFilterCloudComponentPage {
 
     async setSortFilterDropDown(option) {
         await this.sortDropdown.selectDropdownOption(option);
+        await browser.sleep(500);
     }
 
     async getSortFilterDropDownValue(): Promise<string> {
         const sortLocator = $$("mat-form-field[data-automation-id='sort'] span").first();
         return BrowserActions.getText(sortLocator);
     }
 
-    async setOrderFilterDropDown(option) {
+    async setOrderFilterDropDown(option: string) {
         await this.orderDropdown.selectDropdownOption(option);
         await browser.sleep(1500);
     }