Removing implicitFlow in favor of codeFlow

lib/core/src/lib/app-config/app-config.service.ts

@@ -63,7 +63,6 @@ export enum Status {
     providedIn: 'root'
 })
 export class AppConfigService {
-
     config: any = {
         application: {
             name: 'Alfresco ADF Application'
@@ -97,11 +96,10 @@ export class AppConfigService {
      * @returns Property value, when loaded
      */
     select(property: string): Observable<any> {
-        return this.onLoadSubject
-            .pipe(
-                map((config) => ObjectUtils.getValue(config, property)),
-                distinctUntilChanged()
-            );
+        return this.onLoadSubject.pipe(
+            map((config) => ObjectUtils.getValue(config, property)),
+            distinctUntilChanged()
+        );
     }
 
     /**
@@ -170,9 +168,7 @@ export class AppConfigService {
     protected onDataLoaded() {
         this.onLoadSubject.next(this.config);
 
-        this.extensionService.setup$
-            .pipe(take(1))
-            .subscribe((config) => this.onExtensionsLoaded(config));
+        this.extensionService.setup$.pipe(take(1)).subscribe((config) => this.onExtensionsLoaded(config));
     }
 
     protected onExtensionsLoaded(config: ExtensionConfig) {
@@ -227,20 +223,18 @@ export class AppConfigService {
      * @param hostIdp host address
      * @returns Discovery configuration
      */
-     loadWellKnown(hostIdp: string): Promise<OpenidConfiguration> {
+    loadWellKnown(hostIdp: string): Promise<OpenidConfiguration> {
         return new Promise((resolve, reject) => {
-            this.http
-                .get<OpenidConfiguration>(`${hostIdp}/.well-known/openid-configuration`)
-                .subscribe({
-                    next: (res: OpenidConfiguration) => {
-                        resolve(res);
-                    },
-                    error: (err: any) => {
-                        // eslint-disable-next-line no-console
-                        console.error('hostIdp not correctly configured or unreachable');
-                        reject(err);
-                    }
-                });
+            this.http.get<OpenidConfiguration>(`${hostIdp}/.well-known/openid-configuration`).subscribe({
+                next: (res: OpenidConfiguration) => {
+                    resolve(res);
+                },
+                error: (err: any) => {
+                    // eslint-disable-next-line no-console
+                    console.error('hostIdp not correctly configured or unreachable');
+                    reject(err);
+                }
+            });
         });
     }
 
@@ -251,13 +245,11 @@ export class AppConfigService {
      */
     get oauth2(): OauthConfigModel {
         const config = this.get(AppConfigValues.OAUTHCONFIG, {});
-        const implicitFlow = config['implicitFlow'] === true || config['implicitFlow'] === 'true';
         const silentLogin = config['silentLogin'] === true || config['silentLogin'] === 'true';
         const codeFlow = config['codeFlow'] === true || config['codeFlow'] === 'true';
 
         return {
             ...(config as OauthConfigModel),
-            implicitFlow,
             silentLogin,
             codeFlow
         };
@@ -273,5 +265,4 @@ export class AppConfigService {
 
         return result;
     }
-
 }

lib/testing/package.json

@@ -11,7 +11,7 @@
     "url": "https://github.com/Alfresco/alfresco-ng2-components/issues"
   },
   "peerDependencies": {
-    "@alfresco/js-api": ">=7.9.0"
+    "@alfresco/js-api": ">=7.9.0-0"
   },
   "keywords": [
     "testing",

lib/testing/src/lib/protractor/core/pages/settings.page.ts

@@ -32,8 +32,8 @@ export class SettingsPage {
     silentLoginToggleLabel = $(`${materialLocators.Slide.toggle.root}[formcontrolname="silentLogin"] label`);
     silentLoginToggleElement = $(`${materialLocators.Slide.toggle.root}[formcontrolname="silentLogin"]`);
     implicitFlowLabel = $(`${materialLocators.Slide.toggle.root}[formcontrolname="implicitFlow"] label`);
-    implicitFlowElement = $(`${materialLocators.Slide.toggle.root}[formcontrolname="implicitFlow"]`);
     codeFlowElement = $(`${materialLocators.Slide.toggle.root}[formcontrolname="codeFlow"]`);
+    codeFlowLabel = $(`${materialLocators.Slide.toggle.root}[formcontrolname="codeFlow"] label`);
     applyButton = $('button[data-automation-id="settings-apply-button"]');
     providerDropdown = new DropdownPage($(`${materialLocators.Select.root}[id="adf-provider-selector"]`));
 
@@ -64,10 +64,9 @@ export class SettingsPage {
         authHost,
         identityHost,
         silentLogin = true,
-        implicitFlow = true,
+        codeFlow = true,
         clientId?: string,
-        logoutUrl: string = '/logout',
-        codeFlow = true
+        logoutUrl: string = '/logout'
     ) {
         await this.goToSettingsPage();
         await this.setProvider('ECM');
@@ -75,9 +74,8 @@ export class SettingsPage {
         await this.setContentServicesURL(contentServiceURL);
         await this.setAuthHost(authHost);
         await this.setIdentityHost(identityHost);
-        await this.setSilentLogin(silentLogin);
-        await this.setImplicitFlow(implicitFlow);
         await this.setCodeFlow(codeFlow);
+        await this.setSilentLogin(silentLogin);
         await this.setClientId(clientId);
         await this.setLogoutUrl(logoutUrl);
         await this.clickApply();
@@ -89,7 +87,7 @@ export class SettingsPage {
         authHost,
         identityHost,
         silentLogin = true,
-        implicitFlow = true,
+        codeFlow = true,
         clientId?: string,
         logoutUrl: string = '/logout'
     ) {
@@ -99,9 +97,8 @@ export class SettingsPage {
         await this.setContentServicesURL(contentServiceURL);
         await this.setAuthHost(authHost);
         await this.setIdentityHost(identityHost);
+        await this.setCodeFlow(codeFlow);
         await this.setSilentLogin(silentLogin);
-        await this.setCodeFlow(false);
-        await this.setImplicitFlow(implicitFlow);
         await this.setClientId(clientId);
         await this.setLogoutUrl(logoutUrl);
         await this.clickApply();
@@ -143,23 +140,14 @@ export class SettingsPage {
         }
     }
 
-    async setImplicitFlow(enableToggle) {
-        await BrowserVisibility.waitUntilElementIsVisible(this.implicitFlowElement);
-
-        const isChecked = (await BrowserActions.getAttribute(this.implicitFlowElement, 'class')).includes(materialLocators.Slide.toggle.checked);
-
-        if ((isChecked && !enableToggle) || (!isChecked && enableToggle)) {
-            await BrowserActions.click(this.implicitFlowLabel);
-        }
-    }
-
     async setCodeFlow(enableToggle) {
         await BrowserVisibility.waitUntilElementIsVisible(this.codeFlowElement);
 
-        const isChecked = (await BrowserActions.getAttribute(this.codeFlowElement, 'class')).includes(materialLocators.Checked.root);
+        const classElements = await BrowserActions.getAttribute(this.codeFlowElement, 'class');
+        const isChecked = classElements.includes(materialLocators.Slide.toggle.checked);
 
         if ((isChecked && !enableToggle) || (!isChecked && enableToggle)) {
-            await BrowserActions.click(this.codeFlowElement);
+            await BrowserActions.click(this.codeFlowLabel);
         }
     }
 }